Opened 10 years ago

Closed 10 years ago

#3919 closed Bug (fixed)

Justify and indent/outdent commands are broken

Reported by: Neil Jenkins Owned by: Martin Kou
Priority: Must have (possibly next milestone) Milestone: CKEditor 3.0
Component: General Version:
Keywords: Confirmed Review+ Cc:

Description

Latest SVN trunk version (3851), these commands fail to work completely. Seems to be caused by bookmarks not being cleared up but left in the DOM. Actual error thrown is in the getNextParagraph method of the domiterator plugin. (this._.lastNode is null).

Verified on Mac OSX 10.5.7 with Firefox 3.5 and Safari 4.0.2

Attachments (2)

3919.patch (725 bytes) - added by Martin Kou 10 years ago.
3919_2.patch (590 bytes) - added by Garry Yao 10 years ago.

Download all attachments as: .zip

Change History (12)

comment:1 Changed 10 years ago by Frederico Caldeira Knabben

Milestone: CKEditor 3.0

comment:2 Changed 10 years ago by Frederico Caldeira Knabben

Priority: NormalHigh

comment:3 Changed 10 years ago by Artur Formella

Keywords: Confirmed added

The same JS error using Justify

comment:4 Changed 10 years ago by Martin Kou

Owner: set to Martin Kou
Status: newassigned

Changed 10 years ago by Martin Kou

Attachment: 3919.patch added

comment:5 Changed 10 years ago by Martin Kou

Keywords: Review? added

comment:6 Changed 10 years ago by Garry Yao

Keywords: Review+ added; Review? removed

comment:7 Changed 10 years ago by Martin Kou

Resolution: fixed
Status: assignedclosed

Fixed with [3854].

Click here for more info about our SVN system.

comment:8 Changed 10 years ago by Garry Yao

Keywords: Review+ removed
Resolution: fixed
Status: closedreopened

There's a typo in the patch which I didn't figure out.

Changed 10 years ago by Garry Yao

Attachment: 3919_2.patch added

comment:9 Changed 10 years ago by Frederico Caldeira Knabben

Keywords: Review+ added

comment:10 Changed 10 years ago by Garry Yao

Resolution: fixed
Status: reopenedclosed

Fixed with [3865].

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