Opened 14 years ago
Closed 14 years ago
#6375 closed Bug (fixed)
[IE] Exception thrown when hiding editor in some cases
Reported by: | Damian | Owned by: | Frederico Caldeira Knabben |
---|---|---|---|
Priority: | Normal | Milestone: | CKEditor 3.4.2 |
Component: | General | Version: | 3.4 |
Keywords: | IBM HasPatch | Cc: |
Description
When the editor is hidden right after a blur event, e.g. contained in a div with display="none", the call to selection.empty() throws an error. This code is only invoked in IE (< 8)
A work around for this issue is to catch the exception and ignore it, this is provided in attached patch.
Test case to reproduce the problem:
- Open modified Ajax sample attached
- Create editor and ensure focus is inside it
- Hit Hide
- Hit Show
- Put cursor into content area
- Repeat steps 3-5 until exception is thrown
Attachments (3)
Change History (7)
Changed 14 years ago by
Attachment: | 6375.patch added |
---|
Changed 14 years ago by
Modified ajax sample to help reproduce the problem
comment:1 Changed 14 years ago by
Keywords: | HasPatch added |
---|---|
Milestone: | → CKEditor 3.4.2 |
Owner: | set to Frederico Caldeira Knabben |
Status: | new → assigned |
The patch looks good. It's just a matter of fixing the coding style and adding a comment pointing to this ticket to justify the fix.
Changed 14 years ago by
Attachment: | 6375_2.patch added |
---|
comment:2 Changed 14 years ago by
Status: | assigned → review |
---|
btw, please always create patch files from the editor root folder, so the full file path is included on it.
comment:3 Changed 14 years ago by
Status: | review → review_passed |
---|
comment:4 Changed 14 years ago by
Resolution: | → fixed |
---|---|
Status: | review_passed → closed |
Fixed with [5918].
Proposed fix