Article Preview Component

Community feedback

• 

  Marlon Passos•940

  @MarlonPassos-gitPosted about 3 years ago

  hello frog, one thing I noticed in your project that I saw was wrong is the following:

  • Your button is not accessible via the TAB key, this can make accessing the page difficult for people with special needs. To solve this you can put the tag inside a button

  0
• 

  T

  ApplePieGiraffe•30,545

  @ApplePieGiraffePosted almost 4 years ago

  Hey, Front-end Master! 👋

  Good work on this challenge! Your solution looks good and responds well! 👏

  Changing the hero image to be black and white upon hover was a cool touch! 😎

  I only suggest,

  • Not adding cursor: pointer to the image because it's not a clickable element such as a button or link.
  • Adding a little more space between the social media icons in the mobile layout of the site to make them easier for fingers to tap.

  Keep coding (and happy coding, too)! 😁

  1

  ---

  

  Web Frog•1,840

  @MasterDev333Posted almost 4 years ago

  Thanks for your feedback :)

  1
• 

  Abhik•4,840

  @abhik-bPosted almost 4 years ago

  Hi FRONT-END MASTER,

  • Your solution is very responsive
  • really your image transition on hover is 🔥🔥

  You have done a great job, Keep it up 💯

  0