@EmmanuelOloke
Posted
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.
