Skip to content
Submitted almost 2 years ago

Responsive news page using css and tailwindcss

kemorave•290
@Kemorave
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...News homepage
  • HTML
  • CSS
  • JS
2junior

Design comparison

SolutionDesign

Community feedback

  • Sinisa Vukmirovic•1,690
    @SinisaVukmirovicPosted almost 2 years ago

    Hello!

    Looks good. One thing to make it better is to try to use semantic elements in your HTML.

    Errors in your report are because of that.

    This "Document should have one main landmark" means you are missing <main> semantic element in your html. Landmark just means element.

    This "All page content should be contained by landmarks" means all non-semantic elements should be inside of semantic elements, and not directly inside of <body> element.

    More about semantic elements

    Hope this helps! Best of luck!

    Marked as helpful
    0
    kemorave•290
    @KemoravePosted almost 2 years ago

    @SinisaVukmirovic thank you for the info. I actually keep forgetting about semantic elements, I will work on it.

    1

Please log in to post a comment

Log in with GitHub

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