Opened 14 years ago

Closed 14 years ago

#10320 closed Bug (fixed)

Justify plugin should add elements to ACF based on Enter Mode

Reported by: Jakub Ś Owned by: Piotrek Koszuliński
Priority: Normal Milestone: CKEditor 4.1.1
Component: General Version: 4.1
Keywords: Cc:

Description

In full editor use below settings in replacebycode sample.

var editor = CKEDITOR.replace( 'editor1', {
				//allowedContent : true
				enterMode : CKEDITOR.ENTER_BR,
				toolbar : [
				['Font', 'FontSize'], ['TextColor', 'BGColor'], ['Bold', 'Italic', 'Underline', 'Strike'], ['Subscript', 'Superscript'],
				['JustifyLeft', 'JustifyRight', 'JustifyCenter', 'JustifyBlock'], ['NumberedList', 'BulletedList'], ['Outdent', 'Indent', 'Blockquote'], ['Table', 'HorizontalRule', 'SpecialChar'], ['Link', 'Unlink', 'Image'],
				['Cut', 'Copy', 'Paste', 'PasteText', 'PasteFromWord'], ['Undo', 'Redo', '-', 'Find', 'Replace', '-', 'SelectAll', 'RemoveFormat'], ['Source']
				]
			} );

Notice that Justify plugin is disabled no matter what you click on. As discussed with @Rainmar this is because Justify Plugin adds only P to ACF while in other modes (BR and DIV) elements are "wrapped" in DIV when justified (block elements aren't but this rather doesn't change anything). To solve this in other modes, Justify should report DIV to ACF.

Change History (5)

comment:1 Changed 14 years ago by Jakub Ś

Status: new → confirmed

comment:2 Changed 14 years ago by Piotrek Koszuliński

Milestone: → CKEditor 4.1.1
Owner: set to Piotrek Koszuliński
Status: confirmed → assigned

comment:3 Changed 14 years ago by Piotrek Koszuliński

Status: assigned → review

Pushed ​t/10320 on dev and tests.

comment:4 Changed 14 years ago by Olek Nowodziński

Status: review → review_passed

comment:5 Changed 14 years ago by Piotrek Koszuliński

Resolution: → fixed
Status: review_passed → closed

Fixed on master with ​git:5097130 on dev and d48a863 on tests.

Note: See TracTickets for help on using tickets.
© 2003 – 2022, CKSource sp. z o.o. sp.k. All rights reserved. | Terms of use | Privacy policy