Ticket #6207 (closed Bug: fixed)

Opened 4 years ago

Last modified 3 years ago

Indentation using classes fail

Reported by: jorgeoa Owned by:
Priority: Normal Milestone:
Component: Core : Styles Version: 3.4.1
Keywords: Cc:

Description

There is not possible to indent 3 levels, and the remove indentation buttom only works in the first level.

This bug had reported and solved in version 3.1 (Ticket #5152), but in version 3.3.1 [5548] it "came back".

Replace the line:

element.addClass( editor.config.indentClasses[ indentStep - 1 ] );

with this one to solve it:

element.$.className = CKEDITOR.tools.ltrim( className + ' ' + editor.config.indentClasses[ indentStep - 1 ] );

Change History

comment:1 Changed 3 years ago by garry.yao

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

Expired in 3.4.2.

comment:2 Changed 3 years ago by jorgeoa

  • Version changed from 3.4.1 to 3.4.3 (SVN - trunk)

comment:3 Changed 3 years ago by drasgard

I just tried the nightly build and it looks like this is now fixed.

comment:4 Changed 3 years ago by garry.yao

  • Version changed from 3.4.3 (SVN - trunk) to 3.4.1
Note: See TracTickets for help on using tickets.
© 2003 – 2012 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy