Opened 8 years ago

Closed 7 years ago

Last modified 7 years ago

#5152 closed Bug (fixed)

Indentation using classes is buggy

Reported by: Alfonso Martínez de Lizarrondo Owned by: Alfonso Martínez de Lizarrondo
Priority: Normal Milestone: CKEditor 3.3
Component: Core : Styles Version: 3.3.1
Keywords: Review+ Cc: Paweł Wilk

Description

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;
}

Attachments (1)

5152.patch (1.2 KB) - added by Alfonso Martínez de Lizarrondo 8 years ago.
Proposed patch

Download all attachments as: .zip

Change History (8)

Changed 8 years ago by Alfonso Martínez de Lizarrondo

Attachment: 5152.patch added

Proposed patch

comment:1 Changed 8 years ago by Alfonso Martínez de Lizarrondo

Keywords: Review? added
Owner: set to Alfonso Martínez de Lizarrondo
Status: newassigned

comment:2 Changed 8 years ago by Frederico Caldeira Knabben

Milestone: CKEditor 3.3

comment:3 Changed 7 years ago by Alfonso Martínez de Lizarrondo

Cc: Paweł Wilk added

#5249 has been marked as dup

comment:4 Changed 7 years ago by Garry Yao

Keywords: Review+ added; Review? removed

comment:5 Changed 7 years ago by Alfonso Martínez de Lizarrondo

Resolution: fixed
Status: assignedclosed

Fixed with [5211]

comment:6 in reply to:  5 Changed 7 years ago by Jorge

Version: 3.13.3.1

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

comment:7 Changed 7 years ago by Frederico Caldeira Knabben

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

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