TailwindCSS page

Community feedback

• 

  P

  tediko•6,680

  @tedikoPosted over 3 years ago

  Hello, Tin.Pham! 👋 Congrats on finishing another challenge! Your solution looks very good and also responds well. Here's my few tips:

  • Change the alt attributes for the dashboard-image, as it doesn't add any extra context for screen reader users. Since your image is decorative your alt text should be provided empty (alt="") so that they can be ignored by assistive technologies.
  • Wrap your icons into some anchor elements <a>. Remember to add aria-label with a description to it since icons are treated as if there was no content there.

  Good luck with that, have fun coding! 💪

  1

  ---

  

  Tin.Pham•250

  @tin-phamPosted over 3 years ago

  @tediko Tks very helpful tips

  0
• 

  P

  Patrick•14,265

  @palgrammingPosted over 3 years ago

  looks pretty close to the design. you need a little more space around the top and bottom of the main hero img. Also the circles around your social icons to not to look to be the correct color and they are currently lacking the hover effect

  1

  ---

  

  Tin.Pham•250

  @tin-phamPosted over 3 years ago

  @palgramming Oke, tks so much, very appriciate

  0