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

@DamonTham

Desktop design screenshot for the Order summary component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


Any suggestions or feedback are welcomed. Thank you!

Community feedback

Vanza Setia 27,795

@vanzasetia

Posted

Hello, Tham Jia Jun! 👋

Well done on this challenge! The HTML is already well structured. One suggestion for the HTML is to specify the type of the button elements. In this case, set the type of them as type="button". It will prevent the button from behaving unexpectedly.

For the styling,

  • Add some padding to the body element. It will prevent the card from touching the edges of the browser.
  • Code your font size in rem, not pixels. This means the text size will be responsive if people want to adjust the size.
  • Create a custom :focus-visible styling for any interactive elements (e.g. button, links, input, textarea). This will let people who use a keyboard to navigate the site know where they are currently on the page.

I hope you find this useful!

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