Opened 10 years ago

Closed 10 years ago

Last modified 10 years ago

#4567 closed Bug (fixed)

IE8 In source view pres backspace throw error on page

Reported by: lisaj Owned by: Garry Yao
Priority: Normal Milestone: CKEditor 3.1
Component: General Version:
Keywords: Confirmed IE Review+ Cc:

Description

After update to ckeditor 3.0.1 when i delete text in source view in IE error on page is throw but editor work. It is see on demo page

Attachments (2)

4567.patch (529 bytes) - added by Garry Yao 10 years ago.
4567_2.patch (530 bytes) - added by Garry Yao 10 years ago.

Download all attachments as: .zip

Change History (13)

comment:1 Changed 10 years ago by Garry Yao

Keywords: Pending added

Could you please provide more details on it?

comment:2 Changed 10 years ago by lisaj

  1. open ckeditor demo page
  2. switch to source editor
  3. delete character with backspace
  4. Error on the page in ckeditor.js at line 71 char 2262 r.getSelection() is null

Tested in IE8

comment:3 Changed 10 years ago by Wiktor Walc

Keywords: Confirmed IE added; Pending removed

Confirmed in IE6, IE7 and IE8.

comment:4 Changed 10 years ago by Frederico Caldeira Knabben

Milestone: CKEditor 3.1

Regression.

Changed 10 years ago by Garry Yao

Attachment: 4567.patch added

comment:5 Changed 10 years ago by Garry Yao

Keywords: Review? added
Owner: set to Garry Yao
Status: newassigned

comment:6 Changed 10 years ago by Frederico Caldeira Knabben

Keywords: Review- added; Review? removed

Theoretically, the wysiwyg plugin knows nothing about the "source" mode. Actually, we may even have unknown modes. So, checking for "source" there is wrong. It must instead check if it's own "wysiwyg" mode.

Changed 10 years ago by Garry Yao

Attachment: 4567_2.patch added

comment:7 Changed 10 years ago by Garry Yao

Keywords: Review? added; Review- removed

comment:8 Changed 10 years ago by Frederico Caldeira Knabben

Keywords: Review+ added; Review? removed

comment:9 Changed 10 years ago by Garry Yao

Resolution: fixed
Status: assignedclosed

Fixed with [4445].

comment:10 Changed 10 years ago by Frederico Caldeira Knabben

#4706 has been marked as DUP.

comment:11 Changed 10 years ago by Frederico Caldeira Knabben

#4766 has been marked as DUP.

Note: See TracTickets for help on using tickets.
© 2003 – 2019 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy