Closed Bug 1893707 Opened 2 years ago Closed 2 months ago

ARIA: remove aria-expanded from supported attributes from role listbox

Categories

(Core :: Disability Access APIs, enhancement, P5)

Firefox 127
enhancement

Tracking

()

RESOLVED FIXED
151 Branch
Tracking Status
firefox151 --- fixed

People

(Reporter: spectranaut, Assigned: b3nbinoh)

References

(Depends on 1 open bug, Blocks 1 open bug)

Details

(Keywords: good-first-bug)

Attachments

(2 files, 2 obsolete files)

User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:109.0) Gecko/20100101 Firefox/115.0

Expected results:

ARIA is removing aria-expanded from listbox role's supported attribute list: https://github.com/w3c/aria/pull/1862

Rule 5 in CORE-AAM's State and Property Mapping General Rules (https://w3c.github.io/core-aam/#statePropertyMappingGeneralRules) says:

Some WAI-ARIA properties are not global, and are only supported on certain roles. If a non-global WAI-ARIA state or property is used where it is not supported, user agents SHOULD NOT map the given WAI-ARIA property to the platform accessibility API. For example, if aria-checked="true" is specified on <div role="grid">, it should not be exposed in MSAA implementations as STATE_SYSTEM_CHECKED.

The Bugbug bot thinks this bug should belong to the 'Core::Disability Access APIs' component, and is moving the bug to that component. Please correct in case you think the bot is wrong.

Component: Untriaged → Disability Access APIs
Product: Firefox → Core
Blocks: aria
Status: UNCONFIRMED → NEW
Type: defect → enhancement
Depends on: 1726354
Ever confirmed: true
OS: Unspecified → All
Hardware: Unspecified → All

Valerie mentioned these are likely good first bugs, so I'm marking them as such. Please feel free to correct if that's not accurate.

Keywords: good-first-bug
Severity: -- → S4
Priority: -- → P5

Hey I'm a new contributor, I'm gonna take a look at this. ☺

Attachment #9549529 - Attachment description: WIP: Bug 1893707 - ARIA: remove aria-expanded from supported attributes from role listbox → WIP: Bug 1893707 - ARIA: remove aria-expanded from supported attributes from role listbox. Addded back accidentaly deleted file, and deleted the correct other file.
Attachment #9549529 - Attachment is obsolete: true
Assignee: nobody → b3nbinoh
Status: NEW → ASSIGNED

Hey sorry I tried to change my WIP Phab page to a non-WIP one but it looks like it just created a new Phab page.

Attachment #9549511 - Attachment is obsolete: true
Attachment #9557751 - Attachment description: WIP: Bug 1893707: Introduce perftest to track cost of adding ComputedRole call in GetARIAState r?Jamie → Bug 1893707: Introduce perftest to track cost of adding ComputedRole call in GetARIAState r?Jamie
Pushed by jteh@mozilla.com: https://github.com/mozilla-firefox/firefox/commit/23974e2d947e https://hg.mozilla.org/integration/autoland/rev/facf11f6cdea ARIA: remove aria-expanded from supported attributes from role listbox r=eeejay
Status: ASSIGNED → RESOLVED
Closed: 2 months ago
Resolution: --- → FIXED
Target Milestone: --- → 151 Branch
Blocks: 2031407
No longer blocks: 2031407
QA Whiteboard: [qa-triage-done-c152/b151]
Pushed by mreschenberg@mozilla.com: https://github.com/mozilla-firefox/firefox/commit/cfd02a235a96 https://hg.mozilla.org/integration/autoland/rev/36332c45f194 Introduce perftest to track cost of adding ComputedRole call in GetARIAState r=Jamie,mozperftest-reviewers,sparky,perftest-reviewers
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: