QR Card Component

Community feedback

• 

  Purnima Kumar•390

  @purnimakumarrPosted over 2 years ago

  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