Skip to content
Submitted over 2 years ago

Mobile-first solution using HTML/SASS/CSS

DkP-Consult•60
@DkP-Consult
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

Hello World, Can we do a quick code review ? It's for increase clean coding skill. Thanks !

Community feedback

  • Abdul•8,540
    @SamadeenPosted over 2 years ago

    Hey!! Cheers 🥂 on completing this challenge.. .

    Here are my suggestions..

    • You should use <main class="card"> instead of <div class="card">.
    • Go down orderly when you are using the headings h1 down to h2 down to h3 and so on.

    This should fix most of your accessibility issues

    . Regardless you did amazing... hope you find this useful... Happy coding!!!

    Marked as helpful
    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