FIRST CHALLENGE QR code with pure CSS using flexbox

Community feedback

• 

  Michael East•40

  @MrEasty94Posted over 2 years ago

  Hey Andre, great job.... here's some feedback...

  1. I think you could use a <main> tag to wrap the card, I think this would be semantically correct.
  2. I think the 'attribution' section could be in a <footer> tag, again I think this would be semantically correct.
  3. For this, and future challenges, I would recommend setting your box-sizing to border-box. This makes calculating widths, heights, paddings, etc... a lot easier moving forward. One common way to do this is in your CSS, doing the following:

  *, *::after, *::before { box-sizing: border-box; }

  This sets all elements, including the after and before psuedo elements to the border-box model.

  Great effort and keep going! :)

  Marked as helpful

  1

  ---

  

  Andre Franco•70

  @AFranco-devPosted over 2 years ago

  @MrEasty94 thank you a lot for the feedback! I will apply what you said on my next challenge :) the last one will be really useful :D

  0