Skip to content
Submitted almost 2 years ago

Order summary component

Lagxy•110
@Lagxy
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 accept any suggestion/tips to improve my skills

Community feedback

  • Hassia Issah•50,650
    @HassiaiPosted almost 2 years ago

    Replace <section class="card"> with the main tag and <h4> with <h2> to fix the accessibility issue.

    The background-image is not visible, to make it visible add"./" to the beginnig of url value. background-image: url('./images/pattern-background-mobile.svg'); instead of background-image: url('images/pattern-background-mobile.svg'); Repeat this in the media query for the desktop design.

    There is no need for background-position in the body.

    Hope am helpful.

    Well done for completing this challenge. HAPPY CODING

    Marked as helpful
    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