QR Code Component Using Tailwind CSS

Community feedback

• 

  Melvin Aguilar 🧑🏻‍💻•61,140

  @MelvinAguilarPosted about 1 year ago

  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

  @yogskrPosted about 1 year ago

  @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

  @AakashGujarPosted about 1 year ago

  Hey great one, I just completed this challenge too✌️

  0

  ---

  

  Yoga•240

  @yogskrPosted about 1 year ago

  @Bexst24

  I saw yours, it's crazy I like it 😄👍

  0