-
Notifications
You must be signed in to change notification settings - Fork 27
add/revise popover related attribute mappings #481
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
Changes from 14 commits
39ae862
e4694e5
9eed4ec
5fde9c0
cbd0cc6
4cad167
e64eaae
0a3851e
5829073
75d44ff
4691b00
ed8270a
82b42e3
2b36d6f
bfa65de
bcdb24e
beccc2c
ce1773d
7ed1441
1fde1de
bf0a2fe
fd42971
5e31f10
44d2432
71c0c0b
fcdd9ac
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -13594,6 +13594,208 @@ `playsinline` |
|
`popover`
|
|
|
|
|
| HTML Specification |
|
|
|
|
|
|
`popover` |
|
|
|
|
|
|
|
|
|
|
|
| Element(s) |
|
|
|
|
|
|
HTML elements |
|
|
|
|
|
|
|
|
|
|
|
| [[WAI-ARIA-1.2]] |
|
|
|
|
|
|
|
Not mapped
|
|
|
|
|
|
|
|
|
|
|
| MSAA + IAccessible2 |
|
|
|
|
|
|
Relations: `IA2_RELATION_DETAILS_FOR` points to invoking element |
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. It feels weird that we express this as a platform specific relation, but we express popovertarget's details relation as a cross-platform aria-details relation. Is this just because we have no way of expressing reverse relations for ARIA mappings and we want to be explicit here? I wonder whether it'd be better to link to a more general note about implicit reverse relations or something like that. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I'd be fine with either replacing the cross-platform aria-details relation with the specific platform mappings, or keeping the cross-platform relation and removing the platform specific and replacing with a note about the reverse relations. Whatever makes the most sense for those that need to read the spec is what I want to do, so just let me know which is preferred. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I assume this shouldn't happen if we don't expose aria-details due to the invoking element being an immediate sibling or having popovertargetaction=hide? Again, hopefully common sense will prevail, but probably best not to rely on that.
scottaohara marked this conversation as resolved.
Show resolved
Hide resolved
|
|
|
|
|
Object attributes: |
|
|
|
ispopup: <value> where <value> reflects the popover type
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| UIA |
|
|
|
|
|
|
??? |
|
scottaohara marked this conversation as resolved.
Show resolved
Hide resolved
|
|
|
|
|
|
|
|
|
|
|
| ATK |
|
|
|
|
|
|
|
|
|
Relations: `RELATION_DETAILS_FOR` points to invoking element |
|
scottaohara marked this conversation as resolved.
Show resolved
Hide resolved
|
|
|
|
|
|
|
|
Object attributes: |
|
|
|
ispopup: <value> where <value> reflects the popover type.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| AX |
|
|
|
|
|
|
??? |
|
scottaohara marked this conversation as resolved.
Show resolved
Hide resolved
|
|
|
|
|
|
|
|
|
|
|
| Comments |
|
|
|
|
|
|
If specified on an element with an implicit role of `generic`, then the element's role instead maps to `group`. |
|
scottaohara marked this conversation as resolved.
Show resolved
Hide resolved
|
|
|
|
|
|
|
|
| |
||||||||||||
`popovertarget`
|
|
|
|
|
| HTML Specification |
|
|
|
|
|
|
`popovertarget` |
|
|
|
|
|
|
|
|
|
|
|
| Element(s) |
|
|
|
|
|
|
`button`; |
|
|
`input type=button, image, reset, submit` |
|
|
|
|
|
|
|
|
|
|
|
| [[WAI-ARIA-1.2]] |
|
|
|
|
|
|
|
|
|
|
|
|
If the associated element is displayed as a popover: |
|
|
`aria-expanded=true` |
|
|
|
|
|
|
|
|
If the associated element is hidden, or not present in the DOM: |
|
|
`aria-expanded=false` |
|
|
|
|
|
|
|
|
If the associated element is an accessibility ancestor of the element with the `popovertarget` attribute: |
|
|
`aria-expanded=undefined` |
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This doesn't appear to be implemented in Chromium: There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. yes it should not be getting the expanded state for the hide button, this will need to be adjusted in chromium There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Have we filed a bug for this?
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| MSAA
|
|
|
+ IAccessible2 |
|
|
|
|
|
|
|
Use WAI-ARIA mapping
|
|
|
|
|
Object attributes: `details-roles:popover` |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| UIA |
|
|
|
|
|
|
|
Use WAI-ARIA mapping
|
|
|
|
|
|
|
|
|
|
|
| ATK |
|
|
|
|
|
|
|
Use WAI-ARIA mapping
|
|
|
|
|
Object attributes: `details-roles:popover` |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| AX |
|
|
|
|
|
|
|
Use WAI-ARIA mapping
|
|
|
|
|
|
|
|
|
|
|
| Comments |
|
|
|
|
|
|
|
|
|
User Agents MUST expose an `aria-details` relation with the associated popover element |
|
|
except under either of the following conditions: |
|
scottaohara marked this conversation as resolved.
Show resolved
Hide resolved
|
|
|
|
|
|
|
|
|
|
|
|
|
scottaohara marked this conversation as resolved.
Show resolved
Hide resolved
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
||||||||
`popovertargetaction`
|
|
|
|
|
| HTML Specification |
|
|
|
|
|
|
`popovertargetaction` |
|
|
|
|
|
|
|
|
|
|
|
| Element(s) |
|
|
|
|
|
|
`button`; |
|
|
`input type=button, image, reset, submit` |
|
|
|
|
|
|
|
|
|
|
|
| [[WAI-ARIA-1.2]] |
|
|
|
|
|
|
|
|
|
Not mapped |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| MSAA +
|
|
|
IAccessible2 |
|
|
|
|
|
|
|
Not mapped
|
|
|
|
|
|
|
|
|
|
|
| UIA |
|
|
|
|
|
|
|
Not mapped
|
|
|
|
|
|
|
|
|
|
|
| ATK |
|
|
|
|
|
|
|
Not mapped
|
|
|
|
|
|
|
|
|
|
|
| AX |
|
|
|
|
|
|
|
Not mapped
|
|
|
|
|
|
|
|
|
|
|
| Comments |
|
|
|
|
|
|
The value of this attribute can impact the mappings of its related `popovertarget` attribute. |
|
|
|
|
|
|
|
|
| |
||
`poster`
|
| |
Uh oh!
There was an error while loading. Please reload this page.