#4716 closed Bug (fixed)
[IE7] Cursor remains in editor when you tab to following form button.
| Reported by: | Joe Kavanagh | Owned by: | Garry Yao |
|---|---|---|---|
| Priority: | Normal | Milestone: | CKEditor 3.3 |
| Component: | General | Version: | 3.0.1 |
| Keywords: | IBM Confirmed IE Review+ | Cc: | dchojna@…, philip_andrew@… |
Description (last modified by )
If the editor is the last component in a form and you tab out to apply focus to a button the cursor remains visible. If you enter text it will go into the editor at the cursor. If you press Enter the button, which has focus, will be pressed.
Insert an input field after the editor and before the button. Tab from the editor to the input field. The cursor is moved from the editor to the input field as normal.
Attachments (3)
Change History (19)
comment:1 Changed 9 years ago by
| Description: | modified (diff) |
|---|---|
| Milestone: | CKEditor 3.1 |
comment:2 Changed 9 years ago by
| Keywords: | Confirmed added |
|---|
Nice catch, have the same IE bug behind with #4188.
comment:3 Changed 9 years ago by
| Milestone: | → CKEditor 3.2 |
|---|
comment:4 Changed 9 years ago by
| Keywords: | IE added |
|---|---|
| Owner: | set to Garry Yao |
| Status: | new → assigned |
The problem only affects IE7.
Changed 9 years ago by
| Attachment: | 4716.patch added |
|---|
comment:5 Changed 9 years ago by
| Keywords: | Review? added |
|---|
comment:6 Changed 9 years ago by
| Keywords: | Review- added; Review? removed |
|---|
The patch is working fine for IE7, but I can reproduce the problem also in IE6 with the replacebyclass sample
Changed 9 years ago by
| Attachment: | 4716_2.patch added |
|---|
comment:7 Changed 9 years ago by
| Keywords: | Review? added; Review- removed |
|---|
Reproduced on IE6 standard also.
comment:8 Changed 9 years ago by
| Keywords: | Review+ added; Review? removed |
|---|
comment:10 Changed 9 years ago by
| Milestone: | CKEditor 3.2 → CKEditor 3.3 |
|---|---|
| Resolution: | fixed |
| Status: | closed → reopened |
comment:11 Changed 9 years ago by
| Keywords: | Review- added; Review+ removed |
|---|
Changed 9 years ago by
| Attachment: | 4716_3.patch added |
|---|
comment:12 Changed 9 years ago by
| Keywords: | Review? added; Review- removed |
|---|
comment:13 Changed 9 years ago by
| Keywords: | Review+ added; Review? removed |
|---|
comment:14 Changed 9 years ago by
| Keywords: | Review- added; Review+ removed |
|---|
What about putting the "if" before the listener, so the rest of the browsers ignore it?
comment:15 Changed 9 years ago by
| Keywords: | Review+ added; Review- removed |
|---|---|
| Resolution: | → fixed |
| Status: | reopened → closed |
Note: See
TracTickets for help on using
tickets.
I