Ticket #4522 (closed Bug: fixed)

Opened 5 months ago

Last modified 3 months ago

[IE] Unable to redo on inserted element + typing

Reported by: garry.yao Owned by: garry.yao
Priority: Normal Milestone: CKEditor 3.1
Component: Core : Undo/Redo Version: SVN (CKEditor)
Keywords: IE Review+ Cc:

Description

Environments

IE6/IE7 (IE8 works well)

Procedures

  1. Focus the editor;
  2. Open the 'Smiley' dialog and insert an icon;
  3. Start typing a few words;
  4. Press 'Undo' button or 'Ctrl-Z' key;
  • Actual Result: Typed words are removed;
  1. Press 'Undo' button or 'Ctrl-Z' key again;
  • Expected Result: Icon is removed.
  • Actual Result: Unable to remove the icon, even 'Undo' button is enable there.

Attachments

4522.patch Download (1.0 KB) - added by garry.yao 5 months ago.
4522_2.patch Download (0.9 KB) - added by garry.yao 4 months ago.
4522_3.patch Download (1.1 KB) - added by garry.yao 4 months ago.

Change History

Changed 5 months ago by garry.yao

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

I don't see any difference after the patch.

Changed 4 months ago by garry.yao

Changed 4 months ago by garry.yao

  • keywords Review? added; Review- removed
  • version set to SVN (CKEditor)

@fredck You're using IE8 quirks mode for testing and the above patch has wrong detection.

It's even safer to target the fix to all browsers.

Changed 4 months ago by fredck

  • keywords Review- added; Review? removed

We should not make this fix viral, impacting on the performance of good browsers. Let's limit this fix to IE only, using CKEDITOR.env.ie8Compat to avoid it into IE8 also (as long as it's really true that IE8 is ok with it).

Changed 4 months ago by garry.yao

Changed 4 months ago by garry.yao

  • keywords Review? added; Review- removed

Changed 3 months ago by fredck

  • keywords Review+ added; Review? removed

Changed 3 months ago by garry.yao

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

Fixed with [4605].

Note: See TracTickets for help on using tickets.