Skip to content
Submitted almost 2 years ago

Base apparel

Eliseu Caputo•110
@eliseucaputo
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Base Apparel coming soon page
  • HTML
  • CSS
  • JS
1newbie

Design comparison

SolutionDesign

Solution retrospective

Good evening!

I'm sending another project, a little easier but just to train the javascript more. I know it's not perfect and I know I could write more code in javascript but I'm evolving a lot.

Any comments are welcome and help a lot!

Community feedback

  • Sinisa Vukmirovic•1,690
    @SinisaVukmirovicPosted almost 2 years ago

    Hello!

    Your folder structure could be better organized.

    Also, your use of semantic elements is not quite right. There should really be <main> element instead of that first <section class="container">. Plus, no <section> should be inside of another <section>. Thats not a proper way of using section elements.

    More about semantic elements

    Hope this helps! All the best!

    Marked as helpful
    1
    Eliseu Caputo•110
    @eliseucaputoPosted almost 2 years ago

    @SinisaVukmirovic good afternoon friend! Thanks for the comment. Regarding the Main tag I completely forgot. About the section tag I didn't know that. I will study about it. Thank you very much.

    1

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