Opened 6 years ago

Closed 6 years ago

Last modified 6 years ago

#6554 closed Bug (fixed)

Webkit: cannot type after inserting Page Break

Reported by: tobiasz.cudnik Owned by: mani
Priority: Normal Milestone: CKEditor 3.4.3
Component: General Version: 3.3.2
Keywords: Webkit IBM Cc: satya_minnekanti@…

Description

Cannot type after inserting Page Break using Chrome.

  1. Start the editor win en empty document.
  2. Type "foo".
  3. Press the Page Break button.
  4. Type "foo" again.

Expected: There are 2 "foo" paragraphs with the pagebreak in between.

Result: The second paragraph is missing.

Regression in 3.3.2 (worked well in 3.3.1).

Attachments (2)

6554.patch (484 bytes) - added by mani 6 years ago.
6554_2.patch (1.4 KB) - added by garry.yao 6 years ago.

Download all attachments as: .zip

Change History (13)

comment:1 Changed 6 years ago by garry.yao

  • Milestone set to CKEditor 3.4.2
  • Status changed from new to confirmed

comment:2 Changed 6 years ago by garry.yao

  • Version set to 3.3.2

comment:3 Changed 6 years ago by wwalc

  • Milestone changed from CKEditor 3.4.2 to CKEditor 3.4.3

comment:4 Changed 6 years ago by mani

  • Owner set to mani
  • Status changed from confirmed to assigned

Changed 6 years ago by mani

comment:5 Changed 6 years ago by mani

  • Status changed from assigned to review

comment:6 Changed 6 years ago by garry.yao

  • Status changed from review to review_failed

The patch breaks the following case:

  1. Insert a table at the end of document;
  2. Continue to click the spaces above table but below the last paragraph;
    • Actual Result: empty spaces are added on each click.

Changed 6 years ago by garry.yao

comment:7 Changed 6 years ago by mani

  • Status changed from review_failed to review

comment:8 Changed 6 years ago by tobiasz.cudnik

  • Status changed from review to review_passed

R+ for the 6554_2 patch.

comment:9 Changed 6 years ago by mani

  • Resolution set to fixed
  • Status changed from review_passed to closed

comment:10 Changed 6 years ago by mani

fixed with revision [6110]

comment:11 Changed 6 years ago by satya

  • Cc satya_minnekanti@… added
  • Keywords IBM added
Note: See TracTickets for help on using tickets.
© 2003 – 2016 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy