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 Code Component | Flexbox

Kunal 105

@kunals741

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


Feedbacks are welcomed!

Community feedback

Naveen Gumaste 10,480

@NaveenGumaste

Posted

Hay ! Good Job Kunal

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!

Marked as helpful

1

@saeedyari97

Posted

Hi, Kunal

well done, you deserved the success.

For better accessibility and achieve to better indexing, you can use HTML5 semantic tags.

for this purpose you can define a main tag immediately after body. this tag will be your main container. also define a section for your main to holding card container. each page has one main section and every main section has a header tag so you can define your heading (h1 to h6) tag in header tag.

Note: main tag just use one time per page. and every page has one and more main header (one recommended) which appears in header tag.

enjoy the coding!!!

Marked as helpful

0

Kunal 105

@kunals741

Posted

@saeedyari97 Thank you so much for the advice.

0

@mrfinesse47

Posted

looks very good, pretty much identical to the original.

1

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