@AgataLiberska
Posted
Hi @DarKaRor! this looks awesome, I am obsessed with how much attention you paid to the details. The hover effects are fantastic - enough to grab attention, but not so much that they would distract from the content itself. Chef's kiss
I also like how you structured your html, putting the two cards in a div is simple but clever :)
Two things I would slightly amend - I'd love to see some padding/margin on the container - on smaller mobiles the cards are reaching both edges of the screen and it looks off - plenty of people use phones that are at or around 320px wide still :) Another thing is that you could use some more semantic html - you could maybe include a <h1>, a <header>, use <article> tags for the cards.
Overall though, I love it!
Marked as helpful
@DarKaRor
Posted
@AgataLiberska Thanks for the feedback! I've applied all the changes and some other small details, I hope it looks better on smaller screens now and that the tags are correct!
