Confusing accessibility help message in an editor

[joyceerhl]

1. Open settings.json
2. Run Open Accessibility Help
3. See this message
4. I don't really understand what "Disable the aria hint label to open this" means

[meganrogge]

Yeah, I've wondered about this wording. It means disable the aria hint that told you to open this (the accessible view). Perhaps replacing "this" with "the accessible view" would be better?

[joyceerhl]

Oh, do you mean that you would user Alt+F6 to disable the aria hint? Maybe it could say that?

[aiday-mar]

Hi, not sure if I am incorrectly verifying. I do not see any text relating to the aria-hint in my accessible help view?

[meganrogge]

the hint only exists for features which have a corresponding accessibility.verbosity setting. For example, copilot inline/panel chat. @aiday-mar

[meganrogge]

In fact, it was a different bug that Joyce was seeing this when she invoked the help dialog in the editor. That was fixed as well. #191722

[joyceerhl]

Text is better now but imo could still be improved

I'd expect "Toggle the accessibility help for this feature using Alt+F6"

[meganrogge]

It's not a toggle, rather a one-way action by design as users will almost never want to enable a hint they're familiar with.

[joyceerhl]

Right, so "Disable the accessibility help for this feature using Alt+F6"

[meganrogge]

Accessibility help and verbosity are different things.

[joyceerhl]

My bad, "Disable the accessibility verbosity for this feature using Alt+F6"

The point is that the current text doesn't make clear the connection between the parenthesized text and the directive preceding it

[meganrogge]

Oh, for screen reader users, including the keybinding in parenthesis is expected and a recommended practice. It reduces verbosity.

[joyceerhl]

Then should the preceding sentences in the accessibility help for the chat response follow that pattern, instead of "To do __, invoke the __ command"?

[meganrogge]

I thought that would be wordy/redundant:

To disable verbosity for this feature, invoke the command Disable Accessibility Verbosity (alt+f6).

Also, unlike the others, this command doesn't appear in the command palette, so it's not important to indicate the command title here IMO.

[meganrogge]

It is aligning with the accessible view specific actions like "exit this dialog via the escape key". the spacing there currently looks wrong though (no space, not on its own line)

[meganrogge]

I could also tweak that like so for more consistency Exit this dialog (Escape).

[meganrogge]

I see your point though. We should improve the consistency here and also probably lessen the redundancy.