Skip to content
Submitted about 3 years ago

Mobile-first solution using HTML/SASS/CSS

sass/scss
DkP-Consult•60
@DkP-Consult
A solution to the QR code component challenge
Preview (opens in new tab)Code (opens in new tab)

Solution retrospective

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

Code
Couldn’t fetch repository

Please log in to post a comment

Log in

Community feedback

  • Abdul•8,560
    @Samadeen

    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

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