My Article Preview Component

Community feedback

• 

  Azhar•600

  @azhar1038Posted about 2 years ago

  Hi Hunter McGrew, congratulations on completimg this challenge!

  I noticed one small issue in mobile view, when you click on the share button, height of card changes. This is due to different height of share menu and author details.

  An accessibility tip is to use button for share icon instead of div. Since this button contains image instead of text, make sure to give an aria-label and set aria-hidden to the child image. Also you have used section unnecessarily. There is no need to put an image inside section and your header and paragraph can go in same section.

  Try to use hn tags for heading instead of paragraphs.

  Hope it helps :)

  0