Responsive Landing Page and Navbar using HTML, CSS and Javascript

Community feedback

• 

  S MD suleman•3,530

  @sulemaan7070Posted over 1 year ago

  hey 😄Bernard Addy-Sackey, congratulations on completing the challenge... here are a few tips to make your site better.

  1.The mobile menu which you have named as .solid-menu is positioned absolutely... make it fixed so that the user scrolls the site when the mobile-menu is opened it can stay fixed on the screen.

  .solid-menu {
     position: fixed;
  }
  

  2.There seems to unnecessary margin-top on the mobile version... make sure you reduce it..

  Happy coding😄💯✅

  Marked as helpful

  1

  ---

  

  Bernard Addy-Sackey•230

  @boypaida12Posted over 1 year ago

  @sulemaan7070 thanks for the feedback. I'll work on that. Aside that is there anything else I should work on?

  1

  

  S MD suleman•3,530

  @sulemaan7070Posted over 1 year ago

  @boypaida12 another thing is you should not use more than one h1 for the page.. as the h1 is considered as the top-level heading...and only there should one h1

  Marked as helpful

  1

  

  Bernard Addy-Sackey•230

  @boypaida12Posted over 1 year ago

  @sulemaan7070 right. thank you.

  1