Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Frontend Mentor - QR code component solution

Leonardo 30

@cavera

Desktop design screenshot for the QR code component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Community feedback

@dreamCod3r

Posted

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

@cavera

Posted

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

1

Please log in to post a comment

Log in with GitHub
Discord logo

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