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 card responsive website using html, flex-box, sass

@steeven509

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

@Aadv1k

Posted

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

@steeven509

Posted

@Aadv1k thank you very much for the feedback , i will modify the code now

0
Anosha Ahmed 9,300

@anoshaahmed

Posted

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

@steeven509

Posted

@anoshaahmed thank you very much for the feedback , i will modify the code now

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