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

@Grendaizo90

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


Still have a problem with responsive behavior, maybe you could tell me about bad practices in my code, and about best pratices i should use.

Thanks in advance!!!

Community feedback

@rostyslav-nazarenko

Posted

Hi!

Everything looks great! Only a few suggestions.

  • use resets for button as it doesn't inherit font properties from parent elements.
  • provide a more descriptive alt attribute for the image or leave it empty
  • use relative units in media queries, preferably em units. The same goes for component max-width
  • don't use strong for styling, use CSS.
1
Alex 1,310

@AlexKolykhalov

Posted

Looks good, and the markup is also норм)

1

Alex 1,310

@AlexKolykhalov

Posted

@Grendaizo90 ))

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