Indentation using classes is buggy

[Alfonso Martínez de Lizarrondo]

Seems that it's not possible to indent that way a third level, and removing the indent only works if there's a single level.

Tested with

indentClasses : ['Indent1', 'Indent2', 'Indent3'],

and

.Indent1
{
	margin-left: 40px;
}

.Indent2
{
	margin-left: 80px;
}

.Indent3
{
	margin-left: 120px;
}

[Alfonso Martínez de Lizarrondo]

Proposed patch

[Alfonso Martínez de Lizarrondo]

#5249 has been marked as dup

[Alfonso Martínez de Lizarrondo]

Fixed with [5211]

[Jorge]

Replying to alfonsoml:

Fixed with [5211]

Hi,

In version 3.3.1 [5548] the indentation using classes bug "came back".

The code proposed by alfonsoml:

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

was replaced with the buggy code:

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

Using the alfonso's patch works well:

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

The bug also exists in 3.3.2 and 3.4 beta

[Frederico Caldeira Knabben]

@jorgeoa, please open a new ticket for it as this one is already closed.