Article preview component (React | Vite | Vanilla CSS)

Community feedback

• 

  Kitalphar•150

  @KitalpharPosted 2 months ago

  Great work!

  I like the animations on the socials and the fade-in effect for their container, however the same effect seems to be applied to the Author's details, which feels more like an error than a feature if i'm to be honest.

  Hope you have a great day!

  Marked as helpful

  1

  ---

  

  P

  Marc Francis•730

  @marcfrancissPosted 2 months ago

  @Kitalphar Thanks for checking this out and providing feedback! Really appreciate it. 🥰

  May you please suggest what changes should I make to the 'Author's detail' for it to be more appealing?

  0

  

  Kitalphar•150

  @KitalpharPosted about 2 months ago

  @marcfranciss

  This is just my opinion so feel free to ignore.

  On the mobile layout, article-author-container mobile has this fade in effect for the share-nav mobile, which i like, i think it's nice. Clicking on the button again, reveals the Author's details (article-author) with the same fade-in effect....my point is that since the user is already avare of these details, "revealing" it again with the animation feels like it doesn't add to the user experience, if that makes sense? This way the user may think the animation was left over from share-nav mobile by accident....so i think leaving the article-author static or instead giving the share-nav mobile a fade-out animation would look better and feel more purposeful.

  But once again, this is just my opinion, just food for thought. There is no real need to change your solution, i think it works great and is pleasing to the eyes. :)

  Have a great day!

  1
• 

  Mehrine•120

  @DarkMoonLitPosted 2 months ago

  Hello, Well done on the challenge.

  Nice coding!

  Marked as helpful

  1

  ---

  

  P

  Marc Francis•730

  @marcfrancissPosted 2 months ago

  @DarkMoonLit Thanks for checking this out and providing feedback as well! Really appreciate it. 🥰

  0