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

Basic HTML and CSS

Akashh26 10

@Akashh26

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


Please give a feedback.

Community feedback

Rimka 600

@karimfilali

Posted

Hello, congratulations for finishing your first FrontEndMentor challenge ! Your card is very great, the sizes fit good with the expected design. However, you have some accessibility issues. To fix them, you must have a main tag in your body. You can add a <main> tag to your body or add role = "main" to your .content div. Also, every website must have a h1 for accessibility reasons. Your h3 should be a h1. Very great job though !

Marked as helpful

1

Akashh26 10

@Akashh26

Posted

@karimfilali thanks for your feedback. I'll consider the corrections.

0
Naveen Gumaste 10,460

@NaveenGumaste

Posted

Hay ! Good Job Akashh

These below mentioned tricks will help you remove any Accessibility Issues

-> Add Main tag after body like it should be your container. For 1st heading or h1 tag, use header tag and then inside the header put your h1 or h2 etc . But use header tag only once in main heading element.

Keep up the good work!

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