-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
add placeholder support for new at additionalSettings #73
base: main
Are you sure you want to change the base?
Conversation
@@ -41,6 +41,7 @@ | |||
* @property {string} commands[].keypresses[].keystroke single human-readable key or key chord press | |||
* @property {string} [commands[].extraInstruction] human-readable additional instruction to follow | |||
* @property {string} [commands[].settings] this property only exists on v2 tests | |||
* @property {string[]} [commands[].additionalSettings] this property only exists on v2 tests |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This change isn't present in Howard's patch. Since the file needs to match the corresponding file in the aria-at repository, we'll need to resolve the discrepancy before merging.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yep, i noticed that too! i have a PR open to Howard's patch here: w3c/aria-at#1128
a555c2c
to
46a6278
Compare
I don't think we can merge this until w3c/aria-at#1125 lands, so I'll leave that part to you! |
PR to support changes in w3c/aria-at#1125. See https://gist.github.com/howard-e/89a06d917946f5e922740dccee08d4e2 for more info.
All of the new "additional settings" that will be added are for JAWS and the harness + automation doesn't support JAWS, so this PR creates placeholders for where we would support potential settings on NVDA and VoiceOver when they are added in the future.