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

Responsive component using flexbox

Oliver 40

@OliverNikolovski

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


If you have any feedback about improving the code please let me know.

Community feedback

@srikargunnam

Posted

Hi Oliver Nikolovski,

First I would like to congratulate you on completing the challenge 🎉

I have few suggestions for which you can work on

  1. You must use h1 tag (level one heading) in your HTML as per Web Accessibility standards, so better swap your h3 tag with h1 and use CSS for all the required styles.

After you commit your code to GitHub with necessary changes, you need to check out the "view report" option under your submitted solution and re-generate the report to see if any accessibility issues still exists, this way you can make required changes to your code.

Hope this helps 😉, please mark as helpful if it does.

Marked as helpful

1
Travolgi 🍕 31,420

@denielden

Posted

Hi Oliver, great job! Congratulations on completing the challenge.

You can add main tag and wrap the card for improving Accessibility.

Hope this help ;) and happy coding!

Marked as helpful

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