-
Notifications
You must be signed in to change notification settings - Fork 127
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
aria-atomic has no default #1894
Conversation
Its computed value depends on its ancestors.
with this change, should we also not change or removing the word "default" in the following sentence:
|
I had a discussion with @cookiecrook about cases like this and what I think we want to do is distinguish between the DOM attribute and the AT property. Also to make it clear what computations are owned by the AT. The DOM attribute is "true" or "false", invalid, or missing. The AT property depends on the DOM attribute, the computed role, ancestor elements, etc. I will take a stab at demonstrating what that would mean for the specification tomorrow by creating a draft PR for a single property. |
Just to be clear, this proposed change is editorial, correct? This is related to this re-thinking of the "default value" in #1896 but we aren't changing anything normative about the spec, or any behavior of the browser? |
I'm not sure it's editorial (it's not a typo fix and arguably fixes an error), but it shouldn't have normative impact.
Reworking that would be a larger change we'd make as part of what #1896 illustrates. All this is making clear is that it doesn't have an intrinsic default. You need to read the prose to determine the default. |
@annevk said, it's not technically "editorial", but it's clearly "errata" IMO. |
SHA: 7205d35 Reason: push, by jnurthen Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Its computed value depends on its ancestors.
PR tracking
Check these when the relevant issue or PR has been made, OR after you have confirmed the
related change is not necessary (add N/A). Leave unchecked if you are unsure. Read the
Process Document or
Test Overview for more information.
Implementation tracking
Diff