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

HTML and CSS

Asterisk 370

@mosgizy

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


hey will love feedback from you guys

Community feedback

P
Fluffy Kas 7,735

@FluffyKas

Posted

Hey, good start on this challenge! There's a few things to improve.

  1. There's actually no need to add an extra container div. Everything you added there could be just applied to the body!
  2. Your button uses the browser's default sans serif. This always happens when working with buttons, unless you set the font-family specifically when you style the buttons. Alternatively, you can use all: unset to get rid of all browser defaults. As the accessibility report says, it's a good idea to swap the outer div to main to make your html more semantic.

Good luck! :)

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