Skip to content
Submitted over 2 years ago

Responsive QR Card Component using CSS Variables

Brian Johnson•210
@BrianJ-27
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

Any feedback and thoughts about my code are welcome! I am always looking on better ways to make my code more DRY and understandable

Community feedback

  • Brian Johnson•210
    @BrianJ-27Posted over 2 years ago

    Hi Naveen! Thanks for your feedback. So yes I already added the main tag but what I didn't add when I first published this project was the h1 tag (which I believe caused the accessibility issue). I went back and updated the code to include that but it didn't refresh and take away the accessibility flag.

    0
  • Naveen Gumaste•10,480
    @NaveenGumastePosted over 2 years ago

    Hay ! Brian Good Job on challenge

    These below mentioned tricks will help you remove any Accessibility Issues

    -> Add Main tag after body <main class="container"></main>

    -> Learn more on accessibility issues

    Keep up the good work!

    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