docs(accessibility-fundamentals): Updates content as part of audit.

[edonehoo]

Closes #3741

[patternfly-build]

Preview: https://patternfly-org-pr-3762-site.surge.sh

[edonehoo]

thoughts on changing the nav item to "about"?

[jessiehuff]

In general, this all looks really good! :) There's a typo in this sentence that really stood out to me though. Since we're separating that bullet from the guideline, can we add a little bit of specificity? Maybe something like, "Make content appearing on hover available to a screen reader by using aria-describedby."

[thatblindgeye]

Some nits/comments below.

Regarding the page name update, the only reason I'd be against the rename is that we already have an "About" page in several other sections. idk how often a user would have a bunch of of the side nav sections expanded, but in that case there'd be a lot of links with just "About" as its text announced.

[thatblindgeye]

Wondering if we could remove the hidden paragraph and just update the image alt text...

alt="An accessibility spectrum showing examples of the situational, temporary, and permanent situations for different impairments." maybe?

Was removing this image also something we'd want to consider in a follow up? I think we discussed the image in general a bit in the google doc, but not sure if we landed 100% on anything.

[edonehoo]

I personally think the image should be reworked at the very least, but also wasn't sure where we landed. It's tough because I do like the information it conveys, it's just so large.

Maybe it should just be a followup to remove/replace/repurpose the image? An idea I had was to cut the image up so that each "row" is placed within the relevant section (for example, the "sight" row is placed under the "vision" section), but not everything in that image has a place in that case.

[thatblindgeye]

That could definitely be a followup. I do like at least cutting the image up to several, smaller ones as well.

[thatblindgeye]

This is looking good! Other than the open convo about that image, the only other thing from me is the page title/url. It feels like we should update other instances of "About" links as well, but for this PR could we revert back to "Accessibility fundamentals", or maybe "About accessibility"?

For context, here's what the VO rotor menu looks like if I have all of the expandable side nav sections opened:

[edonehoo]

@thatblindgeye oo this is really good to know about the "about" pages. It definitely seems like we should update nav headings across the site in that case. I'll look into opening a separate issue. I think it could be best to go for "About XYZ", because it'll help cue users to visit that page first - but open to objections to this!

And also I'll go for opening a follow up issue to address the image

[jessiehuff]

LGTM! Also, I'm open to continuing the discussion about the image. I think it would make sense to include @maryshak1996 since she was the designer for the image and might have thoughts about how we can best adapt it to the page. I know that she created a few versions of it, but I'm not quite sure where those are right now.