QR Code Component with JS, HTML and CSS

Community feedback

• 

  Travolgi 🍕•31,400

  @denieldenPosted over 2 years ago

  Hey Cam, congratulations on completing the challenge! You did a great job 😉

  Let me give you some little tips for optimizing your code:

  • replace header tag with main tag to improve the Accessibility... main is for content of page and header is ordinary for head or navbar of page
  • remove all unnecessary code, the less you write the better as well as being clearer: for example the div container of image
  • instead of using px use relative units of measurement like rem -> read here

  Hope this help! Happy coding 😁

  0
• 

  Kamasah-Dickson•5,590

  @Kamasah-DicksonPosted over 2 years ago

  Your solution looks great but on smaller devices the card span horizontally across the screen. You can add margin left and right to fix that.

  Besides good job👍 Happy coding👍

  0