Opened 11 years ago
Closed 11 years ago
#13751 closed defect (fixed)
StackContainer: No indication of which page you are on
Reported by: | Katie Vance | Owned by: | Katie Vance |
---|---|---|---|
Priority: | high | Milestone: | 1.7 |
Component: | Accessibility | Version: | 1.6.1 |
Keywords: | a11y | Cc: | Becky Gibson |
Blocked By: | Blocking: |
Description
These problems need to be addressed:
- The css class dijitStackController is no longer used and has be replaced by dijitTabController. This was changed by trac changeset: [25481]. This is why we don't see any changes to the toggle button when it's toggled. I will update the corresponding css files to use this.
- Remove aria-pressed.
- There is a bug with using a toggle button. When the user toggles a button on, then it shows as checked. However clicking the button again will set it to an unchecked state. This is incorrect because that button's corresponding pane is still shown. It's state should only be unchecked whenever it's pane is not shown.
- Jaws doesn't read the buttons right. It always says button 1 of 1. Even if there are 3 buttons.
Attachments (1)
Change History (6)
comment:1 Changed 11 years ago by
Changed 11 years ago by
Attachment: | 13751.patch added |
---|
comment:2 Changed 11 years ago by
Ok, I refreshed the patch to undo the change that was made to switch the css class from dijitStackController to dijitTabController. I still had to make one change to claro Button.css because it never had a class to handle the toggleButtonChecked state for StackContainer.
To address your other issues:
- You can't return if the "if(this._currentChild.id === page.id)" is true because by the time we get to this method the value has already been changed by the ToggleButton parent code.
- The aria-selected setting already exists, so no need to add it.
- It does fix issue #4, by adding presentation roles to Button.html.
- I added a test case.
comment:3 Changed 11 years ago by
Milestone: | tbd → 1.7 |
---|
OK, looks good.
FYI, TabButton extends dijit.form.Button rather than dijit.form.ToggleButton. That might be something to think about for the StackController buttons for the future. Not sure if it makes things easier or harder. But anyway I'll check this in, thanks for the patch.
comment:4 Changed 11 years ago by
Summary: | dijit.layout.StackContainer - No indication of which page you are on → StackContainer: No indication of which page you are on |
---|
Oops, looks like I removed dijitStackController by mistake (as a typo), can you change it back to dojoStackController, and modify your patch?
Also:
if(this._currentChild.id === page.id)
" is true can't you just return?#4
? I don't see any changes that would fix that.