@elaineleung
Posted
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
.
@UserAhmad2001
Posted
@elaineleung Feedback noted Thanks for your comment, Elaine. :) I will make sure to fix them later.
Best regards Ahmad
@elaineleung
Posted
@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!
@UserAhmad2001
Posted
@elaineleung šššš