Opened 13 years ago
Closed 13 years ago
#8706 closed Bug (fixed)
JAWS not switching modes in CKEditor Cell dialog
Reported by: | Satya Minnekanti | Owned by: | Garry Yao |
---|---|---|---|
Priority: | Normal | Milestone: | CKEditor 3.6.3 |
Component: | Accessibility | Version: | 3.0 |
Keywords: | IBM Firefox | Cc: | Damian, Teresa Monahan |
Description
To reproduce the defect:
- Open a CKEditor sample.
- Insert a new table.
- Open the Cell properties dialog using the context menu
- Navigate to a Choose button in the dialog and press Enter to open the Select Color dialog.
- Select Color dialog opens and JAWS reads the first color option. Inspect32 confirms that focus is on the first color in colors listbox.
- Use Right Arrow key.
Expected Result: JAWS should read the next color in the list. Actual Result: JAWS reading the Cancel button name.
There are two issues here
First Issue: JAWS doesn't switch to forms mode correctly to allow navigating the color picker list box with arrow keys.
Second Issue: Odd jump to the Cancel button bypassing the input field on the dialog.
Attachments (5)
Change History (23)
comment:1 Changed 13 years ago by
Status: | new → confirmed |
---|---|
Version: | → 3.0 |
comment:2 Changed 13 years ago by
comment:3 Changed 13 years ago by
Response from FS:
The color picker control is currently implemented as a table with role="listbox". Under that table is a tbody element containing tr elements with each tr containing td elements. Finally, each td contains a link with role="option". There appear to be several issues with this control.
- According to the authoring guide http://www.w3.org/TR/wai-aria-practices/
in section 10, a list box is supposed to have list items as its children with no other elements in between.
- Probably as a result of point 1 above, Firefox does not apply the focusable
state to the table element with role="listbox" and JAWS does not identify this as a listbox control.
- Probably most importantly, this control acts like a grid. That is
left/right arrows move the user left and right along rows and up/down arrows move the user from row to row.
How to fix. There are a couple possibilities on the best way to fix this.
- Make the color picker a grid control with gridcells for the individual
table cells.
- modify the listbox so the direct descendants are list items.
Either of these solutions should let the control work correctly with JAWS
comment:4 Changed 13 years ago by
Status: | confirmed → pending |
---|
This issue WFM with JAWS13 in both IE and FF, can someone confirm it?
comment:5 Changed 13 years ago by
Keywords: | Firefox added |
---|---|
Status: | pending → confirmed |
I was bale to reproduce it in Firefox 11, Chrome 17 on Windows XP and JAWS 13.0.718
Strange that it was not found when testing #8701
Changed 13 years ago by
Attachment: | 8706.patch added |
---|
comment:6 Changed 13 years ago by
Owner: | set to Garry Yao |
---|---|
Status: | confirmed → review |
I did some further testing, it's a random reproduced issue (for the key navigation part) while it's true that "listbox" is not well recognized here, the idea of making it as a "grid" looks like a better choice here, we had the "listbox" option before since at that time the navigation is one way only, later it was later changed to two dimension.
In testing JAWS doesn't work well for marking the "link" as the grid cell, so the impl has to drop the focus directly on a table cell for that, this should be enough to make JAWS speaks:
"Select Color Dialog, Color Options dialog, #660066..."
Changed 13 years ago by
Changed 13 years ago by
Attachment: | 8706_2.patch added |
---|
comment:8 Changed 13 years ago by
New patch addresses the following found issues:
- JAWS on Firefox doesn't speaks out cell color (missing aria-label);
- JAWS doesn't always announce table name (caption helps);
- Restore last focused color cell on revisit;
- Distinguish outline (focus) and hover states;
- Fix focus/blur event on old IEs.
- Add "aria-selected" to color cell when choose.
comment:9 Changed 13 years ago by
Milestone: | → CKEditor 3.6.3 |
---|
comment:10 Changed 13 years ago by
Component: | General → Accessibility |
---|---|
Status: | review → review_failed |
- There should be no cell highlighted when the dialog opens. That looks strange for non-keyboard users. Cells start to get highlighted as soon as the arrows get activated, having the first cell highlighted on first key-press (similar to toolbar combos).
- There is a bit of confusion if you start moving the mouse over the cells, after having done it with keyboard. You see two highlight boxes, one for the keyboard and one for the mouse. We should not have it for the mouse, which should not interfere on the keyboard highlight, unless a mouse click takes place.
- It is not possible to select a color with mouse click.
Changed 13 years ago by
Attachment: | 8706_3.patch added |
---|
comment:11 Changed 13 years ago by
Status: | review_failed → review |
---|
comment:12 Changed 13 years ago by
Status: | review → review_passed |
---|
comment:13 Changed 13 years ago by
Resolution: | → fixed |
---|---|
Status: | review_passed → closed |
Fixed with [7429].
comment:14 Changed 13 years ago by
It's not properly fixed. Jaws just reading as Color Options table when we open Select Color dialog, & when user uses Arrow keys nothing happens,JAWS just reads as Blank or space. But after using arrow keys if you press Tab Focus going to a color in the table and user can use arrow keys after it.
Expected Result: when Select Color dialog opens focus should go to first color option in the table and JAWS should read that option, and user should be able to use arrow keys to navigate between different Colors.
Please re-open this ticket.
comment:15 Changed 13 years ago by
Resolution: | fixed |
---|---|
Status: | closed → reopened |
Changed 13 years ago by
Attachment: | 8706_4.patch added |
---|
comment:16 Changed 13 years ago by
Status: | reopened → review |
---|
comment:17 Changed 13 years ago by
Status: | review → review_passed |
---|
comment:18 Changed 13 years ago by
Resolution: | → fixed |
---|---|
Status: | review_passed → closed |
Fixed with [7444].
Colors haven't been read from CKEditor 3.0.
Focus on first color was introduced in CKEditro 3.3.