Responsive Intro section with dropdown navigation

Community feedback

• 

  Elaine•11,400

  @elaineleungPosted over 2 years ago

  Hi Ahmad Mana, great work! I also completed this challenge this week, and like you, I used the <picture> element in dealing with responsive images. Well done solving your problems :)

  I can see in your report that you have a number of HTML and/or accessibility issues such as needing alt tags in images, so you might want to fix those. Also, your font isn't showing up; it's because the link you used is the font face's main link and not the server link, which should look something like https://fonts.googleapis.com/css2?family=Epilogue with the font weights attached. You'll need to click into the page for the font face and then select the font weights you need, then copy the link given and paste it into your code. Lastly, the hover style for the dropdown nav seems to be changing the entire nav instead of just the link I'm hovering on, so I think you need to put the hover: on your li instead of ul.

  1

  ---

  

  Ahmad_Mana•320

  @UserAhmad2001Posted over 2 years ago

  @elaineleung Feedback noted Thanks for your comment, Elaine. :) I will make sure to fix them later.

  Best regards Ahmad

  1

  

  Elaine•11,400

  @elaineleungPosted over 2 years ago

  @UserAhmad2001 Oh, I forgot to mention that you can just add empty alt="" tags if the image isn't particularly important to the content, but you do need to have the alt tags somehow. Good luck!

  1

  

  Ahmad_Mana•320

  @UserAhmad2001Posted over 2 years ago

  @elaineleung 👍👍👍👍

  0