QR Code Solution using HTML, SASS and Grid

Community feedback

• 

  Koi Hastilow•90

  @KoiHastPosted over 1 year ago

  Hey, great job on your first project! If you want, here are just a couple of tips to improve your code a bit ^_^

  • You want to wrap the main part of your HTML in a <main> element, just to improve semantics and readability.

  • Just to clean up your code, since you deleted the div.attribution from the bottom, you could delete the section of your HTML that pertained to that, as well. It would just make things a bit neater since that code isn't being used anymore.

  I'd analyze your CSS, but I'm still learning a lot of those rules myself, so I don't want to steer you wrong. I hope the HTML tips help you out, though! 🌺✌

  0