Submitted almost 3 years ago

Order summary card responsive website using html, flex-box, sass

steevencode•120

@steeven509

Preview site View code

This is a solution for...Order summary component

HTML
CSS

1newbie

Design comparison

SolutionDesign

Solution retrospective

Hello World ! I've just completed a front-end coding challenge from Frontend Mentor.

Any suggestions on how i can improve from front-end devs in my network are very welcome. Thanks!

Community feedback

Aadvik•1,250
@Aadv1kPosted almost 3 years ago
A simple fix in your body will be to add background-size: 100%; this will make it so that your background image fills the entire width of the page instead of being its default size. Check out this line on my github and add the changes to Line 42 of your project
1
steevencode•120
@steeven509Posted almost 3 years ago
@Aadv1k thank you very much for the feedback , i will modify the code now
0
Anosha Ahmed•9,300
@anoshaahmedPosted almost 3 years ago
Hey good job on this challenge! Your report above shows a few issues though:

wrap everything in your body in <main> ... OR use semantic tags! .... you can also give role="" to the direct children of your <body> but that's a little frowned upon... Read more here

have at least one <h1> in your code

Check out my repo for future reference.

Hope this helps :)
1
steevencode•120
@steeven509Posted almost 3 years ago
@anoshaahmed thank you very much for the feedback , i will modify the code now
0

Please log in to post a comment

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