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

P
Mattā€¢ 50

@Matt-LaRochelle

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


I was working on making things more dynamic while not necessarily covering every edge case this time. What do you guys think? Is it too messy or is this ok?

Community feedback

Daniel šŸ›øā€¢ 44,230

@danielmrz-dev

Posted

Hello @Matt-LaRochelle!

Your project looks really good!

I just have one suggestion:

  • You can use both background-color and background-image together on the body in order to create that background pattern. They will not cancel each other.

šŸ“Œ Like this:

body {
    background-image: url('/* image path here */');
    background-repeat: no-repeat;
    background-size: contain;
    background-color: #E1E9FF;
}

I hope it helps!

Other than that, great job!

1

@Tam643

Posted

@danielmrz-dev

Thank you so much for suggesting. I will use it in this project and another project.

1

@darley-santos

Posted

The card is quite large on my notebook; I think you might have overdone the padding in the content. Also, the background image is missing. Nevertheless, it looks good.

1

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