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

QR Card Component

@boseriddha

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

@purnimakumarr

Posted

Hi Riddha, congrats on completing you first challenge 🙌

Here are some suggestions to make you code more accessible:

  1. Using an <h1></h1> tag. It is a good practice to use heading tag for the heading in a website and as the name suggests an <h1></h1> is used for the very first heading of the website. So, you can replace <p class="desc-heading">Improve your front-end skills by building projects</p> with <h1 class="desc-heading">Improve your front-end skills by building projects</h1>

  2. The alt-text="qr-code" isn't very descriptive. alt-text attribute is used to describe an image so that users using screen readers can know what an image is about or what is the purpose of using the image. Something like alt-text="A QR code to Frontend Mentor" would be better.

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