Skip to content
Submitted over 2 years ago

Qr code HTML/CSS flexbox

Adam Mā€¢550
@AdamMzkr
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...QR code component
  • HTML
  • CSS
1newbie

Design comparison

SolutionDesign

Solution retrospective

Hi, I come back from a 1year break to the front-end and I try to remind important topics. I hope with the realization of the front-end mentor challenge my skills come back and even grow.

I am ready for critics :)

Community feedback

  • Shashree Samuelā€¢9,260
    @shashreesamuelPosted over 2 years ago

    Hey good job completing this challenge.

    Keep up the good work

    Your solution looks great however I think the border radius can be decreased just a little bit.

    In terms of your accessibility issues simply wrap all your content between main tags

    I hope this helps

    Cheers

    Happy coding šŸ‘

    Marked as helpful
    0
  • Byronā€¢2,290
    @byronbyronPosted over 2 years ago

    @AdamMzkr Looks great!

    Just missing a <main> tag, that's all.

    Nice job šŸ‘

    Marked as helpful
    0
  • Adam Mā€¢550
    @AdamMzkrPosted over 2 years ago

    I did not expect such a large feedback;)

    Thanks for all advice I will update code.

    Till next time.

    0
  • Yehan Nilangaā€¢670
    @Yehan20Posted over 2 years ago

    It looks really great , Nice job

    0

Please log in to post a comment

Log in with GitHub

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord