Skip to content
Submitted about 2 years ago

Mobile first CSS HTML pure JS training layout management (grid & flex)

y4rb0w•260
@Yavanha
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...E-commerce product page
  • HTML
  • CSS
  • JS
3intermediate

Design comparison

SolutionDesign

Solution retrospective

Hello everyone,

Finally my first intermediate project, it was very instructive.

Please if you want to give me any feedback, I would appriciate if you mainly focus on HTML & CSS. That's what I'm currently working on.

The JS I did something very quickly to make it work, can be improve a lot :).

Thank you

Community feedback

  • Joshua Hellard•545
    @jhellardPosted about 2 years ago

    Hey y4rb0w! Your solution looks great! The HTML looks good, great use of the semantic tags. Everything looks super close to the design, the lightbox functions very well. I love the animated shopping cart, really nice touch. I did notice when scaling down the screen size the header starts to overlap before hitting the mobile breakpoint, I'd take a look at that.

    The carousel-thumbnails div wraps before hitting that breakpoint causing one of those images to take full width.

    Awesome work, keep on coding!

    Marked as helpful
    1
    y4rb0w•260
    @YavanhaPosted about 2 years ago

    @jhellard It's comments like this that give me the energy to continue :). Thank you

    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