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 Order Summary Card Component

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


I am starting with web development, so I would like to request feedback in general about code best practices and if my code is clear.

Tips and suggestions would be nice!! :D

Community feedback

Muhammadh 1,140

@Mr-jaw

Posted

Hello there 👋

Congratulations on completing the challenge

HTML 📰

  • avoid using the <div> tag to display text alone. Since the <div> tag is mostly used to group elements. You can rather use <p> long descriptive text and <em>, <strong> or <small> to display short texts (i.e <div class="plan">Annual Plan</div> and <div class="value">$59.99/year</div>)

CSS 🌈

Nothing to say about your CSS much. You have done a great job on following conventions of CSS custom variables and relative units

Overall your solution looks great 👍

I hope this was useful 😊

HAPPY CODING

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