Frontend Mentor - QR code component solution

Community feedback

• 

  dreamCod3r•40

  @dreamCod3rPosted over 1 year ago

  Hi! I reviewed your solution and I have to say that overall it was well-structured and helpful. I have a couple of suggestions, though. First, it is highly recommended to not leave the alt attribute in an image tag empty. Second, you could use min-height instead of height in the body selector. These are just some things I've learned from the internet.

  Marked as helpful

  1

  ---

  

  Leonardo•30

  @caveraPosted over 1 year ago

  @dreamCod3r you're totally right! I was thinking mostly on the aesthetics part and I forgot about the accessibility part. Thanks!

  1