Opened 16 years ago
Last modified 15 years ago
#4785 review_failed Bug
Incorrect tab key order for table properties dialog
| Reported by: | Garry Yao | Owned by: | brooks |
|---|---|---|---|
| Priority: | Normal | Milestone: | |
| Component: | UI : Dialogs | Version: | SVN (CKEditor) - OLD |
| Keywords: | Cc: |
Description
Reproducing Procedures
- Open 'replacebyclass' sample page insert an default table.
- Click inside one of the table cell and open context menu on it.
- Click 'Table Properties' to open the dialog.
- Press 'Tab' key to iterate over all dialog fields.
- Actual Result: Some fields will be skipped before reach the dialog buttons.
Attachments (2)
Change History (10)
comment:1 Changed 16 years ago by
| Milestone: | CKEditor 3.1 → CKEditor 3.2 |
|---|
comment:2 Changed 16 years ago by
| Milestone: | CKEditor 3.2 → CKEditor 3.3 |
|---|
comment:3 Changed 16 years ago by
| Milestone: | CKEditor 3.3 → CKEditor 3.4 |
|---|
Changed 15 years ago by
| Attachment: | 4785.patch added |
|---|
comment:4 Changed 15 years ago by
| Owner: | set to brooks |
|---|---|
| Status: | new → assigned |
comment:5 Changed 15 years ago by
| Keywords: | Review? added; Confirmed removed |
|---|
- the simplest way to fix this problem is not make a default focus, cos it disarranged the focus sequence. this is my chosen
- an optional way may be rearrange the dialog's elements layout sequence, e.g. switch the 'selHeaders' select with 'txtWidth' input( and other element switch);
- adjust the focusList by hand, I think this is not scalable .
Changed 15 years ago by
| Attachment: | 4785_2.patch added |
|---|
comment:7 Changed 15 years ago by
| Keywords: | Review- added; Review? removed |
|---|
I'm not sure this will be a good deal, as the smart focus is a friendly feature also exists in other places, e.g. Link dialog, where we have the same problem, we may even consider a wontfix for this one if no decent solution could be found.
comment:8 Changed 15 years ago by
| Milestone: | CKEditor 3.4 |
|---|
