@En-Jen
Posted
Hey Giovanni, really nice job with this challenge! I really like the animations and the hover state on the buttons and it looks really close to the design :) I just have a few suggestions for you:
-
I would try to stray away from using id's as your CSS selectors and instead try to always use classes. You can read more about the rationale for that in this article (https://paulcpederson.com/articles/css-for-people-who-hate-css/)
-
It would be nice to see some change in the size of the elements/layout for tablet widths. At just under 1200px screen width, it looks like there is too much whitespace.
-
You might consider adding a max-width to the container because at large desktop screen widths, the text and the image get really far apart.
-Jen
@GiovanniRanzato
Posted
@En-Jen Hello Jen thanks for your advice! I will improve the design for sure.