Skip to content
Submitted 10 months ago

Responsive main page using display flex

AlisCodeSpace•110
@AlisCodeSpace
A solution to the QR code component challenge
Preview (opens in new tab)Code (opens in new tab)
Code
Couldn’t fetch repository

Please log in to post a comment

Log in

Community feedback

  • ErwiniaDev•100
    @ErwiniaDev

    Hello,

    Forgive me, this is my first peer review...

    The only thing I can see that might be missing is a padding-bottom on the bottom of your card.

    Nice work I think!

  • Mitha B•150
    @mittens67
    1. The content can be wrapped in a semantic HTML element like <main> or <section>.

    Ex:

    <main class="main--page"> <div class="qr--cont"> ... </div> </main>

    This will enhance the site's accessibility by clearly defining the primary content area.

    1. The alt text is a bit generic. Making it more descriptive can help users relying on screen readers. Ex: alt="Scan this QR code to visit Frontend Mentor's website"

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