Responsive News Homepage with HTML,CSS and JS

Please log in to post a comment
Log in with GitHubCommunity feedback
- @Dytoma
Hey👋
Good job on completing this challenge 👏
I have some feedback for you:
-
First you can look at the Accessibility Report and the HTML validation report. You can find useful information about you code. When using the
ul
element make sure you useli
elements as children and then you can wrap the content in an anchor tag. So your code might look something like this<ul> <li><a href="">Home</a></li> <li><a href="">New</a></li> <li><a href=""> Popular</a></li> <li><a href=""> Trending</a></li> <li><a href=""> Categories</a></li> </ul>
-
Secondly the hover effects provided by the design system are absent, you can implement that too to your code. You can read about hover effect on MDN docs hover📚
Happy coding🙌
Marked as helpful -
- @AhmedMahrouss
good jop my bro Congrats on completing your first challenge!🎊🍻If you have any questions or need further clarification, feel free to reach out to me. Happy Coding! 🎆🎊
Join our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord