Responsive QR code component Page Using CSS Flexbox

Community feedback

• 

  PTMeen•150

  @PTMeenPosted about 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 ✌

  Marked as helpful

  1
• 

  Aviral Sharma•220

  @aviralsharma07Posted about 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!

  Marked as helpful

  0

  ---

  

  Deepanshu Bundel•20

  @deepanshu1894Posted about 2 years ago

  @aviralsharma07 I'll Surely will implement your suggestions in my next project, thank you.

  1