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

Tariq Auwalβ€’ 90

@Tareeqcodes

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


Any feedback or suggestion please

Community feedback

Olaniyi Ezekielβ€’ 7,600

@Ezekiel225

Posted

Hello there πŸ‘‹.

Good job on completing the challenge !

Your project looks really good!

I have suggestion about your code that might interest you.

Consider changing the background-color of the body element to hsl(225, 100%, 94%) so as to match the original design.

body {
  background-color: hsl(225, 100%, 94%);
}

I hope it helps!

Other than that, great job!

Happy coding.

Marked as helpful

0
Daniel πŸ›Έβ€’ 44,230

@danielmrz-dev

Posted

Hello @tariqauwal2020!

Your project looks great!

I noticed that you used margin to place the card in the middle of the page. Here's a very efficient (and better) way to center the card:

  • Apply this to the body (in order to work properly, don't use position or margins):
body {
    min-height: 100vh;
    display: flex;
    justify-content: center;
    align-items: center;
}

I hope it helps!

Other than that, great job!

Marked as 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