Closed Bug 1376111 Opened 7 years ago Closed 7 years ago

AXRoleDescription for ARIA heading role should be "heading" not "AXHeading"

Categories

(Core :: Disability Access APIs, enhancement)

Unspecified
macOS
enhancement
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla56
Tracking Status
firefox56 --- fixed

People

(Reporter: jdiggs, Assigned: jdiggs)

Details

Attachments

(1 file, 1 obsolete file)

As per the Core AAM [1], the value of AXRoleDescription for the heading role should be "heading". But Gecko is exposing "AXHeading".

[1] https://rawgit.com/w3c/aria/master/core-aam/core-aam.html#role-map-heading
Attached patch proposed patch (obsolete) — Splinter Review
Assignee: nobody → jdiggs
Status: NEW → ASSIGNED
Attachment #8881060 - Flags: review?(mzehe)
Comment on attachment 8881060 [details] [diff] [review]
proposed patch

Is there a list somewhere what the localized strings should be for this, or are they free form? If there is a source, please add a comment above the localized string to indicate this. Thanks! r=me.
Attachment #8881060 - Flags: review?(mzehe) → review+
(In reply to Marco Zehe (:MarcoZ) from comment #2)
> Comment on attachment 8881060 [details] [diff] [review]
> proposed patch
> 
> Is there a list somewhere what the localized strings should be for this, or
> are they free form? If there is a source, please add a comment above the
> localized string to indicate this. Thanks! r=me.

Done. Thanks!
Attachment #8881060 - Attachment is obsolete: true
Keywords: checkin-needed
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/a7effdfd856a
Fix AXRoleDescription for ARIA heading role. r=MarcoZ
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/a7effdfd856a
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla56
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: