article preview component

Community feedback

• 

  Anwar Mestycer•420

  @AnwarMestycerPosted 8 months ago

  Hi Alejandro, You did well with the design; it looks good.

  • For mobile, try to place share-icon and share_div in the same div and hide the share_div initially. Once the user clicks the share_icon, hide the user_div and display the share_div, displaying them on top of each other is not a good practice.

  • Add the else case once the user want to hide the share options, it's not included in your code just by adding an else statement that undoes what you changed in the if statement .

  • Add some semantic HTML to make your code accessible to screen readers.

  I Hope this was helpful

  Marked as helpful

  0