Responsive main page using display flex

Community feedback

• 

  ErwiniaDev•40

  @ErwiniaDevPosted about 1 month ago

  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
• 

  Mitha B•100

  @mittens67Posted about 1 month ago

  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.

  2. 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