Skip to content
Submitted over 2 years ago

Order Summary Component

Julien Kombora•230
@JulienK94
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Order summary component
  • HTML
  • CSS
1newbie

Design comparison

SolutionDesign

Solution retrospective

All Feedbacks are welcome. 1st commit on this challenge

Community feedback

  • Fatlind Shehu•2,230
    @fatlindshehuPosted over 2 years ago

    Hi there,

    Your component looks great, there are some small changes I think you should make:

    • background-size: cover; because the background doesn't fill the entire space.
    • The button has more padding on top & bottom and a smaller border radius.
    • I would also avoid using px, instead, I would choose between rem or em and stick to that the whole project.

    Happy coding...

    Marked as helpful
    0
    Julien Kombora•230
    @JulienK94Posted over 2 years ago

    @fatlindshehu thanks for your help

    0

Please log in to post a comment

Log in with GitHub

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