@Nam-Hai
Posted
I think it's a wierd choice to have a differente margin-top value as margin-left and margin-right for the .box-image. Either way, nice use of the flexbox. Good use of max-width and min-width too.
Try to avoid % for height and width tho or at least be carefull. Here when on mobile laylout, the over effect is bigger than the image. To fix the probleme, two solution, set a fixed height and width for mobile. But you would not learn much from that hehe. You could also, put the over effect and the image in the same container, which would have the width and height of the image, and give it overflow: hidden
I hope i was clear ^^
Marked as helpful
@deserthawk
Posted
@Nam-Hai Thank you for the feedback. This was the part I had the most difficulty with. I'll keep an eye out for this hover part.
