Opened 6 years ago

Closed 5 years ago

#8247 closed Bug (fixed)

IE: throwing an error when we apply Alignment to entire page content containing Page break at the end

Reported by: satya Owned by: garry.yao
Priority: Normal Milestone: CKEditor 3.6.3
Component: General Version: 3.6
Keywords: IBM IE Cc: damo, tmonahan


To reproduce the defect:

  1. Open any sample type few paragraphs of text & click on Insert Page break icon.
  1. See that Page break is inserted at the end of text.
  1. Select all the content in Editor using Ctrl + A and click on Align Right icon.

Expected Result: All paragraphs are aligned to the right.

Actual Result: we are getting an Error. Error is 'null' is null or not an object

Attachments (1)

8247.patch (4.5 KB) - added by garry.yao 5 years ago.

Download all attachments as: .zip

Change History (6)

comment:1 Changed 6 years ago by j.swiderski

  • Keywords IE added
  • Status changed from new to confirmed

Reproducible from CKEditor 3.6 rev [6904].

Message: Can’t get property „getNextSourceNode”: object is empty or undefined
Line: 88
URI: /3.6.1/ckeditor/_source/plugins/domiterator/plugin.js

NOTE: The first "Align Right" produces JS error, The second moves the contents to the right.

Changed 5 years ago by garry.yao

comment:2 Changed 5 years ago by garry.yao

  • Owner set to garry.yao
  • Status changed from confirmed to review

Patch deals with the case where block iterator is invoked on a collapsed range inside of an empty paragraph.

Last edited 5 years ago by garry.yao (previous) (diff)

comment:3 Changed 5 years ago by garry.yao

Last edited 5 years ago by garry.yao (previous) (diff)

comment:4 Changed 5 years ago by fredck

  • Milestone set to CKEditor 3.6.3
  • Status changed from review to review_passed

comment:5 Changed 5 years ago by garry.yao

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

Fixed with [7410].

Note: See TracTickets for help on using tickets.
© 2003 – 2017 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy