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

jiju278 50

@jiju278

Desktop design screenshot for the Order summary component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Community feedback

Alamin 2,000

@CodePapa360

Posted

Hi jiju278👋 Great job on completing this challenge! 🥳

I noticed a few areas where your solution to the challenge could be improved, and wanted to share my suggestions with you.

  • Instead of using a global selector (*) to apply box-sizing to all elements, you could use a more specific selector such as html or body to avoid unnecessary style inheritance. For example:
  html {
    box-sizing: border-box;
  }
  • You could use the rem unit instead of px for sizes to make your layout more responsive and scalable. For example:
  .card {
    width: 18rem;
  }
  • You could use the em unit instead of % for font sizes to make them more flexible and relative to the parent element's font size. For example:
  .header__title {
    font-size: 1.5em;
  }

Overall, this is a very well done solution to the challenge. I hope these suggestions help you improve your code and take it to the next level!

Hope I'm Helpful! 👍

Keep up the good work! 😊❤️

Marked as helpful

1

jiju278 50

@jiju278

Posted

@CodePapa360 Thank you very much for your advice and your encouraging message. It is very motivating for the future 😊❤️

1

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