Skip to content
Submitted almost 3 years ago

Mobile-first solution using flexbox- QR Code Component

KENDRA•50
@KNWard
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

What are you most proud of, and what would you do differently next time?

I completed this challenge a long time ago, I don't remember.

What challenges did you encounter, and how did you overcome them?

I completed this challenge a long time ago, I don't remember.

What specific areas of your project would you like help with?

I completed this challenge a long time ago, I don't remember.

Community feedback

  • Rimka•600
    @karimfilaliPosted almost 3 years ago

    Hello, congratulations for finishing your first FrontEndMentor challenge ! Your card is very great, the sizes fit good with the expected design. However, you have some accessibility issues. To fix them, you must have a <main> tag in your body. The .container div and the .attribution div must be int the main tag Also, every website must have a h1 for accessibility reasons. Your h2 should be a h1. But use <h1> tag only once in main heading element.

    Very great job though !

    Marked as helpful
    0
    KENDRA•50
    @KNWardPosted almost 3 years ago

    @karimfilali Thank you so much. I think I was in such a rush to get it done I didn't even notice the little mistake. Thanks again.

    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