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

gabymarchingo• 120

@gabymarchingo

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


background shows on my editor but doesn't on here as GitHub doesn't like leading /'s it's also centered properly on my editor/browser

Community feedback

@the-coding-beekeeper

Posted

Hello gabymarchingo!

Congratulation for finishing the project! There are some issues that should be fixed. My suggestion to get closer to the design-template:

body {
width: 100vw;
min-height: 100vh;
display: flex;
flex-direction: column;
align-items: center;
justify-content: center;
background-image: url(/images/pattern-background-desktop.svg); 
background-repeat: no-repeat;
background-size: contain;
background-color: hsl(225, 100%, 94%);
}
.card {
/* margin-top: 130px;
margin-left: 580px;*/
display: flex;
flex-direction: column; 
}

Give the body a width and a height and center the content; Fix the background issues by a working file path and some background properties; delete the hard coded margins at your .card and add flex-direction: column.

I hope that will fix some issues. Happy coding!

Greetings from the-coding-beekeeper

0
Ahmed AbdElAal• 320

@xCordeva

Posted

I checked your code the reason the styles are not working correctly is that you didnt link the path correctly it should be this instead, after the edit it should work properly

<link rel="stylesheet" href="./CSS/style.css">

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