Intro Section With Dropdown Navigation

Community feedback

• 

  Josh•320

  @Ao-chiPosted almost 2 years ago

  Hi! You did great doing this challenge!

  I saw your navigation menu on mobile version and it looks like it is not on top of the body. To make it appear on top you could add z-index: 1 to your nav-menu class.

  Some tips for best practices.

  • You should wrap your main contents into <main> element for more accessibility.
  • Add an alt attribute to your <img> elements like this <img src="/images/client-databiz.svg" class="clientimg" alt="hero banner">
  • You should not wrap a <button> inside an <a> element and vice versa. It is for accessibility practice. You can use either one of them. I you want the button to have a cursor pointer, you can just add this on your css styling cursor: pointer

  Hope this helps! Happy Coding~

  Marked as helpful

  0