Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Order summary component

JhayJahswillβ€’ 50

@JhayJahswiil

Desktop design screenshot for the Order summary component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Community feedback

Akhil-Nagpalβ€’ 180

@Akhil-Nagpal

Posted

Hi Jhay! πŸ‘‹ Congratulations on completing this project! πŸŽ‰

To fix all the accessibility errors, you need to do these things:

  • You must follow a basic web page structure. You used the <main> tag under the <div>. Use <main> only once per page, and put it directly inside <body>. Ideally, this shouldn't be nested within other elements. Read this for a better understandingπŸ‘‰ Document and website structure

  • This structure also applies to headings. The <h1> to <h6> element tags must be in descending order(as per size). Read more πŸ‘‰ The HTML Section Heading elements

Other Suggestions πŸ“–

  • Box shadow is a bit high, put 0px for offset x & spread radius set to negative. This will create a very similar effect.
  • To achieve the desired size of the card, I have a trick, slightly adjust the screen size to 75-80%, which will make your screen 1440px.
  • Always read README & Style Guide pages. These pages will help you to understand what to do.

I hope this will help you. 😊

Otherwise! You did a great job! πŸ‘Œ Happy Coding!

0

Please log in to post a comment

Log in with GitHub
Discord logo

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