Skip to content
Submitted almost 3 years ago

Frontend Mentor | Order summary card

Sachin Shelke•190
@SachinShelke7
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 Completed the Frontend Mentor | Order summary card challenge, if you have any suggestion please suggest.

Community feedback

  • Martin Eliáš•890
    @martinelias1312Posted almost 3 years ago

    Hello @SachinShelke7, my few suggestions:

    • main content should be wrapped in <main> element
    • <div class="attribution"> can be wrapped in <footer> element
    • on larger screens than 1440px, background image is not at full size, so i´ll recommend to use background-size: 100%; at <body> element
    Marked as helpful
    0
    Sachin Shelke•190
    @SachinShelke7Posted almost 3 years ago

    @smradupan Thank You For Suggestions I will do that correction.

    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