Skip to content
Submitted almost 3 years ago

HTML and CSS

renan•50
@RenanGuapyassu
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 had an issue with the background for the mobile version, i'd aprecciate if someone can help me.

Community feedback

  • Saran•460
    @Saran-73Posted almost 3 years ago

    Hi Renan, I just want to mention a few things:

    • Change the container <div> to <main> tag to resolve the accessibility issue.

    • You don't have to set the background-size:100% or cover and use background-repeat: repeat-x; in both of your background patterns.

    • Then change your body background-color: hsl (225, 100%, 94%); to this value.

    Best of luck, hope this helps.

    Marked as helpful
    0
    renan•50
    @RenanGuapyassuPosted almost 3 years ago

    @Saran-73 Thanks a lot, it was very helpful. I'm a complete beginner, so much to learn...

    1

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