LetsFrontend• 500
@B1N4R1
Posted
Hi Julien! Congrats on completing the challenge, your solution looks fantastic!
Three things:
-When you hover the share button I would add cursor: pointer to the element so the user knows that the button is clickable.
-Watch out the footer, in really small devices it overlaps with the card and makes it difficult to hit the share button or read anything behind it.
-Be careful with the tooltip, when the tooltip is out in desktop mode and you change the size of the window to a mobile size the tooltip goes crazy.
Overall, these are just little things the real deal is already done
Keep the great work!
Cheers!
Marked as helpful
0
