Annalisa• 170
@Annalisa11
Posted
Your solution is well done :) The html is well structured with the right use of semantic html tags and the overall look and feel is perfect. I especially like the transition you added on the hover of the image, nice touch!
The css classes are understandable and named well.
The only thing I could say something about is the fact that you are using unnecessary css properties sometimes.
For example: when using flexbox you always write out the flex-direction: row; even if it is already per default row and doesn't need to be explicitly written out. Same goes for things like justify-content: flex-start;
Other than that nitpick, nice work!
Marked as helpful
1
Maryam• 260
@Maryam1982
Posted
@Annalisa11 Thanks for taking time to review my work.
0
