Hoover for the equilibrium image

Community feedback

• 

  Vanza Setia•27,775

  @vanzasetiaPosted over 2 years ago

  Hello there! 👋

  Good effort on this challenge! 👍

  Some suggestions from me.

  • I recommend swapping all article and section with div elements. If you see the HTML issues that has been reported, article and section cause some issues. It's okay to use div as long as it doesn't cause any accessibility issues.
  • You can read this article about div, written by Scott O'Hara. It will tell you when and how to use a div. Also, it might change the way you think about div.
  • For the white space, I suggest using margin or padding instead of nbsp.
  • All elements that have interactivity should be wrapped by interactive elements like anchor tags.
  • Those icons are decorative images which means that if they don't exist, there will be no missing information, it's just made the site looks less pretty. So, for decorative images leave the alt empty.
  • Alternative text for images should not contain the word "icon" or any words that are related to "image".
  • Alternative text for the avatar should be the author's name.

  That's it! Hope this helps.

  Marked as helpful

  1

  ---

  

  Kohsey•40

  @KohseyPowerPosted over 2 years ago

  @vanzasetia Hello !! I am grateful to you for detailing my mistakes. I will correct my 'alt' things and my not really sexy 'nbsp'. However, I have a last question. Should I replace my equilibrium interactive image by an anchor tag ? Thanks in advance for answering.

  Right after posting this, I will go to your link about 'div'.

  Have a nice day ! 😘

  0

  

  Vanza Setia•27,775

  @vanzasetiaPosted over 2 years ago

  @KohseyPower Hahaha, you're welcome! 👍

  You should replace the section tag with an anchor tag. That's what I mean. 🙂

  1

  

  Kohsey•40

  @KohseyPowerPosted over 2 years ago

  @vanzasetia Okay got it!

  0