Hello, my younger self. How are things? I understand you're learning about the amazing world of web accessibility, something I wholeheartedly endorse.
Actually, that's why I'm writing to you. What I'm about to tell you might hurt, but I think you can handle it. There are some things you're doing right now, things that you might think are helpful. Well, uh, they really aren't. In fact, they're pretty terrible.
First, why so many
navs on a single page? Multiple
nav elements can create a lot of unnecessary noise for screen reader users. Remember what the
[nav element spec](https://html.spec.whatwg.org/multipage/sections.html#the-nav-element) says:
Not all groups of links on a page need to be in a nav element — the element is primarily intended for sections that consist of major navigation blocks [emphasis mine].
Major navigation blocks. Not every single link list in the
footer. C'mon, man!
Oh, and if you must have multiple
navs, make sure you label any that aren't the primary site navigation:
<!-- links in here -->
Alternatively, you could use
<!-- links in here -->
Just keep in mind that in the far, far distant future of 2021, content in
aria-label still isn't translatable.
Here's a snippet of actual code you wrote with your actual hands:
<a role="menuitem" class="..." href="..." aria-expanded="false" aria-controls="..." aria-haspopup="true">
<!-- more insanity omitted -->
I see you've learned about
aria attributes. That's...nice.
There's just so much wrong here, but what I want to focus on is the
role="menuitem". All menu roles are meant for application menus, not navigation links. Unless you're building the browser equivalent of Photoshop, stay away from them.
Also, I don't know what you're up to with all those
aria attributes, but I know it can't be good.
Remember when you coded up that swanky dropdown menu that faded in and out? Yeah, that was slick! Remember the CSS you used?
transition: opacity 200ms;
// other stuff
First of all, kudos on animating
opacity. You're thinking about performance, and that's a Good Thing.™ But I think you're missing something.
Did you know that elements set to
opacity: 0 can still be read by screen readers? By the look on your face, I can see you didn't know that. Something else you probably didn't realize is since
opacity: 0 doesn't remove elements from the focus order, keyboard users get to tab through all 500 invisible links in your mega site nav.
If you are hiding elements for sighted users to reduce cognitive noise, you should do the same for screen readers. Remember that accessibility is about creating equivalent experiences.
display: none is a much better choice here to completely hide the panel from everyone.
What's that? You can't animate
opacity when you're using
There's more to discuss, for sure, but frankly, I don't have the strength to go on. Perhaps some other time. I want to leave you with some advice.
First, one day you'll want to write a daily blog post for an entire month. Make sure you do more planning so you don't end up writing weird stuff like this at the last minute. Second, reach out to me through this magical time machine internet connection and say hi to me on Twitter. I get lonely sometimes.
See you in the future!