ecommerce-product-page-main

Solution retrospective
all feedback is good :)
Please log in to post a comment
Log in with GitHubCommunity feedback
- @EmmanuelOloke
Hello @CaioPaulin0, great work on this challenge, especially the mobile view. Looks really clean, however, there are some observations I've made that I'd like to bring to your notice:
-
The contents are bigger than the provided design file recommendations. Just a few adjustments in your HTML and CSS will fix this.
-
Clicking the thumbnails on the main page opens up the modal which I don't think conforms with the intended performance, it should instead change the main image to the larger version of the thumbnail image clicked. Also, the active state on the clicked thumbnail isn't implemented.
-
On the modal, there are no next and previous buttons. Also, the active state of the selected image in the thumbnail isn't implemented.
These are just some observations and improvements I think can make the solution look even better. Once again, great work on the challenge. Thanks.
-
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