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

HTML and CSS only. Vercel deployed

@Andre-POF

Desktop design screenshot for the QR code component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

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

Posted

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