All comments

• 

  Jishnu Jeevan• 10

  @JishnuJeevan

  Submitted 3 days ago

  

  QR Code Component Solution

  2

  20

  

  Ammon Quarshie• 20

  @niiquash

  Posted 2 days ago

  1. I love how you used variables in your css. Overall this turned out really great.
  2. On the topic of semantic html, I think you used a lot more div elements than normal. To ensure a more sematic approach, try using a structure that kinda tells a story. For example, I had this structure within the body element: "<article>

  <figure> <img src="" alt=""> <figcaption>bold font here</figcaption> </figure> <p>small font here</p> </article>"

  And then from there you can add the classes you want and style them from there. Great job overall.

  0