Skip to content
Submitted over 3 years ago

HTML, CSS, JavaScript

joni•365
@joni475
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Arch Studio multi-page website
  • HTML
  • CSS
  • JS
4advanced

Design comparison

SolutionDesign

Solution retrospective

Any comment would be very appreciated. Thank You.

Community feedback

  • Roc Tanweer•2,500
    @RocTanweerPosted over 3 years ago

    Wonderful! The effect you gave on hovering the nav links was awesome! The automatic carousel effect was also awesome but when I am clicking on the numbers to see the first slide, for example, I don't know why but it seems like the automation of carousel gets broken. Unlike before, it's changing the slide without any delay...

    Beside this, everything seems cool! Hope it helps! Happy coding!

    0
    joni•365
    @joni475Posted over 3 years ago

    @RocTanweer Thanks.

    0
  • Raymart Pamplona•16,100
    @pikapikamartPosted over 3 years ago

    Hey some issues about your work.

    1. The slider just goes from normal speed to super fast when I click a slider button. I check your js, the timeout of 4000s is okay, i'm guessing that is caused by the event listener, the onclick that you put on those spans, when it is clicked, triggers the function, but, knowing that is is recursion, an event overlaps the functions then triggers both 2 at the same time, and I click 3 times. It runs 3 simultaneously. Refactoring that will make it good.
    2. Paddings. There are sections where padding is not consistent, example is the footer. From the main page compared to the about page, it changes, from its left padding.
    3. In the contact page, a scrollbar from bottom is visible. This was cause by the text in the .tell-us class. It's width overlaps the body's width and therefore, a scrollbar appears at the bottom
    0
    joni•365
    @joni475Posted over 3 years ago

    @pikamart Thanks for checking it out, I will try to fix it.

    0
    Raymart Pamplona•16,100
    @pikapikamartPosted over 3 years ago

    @joni475 Remember to commit it so that we can see it as well^

    0
    joni•365
    @joni475Posted over 3 years ago

    @pikamart Will do and thank you for taking time to check my code.

    0
    Raymart Pamplona•16,100
    @pikapikamartPosted over 3 years ago

    @joni475 Well everyone helps everyone here^

    0
  • Tushar Mehra•470
    @tushar416Posted over 3 years ago

    AWESOME!!!!

    0
    joni•365
    @joni475Posted over 3 years ago

    @tushar416 Thanks

    0

Please log in to post a comment

Log in with GitHub

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord