Responsive QR code component Page Using CSS Flexbox

Community feedback

• 

  PTMeen•150

  @PTMeen

  Posted over 2 years ago

  Hello friend, you did a great job! just increase the padding a bit to make it look closer to the design.

  Happy coding ✌

  10

  Marked as helpful
• 

  Aviral Sharma•220

  @aviralsharma07

  Posted over 2 years ago

  Hey Deepanshu, Congratulations on completing this Project. Here are a few suggestions for you:

  • Every page must have an <main> Element. Unnecessary <div> is not required. Use <main class="container"> instead of <div class="container">
  • Try to use an HTML semantics tag by wrapping the Card in <article> tag instead of a <div>
  • Use <h1> instead of <h3>
  • Use a <footer> tag for Page Footer than <div>
  • in the .container class use min-height instead of just height.

  Everything else looks fine to me. Hope it helps. Great work, keep it up!

  01

  Marked as helpful