Opened 12 years ago

Closed 12 years ago

#3595 closed defect (fixed)

fix drop down button a11y broken by revision #9350

Reported by: Becky Gibson Owned by: ptbrunet
Priority: high Milestone: 0.9
Component: Accessibility Version: 0.9
Keywords: Cc: Becky Gibson
Blocked By: Blocking:

Description

The dropdown button template was changed to accommodate css images for button - the changes break the a11y of the button. Previously the labelledby attribute was used to make the screen reader speak the correct label for the button and NOT the html entity. The rework on the template breaks this.

The waiRole=button, haspopup and labelledby wai attributes can probably be moved to the actual button element. The button element can probably be the focusNode - will need investigation and testing.

Change History (3)

comment:1 Changed 12 years ago by bill

One thing that threw me here is that for MenuItem? the focus is on just the text part. Not sure if that's better/worse/the same as focusing on the text+the icon+the arrow. Probably just an aesthetic issue, but anyway would be nice to be consistent.

comment:2 Changed 12 years ago by bill

(In [9635]) Patch from Becky: Consolidate escape handling to close popup in popup.js. Make necessary changes to buttons and menu. Need to do additional work for combox and select so not closing the issue. refs #3544. Required some changes to dropdownButton and combobutton that also refs #3595.

comment:3 Changed 12 years ago by Becky Gibson

Resolution: fixed
Status: newclosed

this was fixed as part of #3544

Note: See TracTickets for help on using tickets.