only html and CSS

Community feedback

• 

  T

  Chamu•13,130

  @ChamuMutezvaPosted about 3 years ago

  Greetings Nazeer. Some issues have been raised in the report that needs fixing. These issues may include the following:

  • the alt values for you images must be descriptive enough to benefit assistive tech users who rely on the alt values to visualize the message that is being put across. Words like logo, image, icon etc should not be included in the alt value, they are automatically announced as such by screen readers. Decorative images can be written as alt=""
  • one of the errors mentioned can be fixed by having a main element. It is important to use semantic elements .
  • in your .info__list in particular, the immediate children of a ul element are li elements. A div cannot be a direct child of a ul
  • when using icons in the links as you did in the social__list , the link is supposed to have text that can provide the purpose of the link to screen reader users. You can do that by using a span as a child of the anchor element but it should be visible hidden using the sr-only technique or an other styling that makes sure that the content will be available to screen readers

  Marked as helpful

  0
• 

  Dharmik_487•1,740

  @Dharmik48Posted about 3 years ago

  Hey👋, Good Job on completing the challenge! But there are a few things you should improve:

  • there's too much space in between some elements, decrease it a bit.
  • You have used just too many divs , some elements can be moved to the same div rather than(have same parent) having a different div.
  • You should use semantic html like section, main, etc. It is a very good practice.

  Marked as helpful

  0

  ---

  

  Nazeer Haidari•895

  @Nazeer2020Posted about 3 years ago

  @Dharmik48 thank you, I will fix it

  0
• 

  Nazeer Haidari•895

  @Nazeer2020Posted about 3 years ago

  Hi chamu thank you, I will fix above issues

  0