Skip to content
Submitted about 3 years ago

HTML CSS

Maria•120
@Maria1324
A solution to the Product preview card component challenge
Preview (opens in new tab)Code (opens in new tab)
Code
Couldn’t fetch repository

Please log in to post a comment

Log in

Community feedback

  • Lucas 👾•104,160
    @correlucas

    Hello Maria, congratulations for your challenge solution!

    I've looked your solution and there's a minor issue with the background-color. Note that you've inserted the background inside the .container, its better you insert the background-color <body> to display it filling all the screen. If you look the mobile version It will show that your background is limited by the container bounds.

    You've also applied height: 10vh; to the mobile media querie, this will cause you issues because will display only 10% of the viewport height screen. Fix that like this:

    @media (max-width: 545px) body { height: 100vh; }

    I hope it helps you Maria and happy coding!

    Marked as helpful

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