Bug 181796 - AX: Implement support for Graphics ARIA roles
Summary: AX: Implement support for Graphics ARIA roles
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Accessibility (show other bugs)
Version: WebKit Nightly Build
Hardware: All All
: P2 Normal
Assignee: Joanmarie Diggs
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2018-01-18 05:43 PST by Joanmarie Diggs
Modified: 2018-01-22 11:16 PST (History)
9 users (show)

See Also:


Attachments
Patch (13.92 KB, patch)
2018-01-22 07:21 PST, Joanmarie Diggs
no flags Details | Formatted Diff | Diff
Patch (16.88 KB, patch)
2018-01-22 09:52 PST, Joanmarie Diggs
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Joanmarie Diggs 2018-01-18 05:43:34 PST
There is a graphics-aria spec [1] which introduces three roles:
* graphics-symbol
* graphics-document
* graphics-object

The platform mappings can be found in the graphics-aam spec [2].

Related: At the time we implemented the mappings for SVG defined in the svg-aam [3], I'm pretty sure the svg-aam itself had the platform details; now it says things like "Default Platform WAI-ARIA Role Mappings: graphics-symbol role ..." We might wish to make a similar adjustment in our support, and we should double-check nothing else has changed in the expected mappings.

[1] https://rawgit.com/w3c/graphics-aria/master/
[2] https://rawgit.com/w3c/graphics-aam/master/
[3] https://rawgit.com/w3c/svg-aam/master/
Comment 1 Radar WebKit Bug Importer 2018-01-18 05:43:45 PST
<rdar://problem/36617946>
Comment 2 Joanmarie Diggs 2018-01-19 08:27:26 PST
(In reply to Joanmarie Diggs (irc: joanie) from comment #0)
 
> Related: At the time we implemented the mappings for SVG defined in the
> svg-aam [3], I'm pretty sure the svg-aam itself had the platform details;
> now it says things like "Default Platform WAI-ARIA Role Mappings:
> graphics-symbol role ..." We might wish to make a similar adjustment in our
> support, and we should double-check nothing else has changed in the expected
> mappings.

We're blocked until we get some answers from the SVG AAM folks. So I opened bug 181856 for that work.
Comment 3 Joanmarie Diggs 2018-01-22 07:21:23 PST
Created attachment 331922 [details]
Patch
Comment 4 chris fleizach 2018-01-22 08:12:16 PST
Comment on attachment 331922 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=331922&action=review

> Source/WebCore/accessibility/AccessibilityObject.cpp:2310
> +        { "graphics-document", AccessibilityRole::Document },

You don’t think we need a separate role for this kind of document?
Comment 5 Joanmarie Diggs 2018-01-22 09:52:08 PST
Created attachment 331938 [details]
Patch
Comment 6 Joanmarie Diggs 2018-01-22 10:20:47 PST
(In reply to chris fleizach from comment #4)
> Comment on attachment 331922 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=331922&action=review
> 
> > Source/WebCore/accessibility/AccessibilityObject.cpp:2310
> > +        { "graphics-document", AccessibilityRole::Document },
> 
> You don’t think we need a separate role for this kind of document?

At the time I did the patch, no. But given your question, along with what was stated today in https://github.com/w3c/svg-aam/issues/1#issuecomment-359493725, it seems we may have a use case for a separate/dedicated role for graphics document as well as for graphics symbol. The patch in comment #5 does that.
Comment 7 WebKit Commit Bot 2018-01-22 11:16:57 PST
Comment on attachment 331938 [details]
Patch

Clearing flags on attachment: 331938

Committed r227344: <https://trac.webkit.org/changeset/227344>
Comment 8 WebKit Commit Bot 2018-01-22 11:16:59 PST
All reviewed patches have been landed.  Closing bug.