3 Column Preview Component (updated solution)

Community feedback

• 

  Vanza Setia•27,775

  @vanzasetiaPosted over 2 years ago

  Excellent work on this challenge, Rado! Your solution is responsive and looks great! 👏

  There is some feedback from me.

  • For any decorative images, each img tag should have empty alt="" and aria-hidden="true" attributes to make all web assistive technologies such as screen reader ignore those images. In this case, all images are decorative only.
  • Alternative text for images should not contain any words that related to image (e.g. picture, photo, logo, icon, graphic, avatar, etc). It's already an image element so the screen reader will pronounce it as an image.
  • The Learn More buttons would probably link elements instead of button elements. I think that if this a real website and the user click on the buttons, it will navigate the user to another webpage or area on the site.
  • Next time, always specify the type of the button. It's going to prevent the button from behaving unexpectedly.

  I hope this information is useful! Keep up the excellent work! 👍

  2
• Account deleted

  Hello there! 👋

  Congratulations on finishing your challenge! 🎉

  I have some feedback on this solution:

  • add cursor pointer to button so that the user knows its clickable
  • add tranistion:1s; to the buttons so that when you hover on the button the background changes smoothly

  i hope this is helpful and goodluck!

  1