Better5afe• 850
@better5afe
Posted
Hi Przemysław,
First of all, congrats on completing the project! 🎉
Here are some suggestions from my side:
- transition does not work for properties which have a true/false state, so for example visibility (visible/hidden) or display (block, flex, etc. / none). In this scenario, if you want to achieve a smooth transition on the tooltip, you can add it for some other properties, like opacity or one of the position values (top, bottom, right, left)
- if the tooltip should appear only after the share button is clicked, I would use display property instead of visibility - with visibility it's still on the page, just not visible
- just a minor one with the CSS classes - it should be panel, not pannel and using wrapper instead of wrapp would also be a good idea
Hope it helps. Overall, the project looks good and I hope that you had fun making it!
Happy Coding & greetings from Poland ;)
Marked as helpful
0