Microsoft/vscode

Parameter hints should not use aria.alert

Open

#90,874 创建于 2020年2月18日

在 GitHub 查看
 (7 评论) (1 反应) (2 负责人)TypeScript (10,221 fork)batch import
accessibilitydebteditor-parameter-hintshelp wanted

仓库指标

Star
 (74,848 star)
PR 合并指标
 (平均合并 11小时 43分钟) (30 天内合并 1,000 个 PR)

描述

Instead they should use activeDescendent same as suggest widget. Code pointer where parameterHints are currently using aria alert https://github.com/microsoft/vscode/blob/8451f8bd913914aac17b6b3273b6ed0db137a217/src/vs/editor/contrib/parameterHints/parameterHintsWidget.ts#L251

As @pawelurbanski points out: Using activeDescendant for parameter hints.

  • Currently, the hints are outputted by the alert function, which has 2 nasty attributes:
  • If you do not modify the message with a counter, repeate messages are filtered out,
  • If you modify the message you go crazy after hearing: string appeared 29 times,
  • When there is more than 1 paremter all the items are read as a single string with no pauses,
  • The alert is an offscreen DOM element that is not-focusable so you can’t scroll through the items.
  • I just disable the hints all together and use Ctrl+Shift+Space every now and then…

fyi @alexdima

贡献者指南