Bug 156042 - AX: Update handling of SVG elements with presentational role
Summary: AX: Update handling of SVG elements with presentational role
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: 2016-03-30 15:17 PDT by Joanmarie Diggs
Modified: 2016-03-30 18:31 PDT (History)
9 users (show)

See Also:

Attachments
Patch (10.23 KB, patch)
2016-03-30 15:35 PDT, Joanmarie Diggs 		no flags Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Joanmarie Diggs 2016-03-30 15:17:28 PDT 
The SVG Accessibility Task Force had indicated that in the case of SVG, any element with a presentational role should be excluded from the tree -- even if it had global ARIA attributes. See [1]. This behavior was part of the implementation I did [2]. After some subsequent discussion, the Task Force consensus is that this "golden hammer" behavior is not desired and that global ARIA attributes should trump the presentational role. [3]

Patch shall be forthcoming.

[1] https://github.com/w3c/aria/issues/136#issuecomment-170557956
[2] https://trac.webkit.org/browser/trunk/Source/WebCore/accessibility/AccessibilitySVGElement.cpp?rev=197616#L249
[3] https://lists.w3.org/Archives/Public/public-svg-a11y/2016Mar/0060.html
Comment 1 Radar WebKit Bug Importer 2016-03-30 15:18:02 PDT 
<rdar://problem/25451392>
Comment 2 Joanmarie Diggs 2016-03-30 15:35:12 PDT 
Created attachment 275229 [details]
Patch
Comment 3 Joanmarie Diggs 2016-03-30 16:28:08 PDT 
Comment on attachment 275229 [details]
Patch

Chris: When you have a chance, review please. (This should be an easy one.)
Comment 4 WebKit Commit Bot 2016-03-30 18:31:49 PDT 
Comment on attachment 275229 [details]
Patch

Clearing flags on attachment: 275229

Committed r198870: <http://trac.webkit.org/changeset/198870>
Comment 5 WebKit Commit Bot 2016-03-30 18:31:53 PDT 
All reviewed patches have been landed.  Closing bug.