Opened 11 years ago

Closed 11 years ago

#12324 closed Bug (fixed)

[IE8] Undo steps not recorded when clicking below the text

Reported by: Olek Nowodziński Owned by: Piotrek Koszuliński
Priority: Normal Milestone: CKEditor 4.4.5
Component: Core : Undo & Redo Version: 4.4.4
Keywords: Cc:

Description (last modified by Piotrek Koszuliński)

Occurs when body doesn't fill entire html element.

  1. Open languages samples (because there's just one line of text).
  2. Click below the text and type.
  3. Click somewhere else below the text and type again.
  4. Undo.
  • Expected: only text typed in 3. should be undid.
  • Actual: all typed text has been undone.

Attachments (1)

ie8_undo_444_regr.mp4​ (161.9 KB) - added by Olek Nowodziński 11 years ago.

Download all attachments as: .zip

Change History (11)

Changed 11 years ago by Olek Nowodziński

Attachment: ie8_undo_444_regr.mp4​ added

comment:1 Changed 11 years ago by Olek Nowodziński

Summary: [IE8] Regression in undo manager → [IE8] Regression in undo manager (an undo step is lost)

comment:2 Changed 11 years ago by Marek Lewandowski

Status: new → confirmed

Reproduced with basic preset using keyboard, had no luck with full preset.

comment:3 Changed 11 years ago by Artur Delura

Owner: set to Artur Delura
Status: confirmed → assigned

comment:4 Changed 11 years ago by Artur Delura

I reproduced this on every browser full build (samples)

  1. Open sample with following content: <p>foo bar baz</p>
  2. Make selection using mouse: <p>[foo] bar baz</p>
  3. Press Backspace

Actual Result: <p>^ bar baz</p>

  1. Put caret here using mouse: <p> bar ^baz</p>
  2. Type "X".
  3. Put caret here using mouse: <p>^ bar Xbaz</p>
  4. Type "X".
  5. Press Ctrl + Z

Actual result: <p>bar ^baz</p>

Please confirm for now.

Version 0, edited 11 years ago by Artur Delura (next)

comment:5 Changed 11 years ago by Artur Delura

Status: assigned → review

Changes in ​branch:t/12324.

comment:6 Changed 11 years ago by Piotrek Koszuliński

Status: review → review_failed

2 tests failed in undo/pointer, because you haven't updated them.

comment:7 Changed 11 years ago by Piotrek Koszuliński

Owner: changed from Artur Delura to Piotrek Koszuliński
Status: review_failed → assigned

comment:8 Changed 11 years ago by Piotrek Koszuliński

Description: modified (diff)

comment:9 Changed 11 years ago by Piotrek Koszuliński

Summary: [IE8] Regression in undo manager (an undo step is lost) → [IE8] Undo steps not recorded when clicking below the text

comment:10 Changed 11 years ago by Piotrek Koszuliński

Resolution: → fixed
Status: assigned → closed

Fixed on master with ​git:9fe1013.

Note: See TracTickets for help on using tickets.
© 2003 – 2022, CKSource sp. z o.o. sp.k. All rights reserved. | Terms of use | Privacy policy