QR-code component

Community feedback

• 

  Travolgi 🍕•31,400

  @denieldenPosted almost 3 years ago

  Hi Eero, great job! Congratulations on completing the challenge.

  For remove the scroolbar reset the margin of body:

  body {
     margin: 0;
  }
  

  Overall you did well :)

  Hope this help and happy coding!

  Marked as helpful

  0

  ---

  

  Eero Liuska•60

  @eerolliPosted almost 3 years ago

  @denielden Oh yeah, missed the scrollbar completely. Thank you!

  1

  

  Travolgi 🍕•31,400

  @denieldenPosted almost 3 years ago

  @eerolli You are welcome! I would really appreciate if you mark my comment as helpful if it helped you, thank you very much :)

  0
• 

  2div•170

  @2divPosted almost 3 years ago

  Hi good work.

  I am new in field and i was just finish this challenge and i had many issues , but i got tips from the community to improve my practice.

  I see you use grid layout and it worked well. However i used flex box layout because its very small requirement and doesn't need much code.

  I think the practice its for flex out design.

  this is my humble tip. Again good work

  0
• 

  T

  Chamu•13,130

  @ChamuMutezvaPosted almost 3 years ago

  Well done. The site looks good

  0

  ---

  

  Eero Liuska•60

  @eerolliPosted almost 3 years ago

  @ChamuMutezva Thank you very much!

  0