Skip to content
Submitted over 2 years ago

Order summary component

Jax Teller•670
@piushbhandari
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

critique me. i dare you

Community feedback

  • Prince Roy•1,210
    @iprinceroyyPosted over 2 years ago

    That's excellent work @piushbhandari. You need to focus on some key points here:-

    1. There should be one heading tag on a web page, if there is no heading element requirement, then set its font size to 0. h1{ font-size: 0; }
    2. The content should be wrapped inside the main landmark to avoid accessibility issues. Like this <body><main>your content goes here......</main></body>.
    3. Avoid more than 1 level of nesting div elements, use other tags like <section> <article> to structure.

    Hope it adds to your learning. Happy coding :)

    Marked as helpful
    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