Not Found
Not Found
Not Found
Not Found
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.
Your session has expired please log in again.
Your session has expired please log in again.
Not Found
Not Found
Not Found
Not Found

Submitted

Summary Order

@Bazza-08

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


  1. How could I make may CSS DRY? I've repeated the same margins several times.

  2. Couldn't get the background right, any tips/solutions?

Community feedback

P
visualdennis 8,375

@visualdenniss

Posted

Hey there,

congrats on completing the challenge successfully! Your submission looks pretty nice overall.

It looks like your background color is missing on the desktop view. You can just add this: background-color: #e0e8ff; to the body to solve and match the design. But if it is your preference to keep it white, you can also try adding a box-shadow e.g. box-shadow: rgb(149 157 165 / 20%) 0px 8px 24px; to the .card for an interesting look. Here is a great resource of some cool box-shadows, i also use a lot: https://getcssscan.com/css-box-shadow-examples

Hope you find this feedback helpful!

Marked as helpful

1

@Bazza-08

Posted

@visualdenniss Thank you!

0
P
markus 1,940

@markuslewin

Posted

Try using padding, instead of margin, for space between a container and its content!

0
Atif Iqbal 3,200

@atif-dev

Posted

Hi Barry, congrats🎉 on completing the challenge.

  • Learn The var() Function from here to tackle DRY in your css.
  • For background modify body selector code as:
body{
    margin: 0;
    padding: 0;

    text-align: center;
    background: url("../images/pattern-background-desktop.svg") no-repeat top center;
    background-color: hsl(225, 100%, 94%);

    display: grid;
    min-height: 100vh;
    place-content: center;
}

After applying above block of code also remove following code:

@media(max-width: 700px) {
    
    body {
        background:hsl(225, 100%, 94%);
    }
}

Hope you will find this Feedback Helpful.

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