Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Your session has expired please log in again.
Your session has expired please log in again.
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

css Grid, html, flexbox

@Ericwiz

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


please check out my solution and suggest areas of improvement.

Community feedback

Marius 190

@MariusHor

Posted

Great job so far! 🙂

for the background you could add these lines: background-color: hsl(225, 100%, 94%) background-size: contain background-repeat: no-repeat.

I would also suggest that you dont set fixed heights and widths on elements. Use padding to create space within an element and margins to create space between elements and not top left or position: relative / position: absolute. Google "spacing in CSS" for a better understanding.

Keep going, you are getting there! 🙂

0

@Ericwiz

Posted

@MariusHor thanks so much for the correction I really appreciate!

0
clive 140

@clivepato93

Posted

Hi there for your background-repeat use no repeat

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