Skip to content
Submitted over 2 years ago

QR Code Component Project

Brendan Wilsonā€¢70
@Brendan-A-Wilson
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! This is my second project on Frontend Mentor, which was similar to the Order Summary. Any feedback is most welcome.

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 card needs some margin from the top using margin-top

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

    I hope this helps

    Cheers Happy coding šŸ‘

    1
    Brendan Wilsonā€¢70
    @Brendan-A-WilsonPosted over 2 years ago

    @TheCoderGuru thank you for the feedback, Shashree. Will remember to add those changes for next time.

    0
  • Kelvinā€¢915
    @Kl3vaPosted over 2 years ago

    Try using html semantics and always give your page a h1ā€¦ Your challenge looks nice

    1
    Brendan Wilsonā€¢70
    @Brendan-A-WilsonPosted over 2 years ago

    @Kl3va thanks for the feedback, Kelvin. Will look into making those changes.

    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