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

QR Code Component Using Tailwind CSS

Yogaโ€ข 240

@yogskr

Desktop design screenshot for the QR code component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


Here is my first front-end mentor challenge. I built the page using Tailwind CSS. Feel free to give me feedback!

Community feedback

@MelvinAguilar

Posted

Hello there ๐Ÿ‘‹. Good job on completing the challenge !

I have some suggestions about your code that might interest you.

  • The alt attribute should not contain the words "image", "photo", or "picture", because the image tag already conveys that information.
  • Instead of h-screen (height: 100vh), use min-h-screen (min-height: 100vh) as the class for the body element. min-h-screen ensures that the body element takes at least the full height of the viewport, which is useful for content that might exceed the screen height.

I hope you find it useful! ๐Ÿ˜„ Above all, the solution you submitted is great!

Happy coding!

Marked as helpful

1

Yogaโ€ข 240

@yogskr

Posted

@MelvinAguilar

Hey, thanks for the feedback and suggestions!

  • You're right about the alt attribute. I'll change it for usre.
  • I totally didn't think about using min-h-screen at all. Thanks for pointing that out.

Cheers!

0
Aakash Gujarโ€ข 10

@AakashGujar

Posted

Hey great one, I just completed this challenge tooโœŒ๏ธ

0

Yogaโ€ข 240

@yogskr

Posted

@Bexst24

I saw yours, it's crazy I like it ๐Ÿ˜„๐Ÿ‘

0

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