Skip to content
Submitted about 2 years ago

E-commerce website with react , included bootstrap and material-UI .

Rohit•240
@Halix7
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

This is litterally a challange for me but learned a lot whenever I faced some problems .

Community feedback

  • Eugenia•950
    @JaneMorozPosted about 2 years ago

    Hey,I really like your solution!

    The only things I've noticed straight away:

    • You need to wrap the main content of the page into the <main> tag. It will solve all this landmark issues. You can read more here.
    • I've notice that cart, minus and plus icons are just images, you need to wrap them in <button> tag, because they are supposed to be buttons ☺️ It'll improve accessibility. I think that bootstrap has some classes for buttons with icons.
    • Also 'next' and 'previous' buttons are not working in the lightbox.

    Keep it up! And good luck 🍀

    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