Skip to content
Submitted almost 2 years ago

Responsive Order Summary Card Component

Matheus de Farias Cavalcanti Santos•40
@matheusdefarias
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Order summary component
  • HTML
  • CSS
1newbie

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-jawPosted almost 2 years ago

    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

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