intro-section-with-dropdown-navigation-main

Community feedback

• 

  Adriano•38,410

  @AdrianoEscarabotePosted about 2 years ago

  Hi sathaporn, how are you?

  I really liked the result of your project, but I have some tips that I think you will enjoy:

  • To improve the accessibility of the project you could have put an h1. Every page must contain a level 1 header, for people who use screen readers, identity what the main title is.
  • Consider using rem for font size .If your web content font sizes are set in absolute units, such as pixels, the user will not be able to re-size the text or control the font size based on their needs. Relative units “stretch” according to the screen size and/or user’s preferred font size, and work on a large range of devices.
  • You have used <br> , using <br> is not only bad practice, it is problematic for people who navigate with the aid of screen reading technology. Screen readers may announce the presence of the element. This can be a confusing and frustrating experience for the person using the screen reader. You can read more in MDN.

  The rest is great!

  I hope it helps... 👍

  Marked as helpful

  1

  ---

  

  sathaporn•480

  @NiLoBolPosted about 2 years ago

  @AdrianoEscarabote thank you very much I gained a lot of new knowledge.

  0

  

  Adriano•38,410

  @AdrianoEscarabotePosted about 2 years ago

  @NiLoBol no problem, keep coding!!

  0