Skip to content
Submitted over 2 years ago

E-Comerce Page

Eddy•60
@EddyBess
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

I am still working on the mobile version , but any advice the projet or my code is welcome.

Community feedback

  • Zak Schenck•40
    @ZakSchenckPosted over 2 years ago

    Hey @EddyBess ! Good job so far. I have a few tips from a design/functionality standpoint. First, when I click on your cart, I'm unable to toggle the menu to close it. You can set a toggle classlist to the cart icon and easily add that. Alternatively, you can add an X button on it which is even better. I'd also add more spacing between the Nav Items, as well as the item counter. Makes the site less cluttered and have a better feel. The last suggestion I have which is the least important since you have an X button: In your lightbox it's always a good touch to be able to click anywhere on the page to get rid of your lightbox. In this instance, clicking on the black opacity background.

    Again, good job man. I hope these were helpful in some way. Cheers!

    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