Skip to content
Submitted almost 2 years ago

Order summary component (BEM Methodology)

Kévin Burle•370
@kevinburle
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Order summary component
  • HTML
  • CSS
1newbie

Design comparison

SolutionDesign

Solution retrospective

Hello world,

I've used the BEM Methodology to name my classes. Let me know if it's incomplete, messy and how i can improve it.

thanks !

Community feedback

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

    Hello!

    To improve your code, use semantic HTML5 elements.

    Semantic elements

    Errors in your report are about that.

    This = "Document should have one main landmark", means <main> semantic element.

    This - "All page content should be contained by landmarks", means semantic elements. Landmarks are just element. And all <div>s and other non semantic elements should be inside of those semantic elements, not directly inside <body> element.

    Try to use them in your html code, hope your errors go away!

    Marked as helpful
    1
    Kévin Burle•370
    @kevinburlePosted almost 2 years ago

    @SinisaVukmirovic thanks a lot for your usefull explanation !

    0

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