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

Responsive page using flexbox and grid

@om-prabhat

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


This is a solution to the Order summary card challenge on Frontend Mentor. Frontend Mentor challenges help you improve your coding skills by building realistic projects.

Users should be able to:

  • See hover states for interactive elements

  • Semantic HTML5 markup

  • CSS custom properties

  • Flexbox

  • CSS Grid

Community feedback

@DivineUgorji

Posted

Hello @om__prabhat, congratulations on completing and submitting this challenge. Your implementation looks really nice but I could not help but notice that your background image is not well positioned. Apply the following properties to your code:

  background-color: hsl(224, 100%, 94%);
  background-image: url(image-url);
  background-position: top;
  background-repeat: no-repeat;
  • I would also suggest the use of BEM methodology for your CSS naming conventions.
  • I observed the overuse of the pixel unit in your code base. It's always good practice to learn how to use relative units in your projects as it helps with responsiveness. You can check out this blog to learn more about this.

overall, your implementation is very nice, and I hope these few suggestions of mine will help you to improve it better.

All the best 🙏

Marked as helpful

1

@om-prabhat

Posted

Thank you for giving feedback. I will surely look it out.@DivineUgorji

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