Responsive mobile first webpage built with Just HTML and CSS

Community feedback

• 

  P

  Ken•4,915

  @kens-visualsPosted about 3 years ago

  Hey @RandyKdev 👋🏻

  I've got a couple of accessibility suggestion for the project.

  • For the star icon icons, add aria-hidden="true”, because they are for decoration. You can read more about aria-hidden here. For example: <img class="star" src="./images/icon-star.svg" alt="" aria-hidden="true”>
  • Next, the alt tag for the people should be their full names, like so <img class="profile-pic" src="./images/image-colton.jpg" alt="Colton Smith">.
  • And since you followed the semantic rules of HTML, I suggest changing <div class="header">...</div> to <header class="header">...</header>

  I hope this was helpful 👨🏻‍💻 other than that, you did an excellent job, well done. Cheers 👾

  Marked as helpful

  1

  ---

  

  Randy Kwalar•75

  @RandyKdevPosted about 3 years ago

  Thanks @kens-visuals for this insightful comment.

  0

  

  P

  Ken•4,915

  @kens-visualsPosted about 3 years ago

  @RandyKdev you're welcome 😇

  0