Opened 6 years ago

Closed 6 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 Minnekanti Owned by: Garry Yao
Priority: Normal Milestone: CKEditor 3.6.3
Component: General Version: 3.6
Keywords: IBM IE Cc: Damian, Teresa Monahan

Description

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 6 years ago.

Download all attachments as: .zip

Change History (6)

comment:1 Changed 6 years ago by Jakub Ś

Keywords: IE added
Status: newconfirmed

Reproducible from CKEditor 3.6 rev [6904].

JS ERROR:
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 6 years ago by Garry Yao

Attachment: 8247.patch added

comment:2 Changed 6 years ago by Garry Yao

Owner: set to Garry Yao
Status: confirmedreview

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

Last edited 6 years ago by Garry Yao (previous) (diff)

comment:3 Changed 6 years ago by Garry Yao

Version 0, edited 6 years ago by Garry Yao (next)

comment:4 Changed 6 years ago by Frederico Caldeira Knabben

Milestone: CKEditor 3.6.3
Status: reviewreview_passed

comment:5 Changed 6 years ago by Garry Yao

Resolution: fixed
Status: review_passedclosed

Fixed with [7410].

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