#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 |
I