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

Git, Gitflow, HTML/CSS, Flexbox, and Positioning

Have-Samuelβ€’ 370

@Have-Samuel

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


Hello Guys, Ideas, suggestions, and compliments are welcome. Cheers!!!!

Community feedback

@MelvinAguilar

Posted

Hi @Have-Samuel πŸ‘‹, good job on completing this challenge! πŸŽ‰

Here are some suggestions you might consider:

  • You could use the CSS backgrounds properties to set the background:
body {
  background-color:  hsl(225deg, 100%, 94%);
  background-image: url(./images/pattern-background-desktop.svg);
  background-repeat: no-repeat;
  background-size: contain;
  background-position: left top;
}

The background property is shorthand for all the properties mentioned above:

background: hsl(225deg, 100%, 94%) url(./images/pattern-background-desktop.svg) no-repeat left top/contain;

References:

.

  • Centering the element with position would make your element behave strangely on some mobile devices. - There are two modern CSS techniques to center elements instead of using the position property:

Using flexbox layout:

body {
   width: 100%;
   min-height: 100vh;
   display: flex;
   flex-direction: column;
   justify-content: center;
   align-items: center;
}

Using grid layout:

body {
   width: 100%;
   min-height: 100vh;
   display: grid;
   place-content: center;
}

Links with more information:

I hope those tips will help you.

Good job, and happy coding!

Marked as helpful

0
Thiago Moraisβ€’ 420

@thirraz

Posted

Firstly, your solution is very good! My only tip is to use semantic HTML. That way you won't have multiple <div> in your code. This link may help you πŸ˜ŽπŸ‘

  • https://developer.mozilla.org/en-US/docs/Glossary/Semantics

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