Article preview component

Community feedback

• 

  Better5afe•850

  @better5afePosted over 1 year ago

  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