Opened 6 years ago

Closed 6 years ago

#3919 closed Bug (fixed)

Justify and indent/outdent commands are broken

Reported by: neilj Owned by: martinkou
Priority: High 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 martinkou 6 years ago.
3919_2.patch (590 bytes) - added by garry.yao 6 years ago.

Download all attachments as: .zip

Change History (12)

comment:1 Changed 6 years ago by fredck

  • Milestone set to CKEditor 3.0

comment:2 Changed 6 years ago by fredck

  • Priority changed from Normal to High

comment:3 Changed 6 years ago by arczi

  • Keywords Confirmed added

The same JS error using Justify

comment:4 Changed 6 years ago by martinkou

  • Owner set to martinkou
  • Status changed from new to assigned

Changed 6 years ago by martinkou

comment:5 Changed 6 years ago by martinkou

  • Keywords Review? added

comment:6 Changed 6 years ago by garry.yao

  • Keywords Review+ added; Review? removed

comment:7 Changed 6 years ago by martinkou

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

Fixed with [3854].

Click here for more info about our SVN system.

comment:8 Changed 6 years ago by garry.yao

  • Keywords Review+ removed
  • Resolution fixed deleted
  • Status changed from closed to reopened

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

Changed 6 years ago by garry.yao

comment:9 Changed 6 years ago by fredck

  • Keywords Review+ added

comment:10 Changed 6 years ago by garry.yao

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

Fixed with [3865].

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