Submitted about 5 years ago
Responsive page using Html5, CSS3, JavaScript and FlexBox
@B-Shadoink
Design comparison
SolutionDesign
Community feedback
- @mattstuddertPosted about 5 years ago
Really nice work on this challenge Brahim! It looks great and I like the way you've done the mobile navigation. There's just a small accessibility issue that it's worth pointing out:
- You're uppercasing the text for the navigation in your HTML. Be aware that screen reader software will read these words out letter-by-letter, which will make the content very hard to read for people with a visual impairment. Instead, I'd recommend lowercasing the text in your HTML and then using
text-transform: uppercase;
in your CSS to get it looking like the design.
I hope that helps. Keep up the great work! 👍
0@B-ShadoinkPosted about 5 years ago@mattstuddert glad you like it man, thanks for pointing out that accessibility issue i'll make sure i don't make that mistake again.
0 - You're uppercasing the text for the navigation in your HTML. Be aware that screen reader software will read these words out letter-by-letter, which will make the content very hard to read for people with a visual impairment. Instead, I'd recommend lowercasing the text in your HTML and then using
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