Skip to content
OverviewSolutions
1
Comments
5
Bickey kumar
@Bickeykr

All comments

  • Bickey kumar•30
    @BickeykrPosted almost 3 years ago

    @NatanielCosta you are welcome.

    0
  • Bickey kumar•30
    @BickeykrPosted almost 3 years ago

    Great Job there @NatanielCosta, you can get rid of Accessibility issue by putting the attribution in a footer tag and you can give a bit padding to h1 and p. rest looks fine.

    0
  • Bickey kumar•30
    @BickeykrPosted almost 3 years ago

    You are Welcome.

    0
  • Bickey kumar•30
    @BickeykrPosted almost 3 years ago

    @Shironim great work, everything looks great and you could increase the border radius a bit. give some padding to h1 and p.

    Marked as helpful
    1
  • 👾 Ekaterine Mitagvaria 👾•7,600
    @catherineisonlineSubmitted almost 3 years ago

    Hello, Frontend Mentor community! This is my solution to the QR code component.

    I have read all the feedback on this project and improved my code. Due to the fact that I published this project very long ago, I am no longer updating it and changing its status to Public Archive on my Github.

    You are still free to download or use the code for reference in your projects but I longer update it or accept any feedback.

    Thank you

    QR code component

    20
    Bickey kumar•30
    @BickeykrPosted almost 3 years ago

    Really great work, By reviewing your solution I made changes in my solution and learned new things. So, thanks for that.

    0