QR Code component using HTML & CSS only

Community feedback

• 

  P

  Ericsson Mura 👦🏻•210

  @ericssonmura

  Posted 7 months ago

  Hello there 👋. Good job on solving this challenge!

  The solution you submitted is great!

  I have some suggestions that you might find interesting:

  • Why not adding a box-shadow on the .qr-container div? For example:

  qr-container {
    box-shadow: 0 4px 22px rgba(0, 0, 0, 0.1);
  }
  

  • When using the margin shortcut, you can omit the last value and instead use 3 values:

  .qr-img {
      width: 288px;
      height: 288px;
      margin: 16px auto 0;
  }
  

  16px [for top] auto [left & right] 0 [bottom]

  Have a great day and keep coding ! 😄

  12

  Marked as helpful
• 

  P

  Shalva Mghebrishvili•30

  @Shako-IT

  Posted 7 months ago

  I think some other HTML elements could be used here too, I mean "section", "h1", or "h2", instead of "div" and "p". I liked the HTML structure, the result is nice da looks appropriate to the challenge.

  11
• 

  Vinayagam K•180

  @vinayagamRVK

  Posted 7 months ago

  Your code to this Challenge looks simplified and also it contains some of the good practices also ......overall it is Good developer Friendly solution @TusharJetagi

  11