Skip to content
Submitted 3 months ago

HTML and CSS only. Vercel deployed

André Ferreira•20
@Andre-POF
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

What are you most proud of, and what would you do differently next time?

I have so much to learn! I feel it was a good first challange anyway!

Community feedback

  • Patolin3084•20
    @Patolin3084Posted 3 months ago

    Hi! These suggestions were made to me at work and I found them very useful: HTML:

    Add a <main> element to wrap all of your content except the <header> and <footer>. This improves accessibility and acts as a landmark for your webpage. Make sure to include your card within the <main> element.

Consider changing your <p class="para1"> tag to an <h2> tag. Typically, a page should only have one <h1> tag for the main heading. Since this challenge doesn’t include a main heading, it would be appropriate to use an <h2> tag instead.

    I hope they serve you too.

    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