Ticket #4522 (closed Bug: fixed)
[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) - OLD |
| Keywords: | IE Review+ | Cc: |
Description
Environments
IE6/IE7 (IE8 works well)
Procedures
- Focus the editor;
- Open the 'Smiley' dialog and insert an icon;
- Start typing a few words;
- Press 'Undo' button or 'Ctrl-Z' key;
- Actual Result: Typed words are removed;
- 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
Change History
comment:1 Changed 4 years ago by garry.yao
- Keywords Review? added
- Status changed from new to assigned
- Owner set to garry.yao
comment:2 Changed 3 years ago by fredck
- Keywords Review- added; Review? removed
I don't see any difference after the patch.
comment:3 Changed 3 years 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.
comment:4 Changed 3 years 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).
Note: See
TracTickets for help on using
tickets.
