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 using html and css

Zeena 175

@kushyzee

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


I couldn't figure out how to position the background image, so i just made use of a solid color. If anybody knows how to go about it, do let me know, thanks

Community feedback

@tesla-ambassador

Posted

Hey Kushyzee. This is pretty impressive, congrats on completing your challenge! Here's a few tips:

  • To answer your question on background positioning, you can use the background-position css property more on this if you follow this link You could also set the background size to match that of the design using background-size css property.
  • The responsiveness is a little off between 810px - 1149px. I think it'd be okay if you centered it and reduced the width of the card component a little to let's say (80% because I think you preffer it to be a little wider)
  • To fix your accessibility issue, you might want to enclose your <div> with the class "attribution" in a footer landmark. Happy coding and keep on coding!

Marked as helpful

0

Zeena 175

@kushyzee

Posted

@tesla-ambassador Thanks so much for your feedback, I followed them accordingly and now i believe it's looking much better

1
P
Fluffy Kas 7,735

@FluffyKas

Posted

Heyo,

You can add the background image together with the colour, they just need to be comma separated. Like so:

body {
  background: url(images/pattern-background-mobile.svg), var(--paleblue);
  background-repeat: no-repeat;
  background-size: contain;
}

Marked as helpful

0

Zeena 175

@kushyzee

Posted

@FluffyKas Thanks very much for this, it helped a lot. It seemed not to work at first but when I switched the background-size to cover, it came out well

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