Closed Bug 1376108 Opened 8 years ago Closed 8 years ago

The separator role should have no AXSubrole and should have AXRoleDescription "splitter"

Categories

(Core :: Disability Access APIs, enhancement)

Product:
Core
Component:
Disability Access APIs
Platform:
Unspecified
macOS
Type:
enhancement
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla56
Tracking Flags:
Tracking Status
firefox56 --- fixed

People

(Reporter: jdiggs, Assigned: jdiggs)

Details

Attachments

(1 file)

proposed patch
1.02 KB, patch
MarcoZ
: review+
Details | Diff | Splinter Review
According to the Core AAM [1], the separator role has the following mapping for AX API: AXRole: AXSplitter AXSubrole: <nil> AXRoleDescription: 'splitter' While the AXRole value is correct, the exposed subrole is AXContentSeparator with an AXRoleDescription of "separator". [1] https://rawgit.com/w3c/aria/master/core-aam/core-aam.html#role-map-separator
Attached patch proposed patch — — Splinter Review
Marco, this is another real simple fix (just delete two lines of code). Please review. Thanks!
Assignee: nobody → jdiggs
Status: NEW → ASSIGNED
Attachment #8881053 - Flags: review?(mzehe)
Attachment #8881053 - Flags: review?(mzehe) → review+
Keywords: checkin-needed
Pushed by cbook@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/132471a25696 Fix macOS mapping for ARIA separator role. r=marcoz
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/132471a25696
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
status-firefox56: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla56
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: