Responsive QR code component

Community feedback

• 

  Hania B.•1,280

  @techanthere

  Posted over 3 years ago

  Hey, great job with the first solution, it looks perfect visually.

  Just a few things I will suggest to change in html structure. Use h1 instead of h4 and then you can change font-size for h1, you should use headings in order. And you don't really need to set explicit width and heights on ImageCenter class.

  A simple way can be first setting width on card like here the first div with class "center" takes some fixed width for the whole card size (no height required) and then when you set width: 100% on img inside it automatically fits the card width. You can apply vertical and horizontal padding on the div with class "ImageCenter" then.

  I will also suggest to use the rem units instead of px, they are more flexible and will be helpful in future projects as well.

  Happy Coding :)

  11

  Marked as helpful