Responsive News Homepage with HTML,CSS and JS

Community feedback

• 

  Dytoma•550

  @DytomaPosted almost 2 years ago

  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 use li 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

  0

  ---

  

  Kinyenje•60

  @KINYENJEPosted almost 2 years ago

  @Dytoma Thanks.I appreciate the help.

  1

  

  Dytoma•550

  @DytomaPosted almost 2 years ago

  @KINYENJE you're welcome dear🙌

  0
• 

  Ahmed Mahrous•980

  @AhmedMahroussPosted almost 2 years ago

  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! 🎆🎊

  0

  ---

  

  Kinyenje•60

  @KINYENJEPosted almost 2 years ago

  @amahrous176 Thanks.😅

  0