chore(SkipToContent): updated a11y docs to template

[thatblindgeye]

Closes #3825

[patternfly-build]

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

[jessiehuff]

I find this description just a tad confusing. I previously tried to explain what to expect with the keyboard interactions surrounding the skip to content, that it was essentially still in the tab order and that users could interact with it or move past it/that it doesn't force the user into using it. I think the part that I find particularly confusing is this line, "Shift Tab navigates to the previous focusable element or the skip to content." For some reason this feels odd, like we're just describing general keyboard interactions?

[thatblindgeye]

That's a fair point, yeah this is basically the keyboard navigation content that is used in other components' a11y docs. It does end up a bit wordy because it's sort of a special case in that the first Tab on a page should focus the Skip to Content.

Do you think this item is necessary, or should we just omit the Shift + Tab part? Could also reword it a little, though not sure if this really improves it significantly:

Depending on where focus is on the page, <kbd>Tab</kbd> navigates to the skip to content or the next focusable element, and <kbd>Shift</kbd> + <kbd>Tab</kbd> navigates to the previous focusable element or the skip to content. Since the skip to content should be the first focusable element on a page, the previous focusable element may typically be an item in the browser toolbar.

Basically trying to convey that Tab and Shift + Tab work and what the expected focus should be.

[jessiehuff]

Yeah, I think the edit makes it a lot clearer. It adds context so I better understand what you're saying. I'm good with that, but if you think it's better to remove it altogether, that's fine with me too.

[thatblindgeye]

@jessiehuff pushed an update!

[jessiehuff]

Awesome job as always! 🎉