Responsive qr code component using cube css

Community feedback

• 

  T

  Josh•400

  @burrijwPosted over 1 year ago

  Nice 👍🏻

  A couple things:

  • You need one (and usually only one) <main> on each page of a website.
  • Overall you could probably simplify your HTML a bit. Nothing major, but there's no need for the <header> inside the card, for example. Keep it simple.
  • Check your padding and border-radius values. I think you could get closer to the design with a bit of tweaking. No need to be pixel-perfect, but you could be closer, IMO.

  Great job -- keep it up!

  Marked as helpful

  1

  ---

  

  Hanane Boukhalfa•40

  @Hanane05Posted over 1 year ago

  Thank you for your feedback, I'll consider that. I'll be more than happy to receive your feedbacks, thanks again :) @burrijw

  0