QR code component challenge build with ReactJS

Community feedback

• 

  Alvaro Guedes•60

  @alvin-devPosted almost 3 years ago

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

  Marked as helpful

  0

  ---

  

  Ali•260

  @alihuseynzade23Posted almost 3 years ago

  @alvin-dev I tried to use border radius, but it doesn't work. Maybe because I'm sitting in the Firefox. When I did component like this before in Google Chrome it worked properly.

  0
• 

  Web Wizard•5,690

  @rsrclabPosted almost 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 ~

  Marked as helpful

  0

  ---

  

  Ali•260

  @alihuseynzade23Posted almost 3 years ago

  @tymren608 thanks a lot :)

  0
• Account deleted

  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

  Marked as helpful

  0

  ---

  

  Ali•260

  @alihuseynzade23Posted almost 3 years ago

  @maqsudtolipov thanks a lot :)

  1