responsive qr code with pure css

Community feedback

• 

  Marko•80

  @smarko-webPosted almost 2 years ago

  First off, Congratulations on your project!

  Some improvements:

  • Add minimum height to the body (use vh unit) to have everything wrapped in the body
  • The way I did mine is using flex box, I encourage you to learn it and apply it to the card

  Hope that helps

  Marked as helpful

  0
• 

  Sinisa Vukmirovic•1,690

  @SinisaVukmirovicPosted almost 2 years ago

  Hello!

  That's an... interesting way to make it responsive... Sadly, that's not the right way.

  Its a lot of work, plus, it doesn't look that good. And this is a simple, tiny project, single card, what when you do a bigger, full page/screen project?

  Check out others code solution to get the idea how its done.

  Good luck!

  Marked as helpful

  0
• 

  Hassia Issah•50,650

  @HassiaiPosted almost 2 years ago

  Replace <div class="row"> with the main tag to fix the accessibility issues. <div class="attribution"> must be within the footer tag not in the main/ <div class="row">

  Give p a font size of 15px.

  Increase the Width of .attribution for it match the width in the design and decrease the margin value for .textarea.

  Use rem or em as unit for the padding, margin, width and preferably rem for the font-size for more on CSS units watch this https://youtu.be/N5wpD9Ov_To

  Hope am helpful HAPPY CODING

  Marked as helpful

  0

  ---

  

  g-zan•10

  @gzandiemerPosted almost 2 years ago

  @Hassiai thank you so much!

  0