qr-code-component-main

Community feedback

• 

  バレンタイン 😈•64,170

  @VCaramesPosted about 2 years ago

  Hey there! 👋 Here are some suggestions to help improve your code:

  • Add a border-radius to the NFT image to better match the FEM design.

  • The Alt Tag description for the QR image needs to be improved upon. Its needs to tell screen reader users what it is and where it will take them to when they scan it.

  • The "Improve your front-end skill by building projects" needs to wrapped in an Heading Element.

  • Change width to max-width in your component’s container to make it responsive. You will also want to remove the height as it is unnecessary.

  • Change width to max-width: 100% in your image to make it responsive. You will also want to remove the height as it is unnecessary.

  If you have any questions or need further clarification, let me know.

  Happy Coding! 👻🎃

  0

  ---

  

  Amr Shamy•30

  @ShaaamyPosted about 2 years ago

  @vcarames thank u <3

  0