Ticket #4567 (closed Bug: fixed)

Opened 5 months ago

Last modified 3 months ago

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

4567.patch Download (0.5 KB) - added by garry.yao 4 months ago.
4567_2.patch Download (0.5 KB) - added by garry.yao 4 months ago.

Change History

Changed 5 months ago by garry.yao

  • keywords Pending added

Could you please provide more details on it?

Changed 5 months 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

Changed 4 months ago by wwalc

  • keywords Confirmed IE added; Pending removed

Confirmed in IE6, IE7 and IE8.

Changed 4 months ago by fredck

  • milestone set to CKEditor 3.1

Regression.

Changed 4 months ago by garry.yao

Changed 4 months ago by garry.yao

  • keywords Review? added
  • owner set to garry.yao
  • status changed from new to assigned

Changed 4 months ago by fredck

  • 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 4 months ago by garry.yao

Changed 4 months ago by garry.yao

  • keywords Review? added; Review- removed

Changed 4 months ago by fredck

  • keywords Review+ added; Review? removed

Changed 4 months ago by garry.yao

  • status changed from assigned to closed
  • resolution set to fixed

Fixed with [4445].

Changed 4 months ago by fredck

#4706 has been marked as DUP.

Changed 3 months ago by fredck

#4766 has been marked as DUP.

Note: See TracTickets for help on using tickets.