Skip to content
Submitted over 2 years ago

Responsive card using flexbox. No media queries

Jesse•110
@vinshield
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

Hi, thanks for checking out my solution. I will appreciate any tips on making my html more semantic

Community feedback

  • Kyle•50
    @CaptainKavemanPosted over 2 years ago

    Hey Jesse,

    To fix your accessibility issues and improve the semantic HTML you requested you should wrap the card divs in a <main></main> tag and the footer divs in a <footer></footer> tag. It is good practice to include these each time. In this case you don't have a need for the header and nav tags, but those are the main four that will typically need to be in the body.

    1
    Jesse•110
    @vinshieldPosted over 2 years ago

    @CaptainKaveman Thanks!

    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