Skip to content
Submitted over 2 years ago

Order summary component

Konstantin Kamenskiy•50
@Grendaizo90
A solution to the Order summary component challenge
Preview (opens in new tab)Code (opens in new tab)

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!!!

Code

Please log in to post a comment

Log in

Community feedback

  • Rostyslav Nazarenko•560
    @rostyslav-nazarenko

    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.
  • Alex•1,310
    @AlexKolykhalov

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

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