Skip to content
Submitted over 2 years ago

I used max-width and flex display in css, image ID

Bahtiyar Rozmetov•30
@zakEther
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...QR code component
  • HTML
  • CSS
1newbie

Design comparison

SolutionDesign

Solution retrospective

the hardest thing for me was figuring out how to get the picture right inside the container. It took more than 7 hours to create a solution

Community feedback

  • Malik Safwan•60
    @ssafwannPosted over 2 years ago

    It's okay if it took 7 hours because you did a good job. I have some suggestions for you, next time instead of setting a width and height on the image, just set width: 100% to the image. By setting it the width to 100%, the image will be 100% the width of the container. The picture will then always be right inside the container. This will really help you later on when you want an image to responsive to its container. Other than that good job, I also like the box-shadow addition you added.

    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