#6207 closed Bug (fixed)
Indentation using classes fail
Reported by: | Jorge | 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 (4)
comment:1 Changed 14 years ago by
Resolution: | → fixed |
---|---|
Status: | new → closed |
comment:2 Changed 14 years ago by
Version: | 3.4.1 → 3.4.3 (SVN - trunk) |
---|
The bug is still present in 3.4.2 and trunk source.
3.4.2
http://dev.ckeditor.com/browser/CKEditor/releases/stable/_source/plugins/indent/plugin.js#L255
Trunk
http://dev.ckeditor.com/browser/CKEditor/trunk/_source/plugins/indent/plugin.js#L252
comment:3 Changed 14 years ago by
I just tried the nightly build and it looks like this is now fixed.
comment:4 Changed 14 years ago by
Version: | 3.4.3 (SVN - trunk) → 3.4.1 |
---|
Expired in 3.4.2.