Intro section with dropdown navigation

Community feedback

• 

  Sean Red•635

  @seanred360

  Posted about 3 years ago

  I have fixed the issue with your dropdown menu. Please check my pull request on GitHub to merge the changes. The problem was you have deprecated jQuery code, you need to use the new syntax or it won't work correctly. You can go to the GitHub page and click on the Issues tab, next to <> Code and Pull Requests to see the bugs I have found and how to fix them.

  01

  Marked as helpful
• 

  NotBenjamin•70

  @Aeteung

  Posted about 3 years ago

  Try to use a closing tag instead of a self-closing tag for the script tag

  00
• 

  Tesla_Ambassador•3,070

  @tesla-ambassador

  Posted about 3 years ago

  Hey Vani, congratulations on completing this challenge, it looks pretty solid! 💪 Here's a few pointers that could be useful:

  • I looked through your code and ran it on github desktop to findout what the jQuery problem was, sadly I couldn't figure it out because it also ran on mine and if I am being honest, I have used very little jQuery but once I come up with a solution, I will notify you!
  • In order to resolve some of your accessibility issues, you might wanna use landmarks in your html code, these help browsers easily navigate your code. So you might consider wrapping your divs in landmarks like <main> or <header> or <footer> you need to do this according to how your page is structured. Incase you want to know more about landmarks, follow this link.
  • You need to add an alt text to your images. This is because it tells the browser what image is being displayed and this can also help those using screen readers to know what image is being displayed on the website. It will also clear some of your HTML validation issues and accessibility issues

  Happy coding and keep up the great work! 👍

  01