#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 8 years ago by
| Attachment: | 14079.patch added |
|---|
comment:2 Changed 8 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 8 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 8 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 8 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