Responsive layout using flexbox for QR code exercise
Design comparison
Community feedback
- @anoshaahmedPosted almost 3 years ago
hey good job on this challenge! your report above shows a few issues though:
-
wrap everything in your body in
<main>
OR use semantic tags! if you're usingdiv
right afterbody
, then give them arole=""
; however, it's a little frowned upon to userole
right after body, so try to stick with semantic tags instead. you can read more about landmarks here -
add
alt=""
to your<img>
tags
here is a list i made of accessibility issues & best practices
hope this helps :))
Marked as helpful1 -
- @NaveenGumastePosted almost 3 years ago
Hay ! Good Job you made it look nearly perfect to the preview
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.
-> always use the "alt attribute" and just add it and keep it empty. There is no ('' backslash in adding the file location, use '/' forward-slash) [This is for HTML issue]
Keep up the good work!
Marked as helpful1
Please log in to post a comment
Log in with GitHubJoin 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