Opened 15 years ago
Last modified 14 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 15 years ago by
Milestone: | CKEditor 3.1 → CKEditor 3.2 |
---|
comment:2 Changed 15 years ago by
Milestone: | CKEditor 3.2 → CKEditor 3.3 |
---|
comment:3 Changed 15 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 14 years ago by
Milestone: | CKEditor 3.4 |
---|