Opened 16 years ago
Closed 16 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 16 years ago by
| Attachment: | 6375.patch added |
|---|
Changed 16 years ago by
Modified ajax sample to help reproduce the problem
comment:1 Changed 16 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 16 years ago by
| Attachment: | 6375_2.patch added |
|---|
comment:2 Changed 16 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 16 years ago by
| Status: | review → review_passed |
|---|
comment:4 Changed 16 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | review_passed → closed |
Fixed with [5918].
Note: See
TracTickets for help on using
tickets.
Proposed fix