qr-code-component-main using scss

Community feedback

• 

  Travolgi 🍕•31,400

  @denieldenPosted almost 3 years ago

  Hello Say, great job! Congratulations on completing the challenge.

  I had a look at your solution:

  • add main tag and wrap the card for Accessibility and don't forget the img element must have an alt attribute with description text, it's very important!
  • try to remove all marginfrom subdiv class and use flexbox to the body for center the card. Read here -> flex guide
  • also set heigth of body to 100vh because Flexbox aligns to the size of the parent container.

  Overall you did well! Hope this help ;)

  0