Opened 16 years ago
Closed 16 years ago
#4354 closed Bug (wontfix)
Using TAB key on toolbar focus disabled buttons
| Reported by: | Tobiasz Cudnik | Owned by: | Tobiasz Cudnik |
|---|---|---|---|
| Priority: | Normal | Milestone: | CKEditor 3.1 |
| Component: | Accessibility | Version: | |
| Keywords: | Confirmed | Cc: |
Description
Using TAB key (or SHIFT+TAB) on toolbar focus disabled buttons, while they shouldn't be focusable.
To reproduce:
- Open editor sample
- Switch to source area
- Focus editing area
- Press ALT+F10
- Press TAB
Result: Save button have focus (disabled)
Expected: New Page button should have focus (first non-disabled)
Attachments (1)
Change History (7)
comment:1 Changed 16 years ago by
| Owner: | set to Tobiasz Cudnik |
|---|---|
| Status: | new → assigned |
Changed 16 years ago by
| Attachment: | 4354.patch added |
|---|
comment:2 Changed 16 years ago by
| Keywords: | Review? added |
|---|
comment:3 Changed 16 years ago by
| Keywords: | Review+ added; Review? removed |
|---|
It's good to have TC covering this kind of accessibility feature, I'm trying to add one while it seems hard, let's compensate it later after the testing API is more stable.
comment:4 Changed 16 years ago by
| Keywords: | NeedsTest added |
|---|---|
| Resolution: | → fixed |
| Status: | assigned → closed |
Fixed with [4171].
comment:5 Changed 16 years ago by
| Keywords: | Review+ NeedsTest removed |
|---|---|
| Resolution: | fixed |
| Status: | closed → reopened |
After discussing with Fred, it's incorrect to jump over the disabled item, the item should be announced properly for better accessibility. Revert it with [4186].
comment:6 Changed 16 years ago by
| Resolution: | → wontfix |
|---|---|
| Status: | reopened → closed |
Note: See
TracTickets for help on using
tickets.
I've added some additional comments to while loops as they were quite not clear to me at first sight.