Frontend Mentor - QR code component solution

Community feedback

• 

  Lucas 👾•104,400

  @correlucasPosted over 2 years ago

  Hello Anie! Welcome to the Frontend Mentor community!

  I'm looking your solution right now and I can see you've done a great job wrapping up all the content inside the div container, consider use main instead of div to improve the semantics. I like that your card is flexible even in smaller screens situations. The only issue you should fix is the image that is stretching with wrong proportions.

  I notice that you've added some properties to the image, but what cause the image stretch is that you add width: 100%; and a different size for the height --> height: 300px; note that if you add display: block; to the image and you set width with 100% there's no need to set the height, by default this value come in automatic.

  .img-box img { width: 100%; max-width: 300px; }

  Hope it helps you and I want to see you around with more challenges! Keep coding!

  Marked as helpful

  0

  ---

  

  Anie•100

  @A-n-i-ePosted over 2 years ago

  @correlucas Thanks for the feedback! I have made the changes 😊​ The name's Anie by the way ​🙃​

  1

  

  Lucas 👾•104,400

  @correlucasPosted over 2 years ago

  @A-n-i-e Hey Anie, sorry if I wrote your name wrong 😅, wasn't on purpose. I'm happy that the tips worked for you. Congrats. 😊

  I fixed my first comment, now your name is right ❤

  1