QR code component challenge build with ReactJS

Community feedback

• 

  Alvaro Guedes•60

  @alvin-dev

  Posted over 3 years ago

  Nice work man. You forget the border-radius on image.

  01

  Marked as helpful
• 

  Web Wizard•5,690

  @rsrclab

  Posted over 3 years ago

  Hi, @AleksHNZ ~

  Congratulate on your solution to the challenge on FM platform. I have studied your work carefully and learned a lot from it.

  Here are some of the tips I like to provide.

  • Please try BEM for naming element classes. It will help you a lot on bigger projects.
  • Image tags must have alt attribute as a standard.

  If it can help you even a bit, it would be happy to me.

  Cheers ~

  01

  Marked as helpful
• Account deletedPosted over 3 years ago

  Nice work @AleksHNZ 👋

  Your design is awesome 😊. One very little detail that decreases description text by 2-4 pixels. They leek a bit high compared to the design

  Cheers Maqsud

  01

  Marked as helpful