QR code component

Community feedback

• 

  Shashree Samuel•9,260

  @shashreesamuelPosted over 2 years ago

  Hey abhishekb740, good job completing this challenge. Keep up the good work

  Your design looks good however I think the font weight of your card title is supposed to be a bit lighter also the color of the card description is supposed to be lighter.

  In terms of your accessibility issues, wrap all your content between <main> tags to avoid get rid of accessibility issues.

  Also I recommend practicing using html semantic tags since they describe the intended usage of the content contained within. You can read more on html semantic tags mad it's usage here

  https://www.w3schools.com/html/html5_semantic_elements.asp

  If you have any doubt about a css property and you would like some more information on its intended usage then feel free to check out https://developer.mozilla.org/en-US/docs/Web/CSS

  I hope this helps

  Cheers Happy coding 👍

  Marked as helpful

  0
• Account deleted

  Hello there! 👋

  Congratulations on finishing your challenge! 🎉

  I have some feedback on this solution:

  • Always Use Semantic HTML instead of div like <main> <header> <footer> , etc for more [info](https://www.w3schools.com/html/html5_semantic_elements.as

  Marked as helpful

  0