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
Your session has expired please log in again.
Your session has expired please log in again.
Your session has expired please log in again.
Your session has expired please log in again.

Submitted

Flexbox

Sachin 830

@Sachin-Mahato

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


feedbacks are welcome and your feedback will help me improve and do better next time.

Community feedback

Hassia Issah 50,650

@Hassiai

Posted

Replace the <h3> with <h1> to fix the accessibility issue.

You forgot to give a body a background-image with a background-size of contain and a background repeat of no-repeat.

This challenge requires a media query, in the media query you only have to change the background-image of the body.

Use relative units like rem or em as unit for the padding, margin, width values and preferably rem for the font-size values, instead of using px which is an absolute unit. For more on CSS units Click here

Hope am helpful.

Well done for completing this challenge. HAPPY CODING

Marked as helpful

0

@thomas-x-69

Posted

Great job, but i have a few notes 1- Texts: You have to differentiate between regular and bold texts 2 - Pay attention to the sizes of the elements in relation to each other

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