Responsive news-homepage using Flexbox, Grid and MediaQuery

Community feedback

• 

  Boluwatife Bonaventure•550

  @BoyutifePosted over 1 year ago

  Hi Manish Tamang 👋. Congratulations on successfully completing the challenge! 🎉

  Job well done .

  I have some additional suggestions for your code that I believe will be of great interest to you.

  HTML 🏷️:

  This solution generates accessibility error reports, "All page content should be contained by landmarks" is due to non-semantic markup, which lack landmark for a webpage. To address this issue, you can review the HTML code of the webpage and add appropriate landmarks where necessary.

  • you consider using <section> and <article> tags for <div class="blog-lists"> and <div class="list">

  HEADINGS ⚠️:

  Heading tags (such as <h1>, <h2>, <h3>, etc.) are an important aspect of HTML markup, as they provide a hierarchical structure to the content on a webpage. Here are some reasons why heading tags are important:

  -Organization: By using heading tags, you can organize your content into a hierarchy, with the main topic of the page represented by the highest-level heading (<h1>) and subtopics represented by lower-level headings (<h2>, <h3>, etc.). This makes it easier for readers to navigate the content and understand the structure of the page.

  -Overall, heading tags are an important aspect of HTML markup, as they help organize and structure the content of a webpage, make it more accessible to users with disabilities, improve SEO, and enhance readability.

  -Try change using h2 , h3 and h4 instead of h1 and h5 , it needs to increases gradually.

  I hope you find this helpful 😄 Above all, the solution you submitted is great !

  Happy coding!

  Marked as helpful

  0

  ---

  

  Manish Tamang•80

  @Manish-456Posted over 1 year ago

  @Boyutife Thank you for the nice feedback.

  0