#14079 closed defect (fixed)
aria-pressed should not be set on a checkbox
Reported by: | Katie Vance | Owned by: | Douglas Hays |
---|---|---|---|
Priority: | high | Milestone: | 1.7.1 |
Component: | Accessibility | Version: | 1.6.1 |
Keywords: | Cc: | Becky Gibson | |
Blocked By: | Blocking: |
Description
The ARIA spec requires that aria-pressed should not be set on a checkbox.
Attachments (1)
Change History (11)
Changed 9 years ago by
Attachment: | 14079.patch added |
---|
comment:2 Changed 9 years ago by
Katie, ToggleButton? sets aria-pressed to false. Is removing the attribute more correct? Why add _setCheckedAttr method to CheckBox? when ToggleButton? already has that method and fiddles with aria-pressed?
comment:4 Changed 9 years ago by
Milestone: | tbd → 1.7.1 |
---|---|
Owner: | changed from Katie Vance to Douglas Hays |
Status: | new → assigned |
Summary: | aris-pressed should not be set on a checkbox → aria-pressed should not be set on a checkbox |
comment:6 Changed 9 years ago by
becky thinks the original removal for checkboxes may be more correct since checkboxes are native input type=checked and thus would be handled correctly by the browser w/o any aria info
comment:9 Changed 9 years ago by
Shouldn't there be a change to StackController.js too? It currently has:
_setCheckedAttr: function(/*Boolean*/ value, /*Boolean?*/ priorityChange){ this.inherited(arguments); this.focusNode.removeAttribute("aria-pressed"); },
Note: See
TracTickets for help on using
tickets.
Remove aria-pressed from checkboxes