Huddle Landing Page (this took me a week)

Community feedback

• 

  Vanza Setia•27,775

  @vanzasetiaPosted over 2 years ago

  Hello, Nguyen! 👋

  Great work on this challenge! Your solution looks pretty good! 😀 It's great that you wrap all the page content with landmark elements! Well done! 👍

  Regarding your questions.

  1. I notice that it use background-size: cover and it works well. I used background-size: cover and it worked well. However, I had background-position: center right; and it looked similar to the design. You can see my solution if you want.
  2. Yes, you can visit cdnjs.com and grab the link tag for the Font Awesome for version six and below. But, I would recommend downloading the needed icon from the Font Awesome website instead of importing the entire icons. It is going to boost the performance of the site.

  For the third question, here are some suggestions from me.

  • 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.
  • Not all images need alternative text. In this case, I would recommend leaving the alt="" empty so that the screen reader users won't have to listen to unrelated content.

  A11y checklist would be a great resource to make your site accessible. Make sure that everytime you submit any solution you check as many items as possible.

  That's it! Hope this helps. 😊

  Marked as helpful

  1

  ---

  

  Nguyen Nguyen•340

  @jesuisbienbienPosted over 2 years ago

  @vanzasetia thank you for answering all my questions. Your feedbacks are very helpful. I really appreciate it.

  0

  

  Vanza Setia•27,775

  @vanzasetiaPosted over 2 years ago

  @jesuisbienbien You're welcome! Glad you found it to be very helpful! 😄

  0