Ticket #3919 (closed Bug: fixed)

Opened 5 years ago

Last modified 5 years ago

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

3919.patch (725 bytes) - added by martinkou 5 years ago.
3919_2.patch (590 bytes) - added by garry.yao 5 years ago.

Change History

comment:1 Changed 5 years ago by fredck

  • Milestone set to CKEditor 3.0

comment:2 Changed 5 years ago by fredck

  • Priority changed from Normal to High

comment:3 Changed 5 years ago by arczi

  • Keywords Confirmed added

The same JS error using Justify

comment:4 Changed 5 years ago by martinkou

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

Changed 5 years ago by martinkou

comment:5 Changed 5 years ago by martinkou

  • Keywords Review? added

comment:6 Changed 5 years ago by garry.yao

  • Keywords Review+ added; Review? removed

comment:7 Changed 5 years ago by martinkou

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

Fixed with [3854].

Click here for more info about our SVN system.

comment:8 Changed 5 years ago by garry.yao

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

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

Changed 5 years ago by garry.yao

comment:9 Changed 5 years ago by fredck

  • Keywords Review+ added

comment:10 Changed 5 years ago by garry.yao

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

Fixed with [3865].

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