Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Responsive main page using display flex

P

@AlisCodeSpace

Desktop design screenshot for the QR code component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Community feedback

@ErwiniaDev

Posted

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!

0
Maalu 10

@mittens67

Posted

  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"
0

Please log in to post a comment

Log in with GitHub
Discord logo

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