Custom Query

We could make a research in the future if it is possible to support Lotus Notes in the same way as we support MS Word.

We should check what HTML information is carried from Lotus Notes to browsers.

The problem:

Now the command state (if it is disable or not) is based on the "setState" method which is called by many plugins. It can be changed when selection change, content in selection change (#12618), mode change, editor is in readOnly mode and maybe some more cases. Also many plugins set state of the commands manually. We get a state spaghetti and it is very often that one part of code try to disable command and another enable it back.

The solution:

There should be one entry point for all parts of code which want to change the state. Good candidate seems to be refresh method and event. The example scenario can be readonly feature. It will:

• add refresh event listener and disable every command if editor is in the read only mode,
• call refresh() function when editor switch to the readonly mode.

This way every time the state of the command is suppose to be changed all listeners are called and we can be sure that we do not enable feature which should be disabled.

In fact direct usage of setState() is always wrong because we can not be sure if any other plugin want to change the state too.

The idea of changes is:

• mark setState() as deprecated,
• refresh event get evt.data.state parameter which can be changed by any listener.
• commandDefinition methods (refresh and exec) are only (other then command.refresh()) methods which can change state directly (by adding data.state parameter or returning new state),
• refresh() method contents may be moved to the refresh listeners, so it will be possible to add listener before them,
• everything what refresh() will do is firing refresh event and setting the state based on what listeners will set.

This way if someone want to disable command and be sure no one else will enable it back, can add refresh listener with 0 priority, set evt.data.state = CKEDITOR.TRISTATE_DISABLED and cancel event.

It seems to be possible to apply these changes without breaking backward compatibility. setState method will still work (as bad as it works now), but it will be deprecated. Also it is pretty huge change, because setState is used in many plugins.

If there are more closing that opening tags, it will cause a type error in JavaScript, example:

[/u]

This can be fixed with additional checks for candidate in parser.onTagClose, but not sure if it's the correct way to do it.

Also this BB code doesn't behave as expected:

[ [u]Test[/u] ]

Should result in underlined Test in square brackets. This requires an additional fix in bbcPartsRegex I guess (for example not matching another opening square bracket).

When you create a table, I can click the right button and open a menu to edit the table.

When I click on "cell" have the option "Insert cell before" and "after entering cell". These sentences are wrong in Portuguese, is written ("inserir célula a esquerda" and "inserir célula a direita") and the right would be ("inserir célula antes" and "inserir célula depois").

(This replaces #12932, which got messy)

Currently implementing hidpi is not documented. This ticket will try and describe what specifically needs documenting. My notes will be broader than hidpi, as I think more can be written about icon referencing in general.

To implement hidpi, hidpi:true needs setting on the plugin definition. This works in tandem with the 'icons' plugin definition property, telling ckeditor to automatically reference for the plugin's images/hidpi/example.png instead of the plugin's images/example.png. Explain that CKEditor is declarative in this way, and uses convention-over-configuration, it doesn't/can't/won't do filesystem searches or have any separate icon registry for plugins, apart from automatic insertion of reference to sprites for those plugins built in via CKBuilder.

It should mention that icons are always implemented using a background-image inline style, not using stylesheets or srcset. This confused me a lot, because you actually detect hidpi in JavaScript and set CKEDITOR.env.hidpi globally, which is an unusual (but effective) approach.

Following on from this, a plugin that sets a button icon via addButton (rather than the 'icons' plugin property), can look at CKEDITOR.env.hidpi to set the hidpi image path as needed.

The API docs for addButton are incomplete: ​http://docs.ckeditor.com/#!/api/CKEDITOR.ui

Some properties on 'definition' are not documented, such as the 'icon' property. For a list of properties one can currently find it deeper in the code at plugins/button/plugin.js -- I suggest going through the properties here and deciding which to document alongside addButton.

When documenting the 'icon' property, also mention that this is not the primary way to set icons, which is to use the 'icons' property on the plugin itself and to use matching icon/button names. Explain it is usually better to do it the primary way because then skins are able to override the icons if they wish.

The API docs for addButton are incomplete: ​http://docs.ckeditor.com/#!/api/CKEDITOR.pluginDefinition

It does not document 'icons'. 'icons' is particularly tricky because it is a comma-separated list, not a native JS list. This caused me quite a lot of confusion when debugging. Make sure it clearly specifies it as a comma-separated list.

Reproduced the following in latest Chrome, IE10, and FF30.

CKEditor 4.4.6 full version instance with following configuration:

"removePlugins" : 'elementspath,magicline', "allowedContent" : true, "pasteFromWordRemoveFontStyles" : false, "pasteFromWordRemoveStyles" : false, "defaultLanguage" : 'en', "disableNativeSpellChecker" : false, "browserContextMenuOnCtrl" : true, "enterMode" : CKEDITOR.ENTER_DIV, "resize_enabled" : false, "removeDialogTabs": 'link:upload',

Paste the following text in the "Source":

<div style="font-size:10pt;font-family:Tahoma">&nbsp;</div>

<div class="MsoNormal"><i>Suggested Tabs<o:p></o:p></i></div>

<div class="MsoNormal">- Activity*<o:p></o:p></div>

<div class="MsoNormal">- Details*</div>

In WYSIWYG mode place your cursor at the end of "-Activty* and hit the "Enter" or "Return" key.

Attempt to paste text of any kind on that line, whether using the right click paste dialog or CTRL + V. In Chrome there will be a brief flash of text but nothing will be pasted. In FF30 and IE10 nothing will happen at all.

Hitting enter a second time allows the user to paste text.

Attempting to reproduce this in the CKEditor demo failed, because after pasting the above code in the Source and returning to the WYSIWYG view the <o:p> tags are removed.

If i insert an <a> with only an id attribute and no name attribute, it will simply be discarded by CKEditor.

Tested in latest version of the demo: ​http://ckeditor.com/demo

Steps to reproduce

• Insert the following HTML in SOURCE-mode:

  <p>hello <a name="xxx" id="xxx">xxx</a> hello</p>
  <p>hello <a id="yyy">yyy</a> hello</p>
  

• Go back to WYSIWYG mode, and now only the first (xxx) anchor is visible.

• If you again return back to SOURCE-mode, the second (yyy) anchor has complete vanished.

In HTML5 the "name" attribute for <a> is been obsolete, "id" should be used instead: ​http://www.w3.org/TR/html-markup/a.html#a-constraints

​KaTeX seems to be very promising library to create mathematical formulas on the client side. We should make a research if we can use it in the CKEditor.

When I build forms with drop down lists without and selected default an error occurs. Afterthat it is impossible to use the rest of the form.

put the example code in the full featured editor in source mode an than switch to normal mode:

​http://ckeditor.com/demo#full

<select>
	<option value="test1">Test1</option>
	<option value="test2">Test2</option>
	<option value="test3">Test3</option>
	<option value="test4">Test4</option>
</select>

Open the console in IE11 or Chrome. Double click on the list to edit it in the form dialog and the error is thrown. Uncaught TypeError: undefined is not a function. selectjs:line 10, column 193

You expect the "selected" attribute but it is optional.

I un-minified the select.js and build a work around. After fixing the one error the next one occures...

here my code parts:

select.js: around line 10(minified version)

[...]
{
	id:"txtValue",
	type:"text",
	widths:["25%","75%"],
	labelLayout:"horizontal",
	label:c.lang.forms.select.value,
	style:"width:350px",
	"default":"",
	className:"cke_disabled",
	onLoad:function()
	{
		this.getInputElement().setAttribute("readOnly",!0)
	},
	setup:function(a,b)
	{
		//console.log( 'a', a );
		//console.log( 'b', b );
		if ( typeof ( b === 'undefined' ) )
		{
			return;
		}
		else
		{
			//console.log( 'b', b );
			//console.log( 'b attr', b.$.attributes['selected'] );
			if( typeof ( ( b.$.attributes['selected'] ) === 'undefined' ) )
			{
				return;
			}
			else
			{
				"clear"==a ? this.setValue("") : "option"==a && b.getAttribute("selected") && this.setValue(b.$.value);
			}
		}
	}
},
[...]

select.js: around line 12(minified version)

[...]
{
	type:"select",
	id:"cmbName",
	label:"",
	title:"",
	size:5,
	style:"width:115px;height:75px",
	items:[],
	onChange:function()
	{
		var a=this.getDialog(),
			b=a.getContentElement("info","cmbValue"),
			e=a.getContentElement("info","txtOptName"),
			a=a.getContentElement("info","txtOptValue"),
			d=g(this);
		i(b,d);
		e.setValue(this.getValue());
		a.setValue(b.getValue())
	},
	setup:function(a,b)
	{
		"clear"==a ? k(this):"option"==a && h(this,b.getText(),
		b.getText(),
		this.getDialog().getParentEditor().document)
	},
	commit:function(a)
	{
		var b=this.getDialog(),
		e=l(this),
		d=l(b.getContentElement("info","cmbValue")),
		c=b.getContentElement("info","txtValue").getValue();
		k(a);
		
		//console.log( 'e', e );
		// count is wrong, MINUS ONE, but there ist a new problem in the list, an empty element occures
		for( var f=0; f<e.count()-1; f++ )
		{
			var g=h(a, e.getItem(f).getValue(), d.getItem(f).getValue(), b.getParentEditor().document );
			d.getItem(f).getValue()==c && ( g.setAttribute("selected","selected"), g.selected=!0 )
		}
	}
}]
[...]

select.js: around line 14(minified version)

[...]
{
	type:"select",
	id:"cmbValue",
	label:"",
	size:5,
	style:"width:115px;height:75px",
	items:[],
	onChange:function(){
		var a=this.getDialog(),b=a.getContentElement("info","cmbName"),e=a.getContentElement("info","txtOptName"),a=a.getContentElement("info","txtOptValue"),d=g(this);i(b,d);e.setValue(b.getValue());a.setValue(this.getValue())
	},
	setup:function(a,b){
		//console.log( 'HIER 2 a', a );
		//console.log( 'HIER 2 b', b );
		if("clear"==a)
		{
			k(this);
		}
		else if("option"==a)
		{
			try
			{
				var e=b.getValue();
				h(this,e,e,this.getDialog().getParentEditor().document);
				"selected"==b.getAttribute("selected") && this.getDialog().getContentElement("info","txtValue").setValue(e)
			}
			catch( e )
			{
				// nothing
			}
		}
	}
}]
[...]

Steps to reproduce:

1. Go to the official ckeditor demo here: ​http://ckeditor.com/demo#inline
2. Choose any editable area and insert an image
3. Click in another editable area

=> The focus is now set to this other editable area, with no image.

4. Click once on the image in the previous editable area.

=> It seems the focus is correctly to the editable area with the image. But, first strange thing, the image is not selected (as it should be).

5. Enter some text with the keyboard

=> Focus is unexpectedly given back to the editable area with no image, and text is added there!

The setupResizerfunction has 200LOC plus there's 30LOC of CSS, so that could be something. And there's nothing more there, so that would be an entire feature, which after some improvements perhaps could be reused by other plugins (like video embedding).

Related: #12746.

1. Open ​http://tests.ckeditor.dev:1030/tests/plugins/uploadwidget/manual/image
2. Drop an image.
3. While it's uploading, click the "x" in the notification. Perhaps you can also do some other action, but I observed that clicking notification is ok. Perhaps it's related to focus.
4. Wait for the upload to finish.
5. Click "undo".

• Expected: Image should be removed.
• Actual: Nothing happens.

1. Create the following in source mode in the ckeditor demo site using the full featured mode:

<p>p not in div</p>
<div>
<p>p in div</p>
</div>

2. Go to wysiwyg mode

3. Place the cursor on the left of the first character in the line "p not in div" and hit the delete key multiple times until all the characters on that line are deleted. Hit the delete key one more time (to remove the remaining <p>&nbsp</p>).

4. The p is removed but the div is now unexpectedly removed also. The results are below:

<p>p in div</p>

5. Expected:

<div>
<p>p in div</p>
</div>

Note: backspace behaves similarly.

The SCAYT plugin does not re-initialize itself after setData is called when using the divarea plugin. Based on the comment in the code it looks like this was a bug for the inline editing mode but was never addressed for the divarea plugin.

From the SCAYT plugin js:

		//#9439 after SetData method fires contentDom event and SCAYT create additional instanse
		// This way we should destroy SCAYT on setData event when contenteditable Iframe was re-created
		editor.on('setData', function() {
			scaytDestroy();

			// in inline mode SetData does not fire contentDom event
			if(editor.elementMode == CKEDITOR.ELEMENT_MODE_INLINE) {
				contentDomtHandler();
			}
		}, this, null, 50);

Here scaytDestroy is called. However, the contentDom event doesn't fire with setData with the divarea plugin and since elementMode is not ELEMENT_MODE_INLINE, contentDomtHandler doesn't get called to reinitialize the plugin.

1. Open in Firefox ​http://ckeditor.com/demo#full
2. Click preview
3. Result: an empty page

Works fine on ​http://ckeditor.com/latest/samples/replacebycode.html where CDN is not used.

Discovered when working on #12964.

See tests pushed in ​branch:t/12983.

It would be useful if we knew that guard will be called only once, because it makes some iterations easier (and guard is the only callback which is called for both element boundaries).

Prior to version 4.4.7 I used to have issues with interfering styles such as table-layout: fixed and underlines on links. I've been able to remove all of my CSS hacks but I have one final interfering style:

input[text='text'], input[type='password'], select {

height: 18px;

}

This affects all the text boxes within the dialogs.

It would be good if the reset could handle this.

Thanks

1. Put attached file into editor samples folder.
2. Editor is empty and starts in source mode.
3. Switch to wysiwyg by clicking Source button.
4. Type e.g. ABCDE
5. Press UNDO button

Result: Text is not removed.

Tested with inline edition only

When CkEditor has the focus, clicking outside the editor triggers a blur event. In order to handle this blur event, you can set a 'blur' handler directly on the div, or you can use the internal CkEditor event management (by setting a handler in CkEditor options). The later solution is way better, as it filters out blur events occurring when using the CkEditor toolbar.

However, when the event that ultimately triggers the blur event (for example a mouse click outside the edition area) also causes the editor destruction (for example because user clicked a button to terminate edition), the internal CkEditor blur event never occurs. This is not the expected behavior: in standard JavaScript, you always get the blur event, even if the div is then immediately destroyed.

How to reproduce:

1. Launch this fiddle: ​http://jsfiddle.net/7dhrjodx/4/.
2. Click inside the editable area => the CkEditor toolbar appears.
3. Click outside the editable area => the CkEditor toolbar disappears and the console displays "Global Blur" and "CkEditor Blur", meaning that the blur event has been caught both on the div and internally in CkEditor API.
4. Click inside the editable area again => the CkEditor toolbar appears.
5. Click on the "Finalize" button => the CkEditor toolbar disappears but the console only displays "Global Blur", meaning the internal CkEditor blur event has been lost.

We have used CKEditor as control as following:

 <CKeditor:CKEditorControl ID="txt_bodyHTML" runat="server" BasePath="/jscripts/CKeditor/"
                    Visible="False" DisableNativeSpellChecker="false"   ShiftEnterMode="P"  EnterMode="BR" >
</CKeditor:CKEditorControl> 

Shift Enter Mode is producing the BR

1. Put attached HTML file into samples folder
2. In first paragraph select part of text and copy/paste it (select only plain text).
3. Either select some plain text in another paragraph or in same paragraph
4. Press Delete

Result: Change event is not fired.

Problem can be reproduced from CKEditor 4.4.4 in Firefox and IE8-11.

Before release 4.4.4 change event worked the following way only in Firefox

1. When copy/paste text change event is fired 2 times
2. When select and delete change event is fired 2 times on delete
3. When select and delete some more text change event is fired 3 times on delete.

1. Create editor instance with the element with ID:

` <p>foo<a href="​http://ckeditor.com" id="link">bar</a></p> `

2. Drag part of the link (ex. letter "a"),
3. Drop it somewhere else (ex. after "fo", before "o").

Result: two elements with the same ID: ` <p>fo<a href="​http://ckeditor.com" id="link">a</a>o<a href="​http://ckeditor.com" id="link">br</a></p> `

Is should not be possible to duplicate ID by drag and drop. Paste event may check if there is element already element with that ID and remove "id" attribute if so.

On the other hand the same issue exists with the native drag and drop on Chrome.

Some helpful methods are missing (we could add them ourselves in the future):

• bender.stub, decorator for sinon.stub which will reset the stub method on tearDown,
• bender.spy, as above,
• assert.*, call sinon.assert.* and increment YUI assertion counter,
• maybe more, needs investigation.

Working on #12955 we agreed that, since counting _doneWeights and _totalWeights is pretty simple and can be stored in redundant variables, counting _doneWeights is over-complicated and can be a reason of the future bugs. Tasks should give its done wights to aggregator and it should could sum of them, when it is needed.

Also with the current implementation it is not possible to update with negative weight, for no apparent reason.

Since this changes may causes some changes in API this ticket should be done before releasing 4.5.0.

Steps to reproduce:

1. create a table
2. click inside the 1st cell
3. right click to get a context menu
4. place the cursor at the edge of the context menu
5. close the menu without selecting an option by left click, to restore the focus inside the cell
6. start typing

result: Unable to type in the 1st cell. Path bar displays 'body->table->tbody->tr->td' but focus is not in the cell.

To make it easy to reproduce, remove rounded borders from cke_panel class in moono skin, e.g.

-moz-border-radius: 3px;
-webkit-border-radius: 3px;
border-radius: 3px;

Chrome and FF were used for testing

Please see comment:4 and comment:5 for more details.

Our web app has been generating countless exceptions in CKEditor. I have finally been able to reproduce one of them. For the demo please visit '​http://jsfiddle.net/ftey46fc/3/' and follow these steps in the editor in the lower right:

1) Type three lines of text with the letters a, b, and c on them (one

letter per line).

2) Select the first two lines and make that a numbered list. 3) Select all three lines and change to a bulleted list.

An exception is generated "TypeError: a is null".

This error is caused because when bookmarks are selected/restored in the selectBookmarks function the starting bookmark has been removed thus it fails to select it in the moveToBookmark function so startNode is null and the setStartBefore call will a null parameters throws an exception. I changed said function during me debugging to:

setStartBefore: function (node) {

if (node == null) {

console.log('* About to throw an exception');

} this.setStart( node.getParent(), node.getIndex() );

}

to better illustrate that.

Why has the bookmark been purged? Well the long story is that a <li> with the start bookmark gets added before the two existing <li>s and so the first <li> has no text in it and it's purged by the 'changeListType' function, but it all seems to come down to the getNative function that looks kind of (this is my debug version now) like this:

getNative: function() {

if ( this._.cache.nativeSel !== undefined ){

return this._.cache.nativeSel;

} if(isMSSelection){

this._.cache.nativeSel = this.document.$.selection;

}else{

var win = this.document.getWindow(); this._.cache.nativeSel = win.$.getSelection();

}

console.log('* new native selection = anchor = ' + this._.cache.nativeSel.anchorNode.nodeName + ' focus node = ' + this._.cache.nativeSel.focusNode.nodeName); return ( this._.cache.nativeSel );

}

which I have modified to break complex lines into pieces and add the logging. When this returns the selection via getSelection the focus node SHOULD be on the 'a' #text node, but it's actually on the OL node. This works fine in Chrome and usually works fine in FireFox, but the CSS I inject into the document seems to trigger this problem. Remove that and it works fine.

That makes me question whether this is actually a Firefox bug that needs to be worked around, but I don't know my browser specs. In any event one quick fix I've done to code around this was a patch to the moveToBookmark function that basically verifies that the startBookmark was found but

a) I don't know the extent that this kind of bug is going to have

on the rest of the system. If there are other places where the start bookmarks premature removal is going to wreak havoc then I'd like to know about it.

b) I don't know if there are other cases where we're going to see this

type of problem because of other CSS that Firefox/CKEditor doesn't work with properly. Like I said we're seeing all kinds of exceptions coming from CKEditor and it could all be permutations of this bug. I don't know yet.

c) When the exception occurs it leaves the end bookmark span in the

HTML and it seems that that can cause later problems.

Let me know if there's any additional info I can provide. I'm just glad to finally get this bug in a form reproducible outside of our code base.

• Thanks, Alan

I have a word document consisting of just two words "Hello to". This is actually a simplified version of the original document. The font-family for this text is Calibri (body). I don't notice any difference in formatting between these two words in the word document.

When attempting to copy and paste this text into ckeditor, the font-family of "Hello" is correctly preserved as Calibri, but the font-family of the word "to" in incorrectly changed to Arial. I don't understand why it doesn't also preserve the font-family of the "to" text.

Here is my config.js:

CKEDITOR.editorConfig = function( config ) {

config.pasteFromWordRemoveStyles = false; config.pasteFromWordRemoveFontStyles = false;

};

Here is the source HTML in ckeditor upon pasting the text:

<p><span style="font-size:10pt"><span style="font-family:calibri">Hello </span>to</span></p>

This behaviour is evident using IE 11. Please can someone assist with this?

Hi Please consider this scenario:

1. change direction to rtl
2. type some words. as you know in arabic/persian languages letters are attached to each other. for example توسعه
3. when you are writing it for the first time, there is no problem but if you put cursor after ع and hit backspace to delete it, the letters س and ه should be attached to each other but they get separated. the result after removing one character should be like this توسه but it's not.

thanks

We don't know to what value set the alt, but perhaps we should create it anyway.

1. Open ​http://tests.ckeditor.dev:1030/tests/plugins/uploadwidget/manual/image
2. Drop image.
3. Get data.
4. The image does not have the alt attr.

I have similar issue to Ticket #7175. However, I'm not using an iframe and am using ckeditor 4.4.7.

I use CKEditor inside a showmodaldialog in IE. If only 1 tab is open on the browser, then the menus for styles/font/size don't work. Works fine if 2 tabs are open.

The code fix for 7175 seems to apply to older version of ckeditor and I'm having trouble to apply the same to 4.4.7. If you have time to provide instruction for 4.4.7. it would be a great help to me.

To reproduce:

Scenario 1

1. Open replacebycode sample in IE11
2. Set inline style like font-size 48px or marker class and type something.
3. Set language to Korean and keyboard to Hanja Convert
4. without moving cursor type something in Korean

Result: text is not styled.

Scenario 2

1. Open replacebycode sample in IE11
2. Set language to Korean and keyboard to Hanja Convert
3. Set inline style like font-size 48px or marker class and type something in Korean.

Result: text is not styled.

Scenario 2

1. Open replacebycode sample in IE11
2. Set language to Korean and keyboard to Hanja Convert
3. Type something in Korean
4. Select text and style it with inline style like font-size 48px or marker class
5. Move cursor to the end of line, press enter and type something in Korean

Result: text is not styled.

Problem can be reproduced from CKEditor 3.0 at least.

I have checked native contenteditable element and styles are continued to the second line or they don't get cancelled when you change input method.

Focus in tabs panel works in a weird way if there are some disabled tabs.

Disabled tabs should be focusable, but user must not be able to activate them. It is the same policy as for disabled toolbar buttons.

Consider following TC:

1. Open ​http://ckeditor.dev/plugins/image2/samples/image2.html
2. Place caret in an image caption.
3. Open link dialog.
4. Focus tab (the only enabled).
5. Press right arrow key.

Expected result:
You should be able to focus any tab despite it being disabled or not.

Current result:
Using left/right keys you can reach the disabled tabs, but you can't leave them then.

Additional info:

1. You might have hard times seeing what's focused if branch ​t/13027 was not yet merged. In such case add following CSS rule:

   a.cke_dialog_tab:focus
   {
   	background: #ebebeb;
   }
   

In Chrome and IE

1. Add dir="rtl" in body element of page.
2. Press maximize button.
3. Now if you want to open any of drpdowns they not work.

In IE8:

1.Create a numeroted list

2.Use Shift+Enter for a line break (adding Br tag)

3.Copy or cut the list

4.Paste the list

Problem: the line break is missing

Detect in 4.4.1, but present here: ​http://ckeditor.com/demo

1. Please use attached File. It contains shapes and images.
2. Copy whole document and paste it into CKEditor in IE11 (Ctrl+A, Ctrl+C, Ctrl+V).

Result: Some images and shapes get duplicated and some omitted. Please see the results image.

Widgets use copybin, while on modern browsers they can now use the dataTransfer integration which we introduced in 4.5.0. The question, though, is whether we have to do this, because on all IEs we'll have to leave the copybin, so we would need to branch the code. There may be a strong argument for this change - recognising content origin (internal, cross editor, external) and filtering Webkit/Blinks awful inline styles.

All application I tested have a different behavior for copy and paste and for drag and drop.

• After paste, caret is at the end of the pasted content (what makes sense, user most probably wants to start typing there).
• After drop, dropped content is selected (what also makes sense, because it may happen that user drops the content in not exactly correct position and wants to drag it again).

At the moment we always put the caret at the end, handle the drop in the "paste" way. The problem is that now insertHtml set the selection and this method does not know what operation causes insertion. Fortunately paste event has such information, so the selection should be done by paste event or information about the type of the selection should be passed to the insertHtml. The second way seems to be better because of backward compatibility.

1. Open ​http:///ckeditor.dev/plugins/image2/samples/image2.html
2. Open image dialog. Type any URL and press ok.
3. Click outside editor (select some text).
4. Doubleclick the created widget.
5. Change alignment and press ok.
6. Press CTRL+Z - the previous alignment should be restored (OK).
7. Press CTRL+Z:
   • Expected: the widget should be removed.
   • Actual: You need to press CTRL+Z twice.

The reason why this happens is that initially snapshot contains widget classes in the following order:

 cke_widget_focused cke_widget_selected

Unfortunately, once editor is blurred, the cke_widget_focused class is removed and when it's added back (on editor focus, on doubleclick) it's added at the end.

The solution would be to always add cke_widget_focused class after cke_widget_selected.

Reproduced on Chrome.

config.extraPlugins = 'timestamp, savebtn';

the "white space / blank" after the comma in the assembled string 'timestamp, savebtn' in CKEditor.config produces a "resource not found" error and leads finally to a missing plugin.

A very tough bug since it is hidden and one expects white spaces usually in such strings to be stripped. So one does recognize the issue immediatly.

Add text then add a form below. Add a CR after the text and then delete it with the delete key. The form html disappears.

After extending ​these tests with selection matching for #12729 it turned out that the bakcspace/delete handling doesn't set correct selection.

To fix this issue we may need to cache bookmark before modification process, and then apply it into content.

Related commit is in ​branch:t/13098.

Just extra information what fails before fix, and what after in ​gist.

When a page break is present in the editor, no indication of its presence is given to screen reader users. Users can still insert/delete page breaks, but are not told that it is there.

I've seen a previous bug raised (#5975) 5 years ago reporting something similar, and I can see now that there is an aria-label attribute on the page break div, but it does not seem that the screen reader acknowledges this.

Reproduction Steps

1 - Open the full featured editor demo at ​http://ckeditor.com/demo#full
2 - Add a page break to the editor
3 - Move the cursor around the document near the page break with JAWS active

When moving up and down lines, the page break is skipped altogether and without the use of sight it seems like the paragraph under the break follows straight on from the paragraph that is above it.

When moving left and right over the end of the preceding paragraph the screen reader reads out "blank" when the cursor is on the break.

Environment

OS - Windows 7 (64-bit) international version
Browser - Firefox 36.0.4
Screen Reader - JAWS 16.0.1925 (64-bit)
CKEditor - This issue was found in 4.4.6 and can be reproduced in 4.4.7. It may have existed before then, but I am not sure.

This has also been reproduced with version 15 of JAWS and on IE9. Using this combination, IE allows you to tab to focus on the page break, but the screen reader just reads out the editor description as if you had just entered it.

This issue is very similar to #3140 (perhaps even a duplicate)

When you select whole table and press any justify button, table contents, instead of whole table get aligned.
When you press on image on the other hand, you are able to move it around (fixed in #7430, tables were omitted).
Perhaps just like in #3140 we need extra plugin that will handle whole complex element instead of its contents.

Please provide the ability to resize Objects(Images, Tables, Flash etc) in CK Editor by just grabbing the corner of the object.

Uploading multiple images creates few thumbnails with low quality. I've change quality settings in config to 100 but that didn't help much. please see attached screen shot.

While using the CKEditor in source mode( in the config startupMode : "source" ) instantiating the CKEditor object, with the method replace, the object doesn't contain many basic attributes e.g. the document that contains the editor or the selection of the user (edit: selection is not accessible in source mode for a reason - see comment:1).

If we use instead the wysiwyg mode, everything works fine and we have those attributes.

1. Go to the demo (​http://ckeditor.com/demo)
2. Remove all contents
3. Choose "Special Container" from the "Styles" dropdown
4. Hit backspace

On Safari, Chrome and Firefox, the box will disappear, which is the expected behaviour. However in IE, nothing happens.

Tested with IE11 on Windows 7.

1. Open any sample with wysiwygarea.
2. CKEDITOR.instances.editor1.document.on( 'click', function() { console.log( 'click1' ) } );
3. Test that it works. OK.
4. CKEDITOR.instances.editor1.setData( 'xxx' );
5. Test that the click listener no longer works. OK.
6. CKEDITOR.instances.editor1.document.on( 'click', function() { console.log( 'click2' ) } );
7. Test that it works. NOK.

In general - if listener to some event was attached before set data, then it will no longer work after setting data and it won't be possible to attach a new listener to that event.

The reason is that setting data (unlike switching modes) keeps the same document, but writing to that document removes all its previous listeners.

On every domObject.on() we push the listener to domObject.getCustomData( '_' ) (the listeners repository) and if a native listener has not been added yet we add it to the native element. So there's only one native listener per event name.

Since setting data keeps the same document, the custom data (our listeners repository) stays the same. But the native listeners are removed. This means that when trying to add a new listener editor thinks that a native listener was already added, because the listeners repository already contains an entry for that event.

The solution will be to clear events repository on set data.

Memory is not released after destroying ckeditor instance.

How to reproduce: -open attached file in chrome -start record heap allocation (Chrome/DeveloperTools/Profiles/RecordHeapAllocations) -click on "add" -click on "remove" -stop record heap allocation -check memory leak

We found an issue in 4.4.x version where in ACF mode, we observe unexpected behaviour when we attempt deleting a white space character following a contenteditable=false span tag.

To reproduce the defect, use the following config:

CKEDITOR.editorConfig = function( config ) {

ACF configs config.allowedContent = true; turn off ACF by default

};

Open Firebug for you test sample page

1. In the console: CKEDITOR.instances.editor1.setData('Hey <span contenteditable="false"><a>there</a></span> :)')

(Notice here the space following the closing of the span tag and before colon character eg. "</span> :"

2. Click on the editor (caret is at the end of the line) and start removing characters using backspace until you reach ':' and click backspace again to remove it.

3. Hit backspace again-> the cursor goes inside the contenteditable area skipping the empty space

1. Open ​http://ckeditor.dev/plugins/widget/dev/nestedwidgets.html
2. Try dragging one of the placeholders.
3. The drop caret is invisible inside nested editable. It is visible only when dragging over the outer editable.

It seemed to work fine on major some time ago.

A user of a user got confused by "Latin text" in the image plugin. The user did too. And I think I also got momentarily confused by it in the past.

The typical user won't know what lorem ipsum is, and also may wonder why the shown text doesn't match the specified alt text. They will think the preview is for the image and nothing else. Of course, the alt text would not be visible, but the user does not necessarily understand any of this.

I propose a few simple changes to make the overall UI nicer...

1) Say "with sample neighboring text" next to "Preview", so it is clear it is a contextual preview 2) hspace/vspace is not friendly. Use unicode arrow symbols to illustrate the point visually 3) The caption for the lock/unlock button should be dynamic. It should say "Unlock" when it is locked (the default). Button captions should always say what *activating* the button does, and this neither does that nor shows current state because it's static.

If there is empty list in pasted from Word data CKEditor will show an alert "It was not possible to clean up the pasted data due to an internal error".

The simplest way to reproduce it is edit /plugins/pastefromword/filter/default.js file. Find the following code:

try {
	data = dataProcessor.toHtml( data );
} catch ( e ) {
	alert( editor.lang.pastefromword.error );
}

and override the data var above it:

data = '<div>Test paste empty list from Word <ul><li></li></ul></div>';

try {
	data = dataProcessor.toHtml( data );
} catch ( e ) {
	alert( editor.lang.pastefromword.error );
}

Open editor, click Paste from Word button type anything and click OK.

Test case: ​http://s.codepen.io/boomerang/43cbe53e7b6cec53f1410ecc00540be61428432853936/index.html

To reproduce:

1) Type some text in CKEditor

2) Focus on the other input field (to blur CKEditor)

3) Click the button to insert a small image

On all browsers except Windows Phone 8.1, the image is inserted at the last cursor location. On Windows Phone 8.1, the image is always inserted at the start of the CKEditor field.

​http://tests.ckeditor.dev:10450/tests/core/selection/editor#tests%2Fcore%2Fselection%2Feditor%20-%20test%20initial%20selection%20after%20set%20data%20in%20autoparagraphing%20inline%20editor

I found out that this test is red when the previous editor has the elementspath plugin enabled. Precisely - when the bottom space is created. This sounds like a nonsense, but should have a manual test if we plan to ignore this test.

Check the manual test in ​branch:t/13154.

1. Open ​http://tests.ckeditor.dev:1030/tests/plugins/clipboard/manual/draganddrop
2. Focus inline editor.

Result: toolbar has let margin and it should not have it.

I've managed to reproduce this bug with the latest nightly build, as follows:

1. Download the nightly build.
2. Download and include the following plugin:
   • ​http://ckeditor.com/addon/image2
3. Download an include the additional required plugins:
   • ​http://ckeditor.com/addon/widget
   • ​http://ckeditor.com/addon/lineutils
4. Add the following to config.js:

config.extraPlugins = 'lineutils'; config.extraPlugins = 'widget'; config.extraPlugins = 'image2'; config.image2_alignClasses = [ 'text-editor-image-left', 'text-editor-image-center', 'text-editor-image-right' ];

5. Add the following to content.css:

.text-editor-image-left {

float: left; margin-right: 10px;

} .text-editor-image-right {

float: right;

margin-left: 10px;

}

5. Open one of the samples: Desktop/ckeditor/samples/replacebyclass.html
6. Double click an image to open the dialog and select Alignment: Right
   • The image floats to the right.
7. Toggle Source mode on/off
   • The image remains floated to the right.
8. Select the image and click the Link icon, and add a URL.
   • The image remains floated right.
9. Toggle the Source mode on/off.
   • The image loses its float right.
   • If you view source mode you'll see that the image is still assigned the correct class, it's just not being detected.
   • If you double click the image to open the dialog the Alignment: Right option is NOT selected.

Browser: Chrome Version: 4.5.0 beta

Open: ​http://tests.ckeditor.dev:10450/tests/plugins/uploadwidget/manual/image

1. D&D some image onto editor.
2. While image is uploading, press Ctrl + Z, to undo upload

There should be information that image upload has been aborted.

3. D&D image once again to upload.

Result: There is no notification present while uploading.

​http://plnkr.co/edit/3cupW1Y2T7zWWTXxtyyF?p=preview

It returns Uncaught TypeError: Cannot read property 'children' of null

Happens when smiley is at the very front of quote.

[quote]:D hello[\quote]

1. See ​http://tests.ckeditor.dev:1030/tests/plugins/embed/manual/embed
2. Try to copy&paste any of the widgets.
3. Empty widget is pasted.

Some details:

• This happens only with some kinds of widgets. It cannot be reproduced with image2, but can be with code snippets.
• This isn't a regression. We didn't notice this (and no one reported this - I guess no one uses widgets with IE8) because code snippets aren't supported on IE8.

Browser: IE11

1. Open: ​http://tests.ckeditor.dev:1030/tests/plugins/clipboard/manual/draganddrop.
2. D&D some widget (image) internally.
3. D&D some external text from textarea located above into editor.

Result: There is an error in ​here.

Instead of trying to override buttons' onclick properties it will be much better to cover the entire toolbar with another div.

1. ​http://tests.ckeditor.dev:1030/tests/plugins/uploadwidget/manual/image
2. Drop an image into editor.
3. While uploading it has a size of about 10x10.
4. Once uploaded it has its correct size.

Hi, I have big problem to maximize CK 4.4.7. For some security I have defined html{display:none;}. My JS then enable html with code: if(window.top === window.self) { document.documentElement.style.display = 'block';} Everything works perfect to situation when I click CK to maximize. All browsers hardcode change style and I see only blank window. I tried in my config this: CKEDITOR.instances.content.on('maximize', function() { document.documentElement.style.display = 'block'; });

It works only in Chrome not in FF and IE. I dont want to change my security improvements, but CK maximize must work.

Next issue it on maximize CK is problem comboboxes. See picture.

1. The configuration looks like a code snippet to copy. Perhaps some border could better indicate it's ready for editing. Maybe similar to what we use for showing currently modified toolbar items in the editor?

2. It is not focused, there is no blinking cursor.

Please refer to: 12284.

1. Paste below code into editor and switch to source:

   <p>text&nbsp;<strong>bold</strong></p>
   <ol>
   	<li>item</li>
   </ol>
   

2. Put cursor at the end of bold (make sure you are inside bold by looking at element's path)
3. Press delete

Result: List item gets bold.

If there is paragraph instead of list below the bold paragraph, it won't get bolded/styled on delete.

<p>text&nbsp;<strong>bold</strong></p>
<p>text</p>

Problem can be reproduced from CKEditor 3.6.5 rev. [7620]

The main bugs are:

1.When a layout with more than 1 column is inserted, a widget out of the layout can only be dragged and dropped into the first column.

2.When there are is 3-column layout. Because of the bug pointed in '2' the widget can only be dragged into the first column.Then the widget can easily be dragged into the middle column but cannot be dragged to the third, even if it was in the middle beforehand.

Here is image of the bug.

1. Open replacebycode sample and press New Page command
2. Type some text
3. Click Ctrl+A and Ctrl+B (text is bold and element's path shows bpdy p strong)
4. Press Backspace or Delete buttons
5. Element's path shows body p and bold button is deactivated.
6. Start typing.

Result: Typed text is bold (First problem), bold button is active and element's path is showing "body p b" (<- notice the conversion, this is second problem).

Problems can be reproduced from CKEditor 3.0 in Blink and Webkit browsers.

1. Open replacebycode sample and press New Page command
2. Type some text
3. Click Ctrl+A and Ctrl+B (Don't press ctrl+B before typing -> it works then)
4. Press Backspace any number of times you want. Text is removed but element's path is still showing "body p strong"

4. Instead of Backspace press Delete. The first key press removes only text. Second key press removes the formatting.

Problem can be reproduced from CKEditor 3.0 in IE8-11.

To Reproduce:

1. Open any sample
2. Type in the console:

CKEDITOR.instances[EDITOR].on('change', function(){console.log('change...')});

3. Type anything in the editor ( event is triggered on every key press )
4. CTRL + A to select all the text or highlight it with the mouse
5. Ctrl + Right click to get browser context menu
6. Select Delete menu option

Problem: The event is not triggered

Test the issue from ​http://ckeditor.com/demo#full

1. Set config.enterMode = CKEDITOR.ENTER_BR;
2. See the differece in behavior between 2.1 and 2.2:

2.1. Type two lines content(<br> will exist), select them, click Formatted. <br> will be replaced /n in side a <pre>
2.2. Click Formatted, type two lines. <br> won't be replaced with /n in side a <pre>

How do solve the inconsistent behavior? Can we keep <br> after clicking Formatted?

Thanks in advance.

We want to use CKEditor to provide a full html document editor, and we want to use the <style> element in the <head> section.

We found that when using IE 8, the attribute type="text/css" in the <style> element is changed to type="text" when switching from source to visual editor then switch back to source view.

I am using the nightly build in the following steps, but I have used the master and the major branch on github and they all have the same problem.

Here are the step I used to reproduce this bug :

• Use IE
• Open the ​full page editing sample (from plugins\wysiwygarea\samples\fullpage.html)
• Open the Dev Console (F12), switch to IE 8 rendering mode (instead of edge)
• Switch the editor to Source mode
• Add <style type="text/css">body { color:red }</style> into the head element
• Switch to visual mode, the text is red
• Switch to Source view, the attribute type has changed to text instead of text/css
• Switch to visual mode, the text is not red anymore

I have tried to fix the bug by myself, but after 2 hours digging into the parser in the javascript console I surrender. Everything seems fine (the source is well parsed), but at some point when the DOM is inserted the content is modified.

If you have hints about how I should help to fix the problem by myself feel free to point me, or if you have any workaround.

I have tried to use bender but was stuck by a strange error in testbuilder.js on master branch. Anyway it was not related to the debugging of the code.

Please see the attachments, it would be cool to make these links a bit more functional.

For example I can edit parent container properties using right click context menu, but if there are multiple containers it is quite hard to do. Would be cool if I can do right click on items in status bar and see the same "Edit container" option.

An additional feature could be to highlight elements in the editor while hovering corresponding elements in the status bar e.g. like google chrome dev. tools do (see the second screenshot).

Best, Alexander

Hey CkEditor team

We found the align button in Ckeditor tool bar does not work correctly with enterMode as CKEDITOR.ENTER_BR

By default, the enterMode is <p>.

Under this mode, the content of editor always have <p></br><p> inside, which can not be removed by backspace(Look up through HTML inspector).

Just because of this, the left-align button is hovered when we click on the editor input area with nothing typed in.

However, under enterMode as CKEDITOR.ENTER_BR, the content is empty inside <body>

Just leak of <p></br><p>, the left-align button is NOT hovered to indicate its current states.

I wonder if the align-button(at least left-align button) is designed to work under mode of CKEDITOR.ENTER_BR.

If yes, should this issue be considered as a bug.

Many thanks John.

It is hard to find matching elements in htmlParser.element. Real DOM has couple of methods like find, findOne, byId, byClass, etc., but HMTL parser has nothing except forEach() which allows you to iterate over the entire sub tree and find whatever you need.

Implementing find/findOne which would work in the same way as in real DOM is reasonable, because the CSS selectors implementation would be huge. Therefore, I propose a simpler method accepting a callback or element name. It can be implemented in few lines of code using forEach().

Based on #12854.

I have a bunch of images displayed as block elements through CSS. When I try to apply a block style to a paragraph containing such images, a new paragraph is created instead of getting the style applied. CKEditor works as expected when removing the display:block style from these images.

Hi, i found a bug in the input select Dialog.

Steps:

1. Click on input select
2. Dialog: add some options
3. Set one added option as default
4. The default Value is now visible in "Value" field (under the "Name" field)
5. Delete the previously setted default option.

=> 6. The value of the option which was deleted is still visible in the upper "Value" field.

Expected Behaviour: => value in field "Value" should be removed on deleting the default option.

Note: However if you open the dialog again (contextMenu edit selection). The value in "Value" field is removed.

So the broken behaviour is only on creating a new selection.

Currently doing:

var range = new CKEDITOR.dom.range( root );
var range2 = range.clone();

will log that containers outside of the root are set. It does not make sense.

Used both on-site installation and demo here Steps to review the bug:

1. create new table using ckeditor (usual base 3 rows 2 columns)
2. add any content to the cells.
3. try to D&D the content -doesn't matter if it's any of text or image - from cell to cell

You'll see that these actions make the table cells disappeared.

Versions checked: from 4.4.7 placed as demo and lower

Checked under Windows 7/Chrome.

Correct behavior:

1. Go to ​http://sdk.ckeditor.com/samples/inline.html
2. Click the top right editable div
3. Select "Lorem ipsum"
4. Use Ctrl+C to copy the selection to the clipboard
5. Use Ctrl+V to paste "Lorem ipsum" 30 times

-> The editable div grows in height to adjust to the additional content.

Unexpected behavior:

Perform the exact same steps, but in step 3, select "Lorem " (Lorem + space) instead of "Lorem ipsum".
-> At step 5, after about 20 paste operations, the editable div starts growing in width instead of height.

When working in the ckeditor, you used to be able insert a check box in one area and then click on top of the check box (you did not have to highlight it) and copy and paste it. You can no longer do that.

If you click on top of the check box and copy it with the context menu, it just pates a space (no check box).

If you very tediously highlight the check box and copy it, it includes a space before the check box and a space after the check box.

Hello, thanks for the most accessible editor anywhere! I can't use a CMS without it. BUT... a BUG: My eyesight requires the 'High Contrast Mode'. (info at: ​http://ckeditor.com/blog/CKEditor-WAI-ARIA-Usable-Accessibility

But two buttons don't work in this mode. In fact they don't show up at all, they dissapear:

'TextColor', 'BGColor' (background color button). Versions I tested with bug: 3.x to 4.4.7

1. Insert below code into editor (div or table)

   <div style="float:right;">TEST</div>
   
   <div align="right">TEST</div>
   
   <table align="right">
   <tr><td>TEST</td></tr>
   </table>
   
   <table style="float:right;">
   <tr><td>TEST</td></tr>
   </table>
   

2. Hover below/over the element

Result: No magic line is displayed and there is no possibility exit the element.

1.Insert below code into editor

<div style="float: right;">TEST</div>

2. Put cursor at the end or simply into element and press enter key few times.

Result: New lines are not created (visually) and cursor stays in first line. You can't exit it with enter key or arrows.

Switch to source mode. The result is:

<div style="float: right;">TEST</div>
<div style="float: right;">&nbsp;</div>
<div style="float: right;">&nbsp;</div>

Problem can be reproduced in all browsers from CKEditor 3.0.

I haven't mentioned <div align="right">TEST</div> because it works with HTML4 doctype but not with HTML5 doctype where this attribute is not supported. On the other hand in native contenteditable elements enter key works as expected so perhaps it is worth looking into this particular issue as well.

How to reproduce:

Working demo to verify: ​http://jsfiddle.net/aZqL9/7/

1 - Add two DIVs

<div contenteditable="true" id="editable"><img src="http://i.imgur.com/M3LOzdI.jpg"/></div>

<div contenteditable="true" id="editable2"><img src="http://i.imgur.com/aD7V3hQ.jpg" width="320"/></div>

2 - Add a button to set the editors as readonly

<input type="button" onclick="editor.setReadOnly(true); editor2.setReadOnly(true);" value="Make readonly">

3 - Execute the loading of the editors in a <script>

CKEDITOR.disableAutoInline = true;

var editor = CKEDITOR.inline( document.getElementById( 'editable' ) );
var editor2 = CKEDITOR.inline( document.getElementById( 'editable2' ) );

5 - When it loads, click the button to set the editors to read only

6 - Do a right click on the first editor

7 - Do a right click on the 2nd editor. Javascript error is shown

Working demo to verify: ​http://jsfiddle.net/aZqL9/7/

Reproducible in: IE11

1. Use below code in HTML page

   var editor = CKEDITOR.replace( 'editor1', {});
   editor.on('instanceReady', function( evt ){
   	editor.setData('test');
   });
   

2. Access denied error gets thrown in IE8-IE11

Message: Access Denied.
URI: /ckeditor-dev/core/dom/node.js
Line: 663

1. Open: ​http://tests.ckeditor.dev:1030/tests/plugins/uploadwidget/manual/image2
2. Add some image into editor.
3. Enable image caption.
4. Drop another one into image caption.

Effect: Image uploading notification is visible, but nothing changes in the editor content.

Perhaps some day in the future it would make sense to make some kind of source code validator plugin to check the correctness of the HTML code modified by user.

There are cases like #13306 where it's even hard to notice that the source code was broken as a result of manual changes in the code. Some tool that could validate the HTML content before the mode is switched back to wysiwyg could help in dealing with mistakes and prevent from creating broken content.

Of course this is a complex task, especially if we consider notifying user in some user-friendly way where the content is possible broken. In case of unclosed tags it's even harder.

When I dragged a list from MS Word 2013 to CKEditor 4.5 I get something what looks like a list but is not a real list:

<p>&bull;&nbsp;&nbsp; &nbsp;Sdf<br />
&bull;&nbsp;&nbsp; &nbsp;Sdfsdf<br />
&bull;&nbsp;&nbsp; &nbsp;sdf<br />
&nbsp;</p>

The content in the paste event is:

"•&nbsp;&nbsp; &nbsp;Sdf<br>•&nbsp;&nbsp; &nbsp;Sdfsdf<br>•&nbsp;&nbsp; &nbsp;sdf<br>

We could fix this and change into a list.

It was working before custom drag and drop was implemented, so before 4.5. I was able to reproduce this issue in Firefox and IE 11. In Chrome dropping list works fine.

1. Open ​http://tests.ckeditor.dev:1030/tests/plugins/clipboard/manual/paste
2. Select all content (which includes image2) in editor.
3. Cut content.
4. Paste removed content again.

Two images are uploaded.

The editor resizing tip is aligned to bottom right while it should be aligned to the bottom left.

See attached snapshot for illustration.

If you have CkEditor zoom plugin and you set zoom level above 100% scroll doesn't work as it should. I have zoom at body set to 1.1 and I go to the bottom of the document. Then I press enter a few times and cursor goes below visible area. Scroller is visible, but it doesn't scroll to the bottom. But then if I press one character, editor scrolls to the bottom.

To reproduce download attachment, run on web server and follow instructions on the page.

Tested on Win8/Chrome.

1. Open ​http://cksource.com/ckeditor/services#accessibility-checker.
2. Scroll down to table in the editor.
3. Open table properties dialog.
4. Set Headers option to both

Actual result: There is an error in the console.

That's because editor table HTML is wrong - there are empty tr`s in the table. Removing them will fix the issue.

Browser: IE8

1. Open ​http://tests.ckeditor.dev:1030/tests/tickets/13011/1
2. D&D image somewhere (from the last line).
3. D&D anchor somewhere (from the first line).

Result: Dropped element is image but not anchor.

Issue: With enter mode as <br>, decreasing indent will cause different lines form back into 1 single line.

Steps to reproduce:

1. Use enterkey.html in sample of entryKey plugin.
2. Change "When Enter is pressed:" to "Break the line with a <BR>".
3. Type 123 and then press Enter key, do the same for 456, 789.
4. Now we have 123,456,789 in THREE lines.
5. Select them all to insert bulleted list.
6. Select the last line 789 and click on decrease indent.
7. Now 789 is removed from the list. And now do the same to 456.

Expected Result: 456,789 are in two lines.

Actual Result: 456,789, are in the same line.

We have figure out the root cause and one possible solution.

See ​https://github.com/liferay/liferay-ckeditor/pull/4

1. Test the issue in ​http://ckeditor.com/demo#full
2. Click Paste plugin, and it display the dialog box.
3. Resize the dialog box.

Actual result: the text area didn't adjust to the changed dialog box. Expected result: the text area should adjust to the dialog box.

I have prepared the fix from ​https://github.com/yuhai/liferay-ckeditor/commit/bac522635b92b29959fd0b2288e2887e359a982a. Could you please help check it? Please help solve the issue.

Thanks.

Steps to reproduce

1. Open ckeditor.com/demo#bbcode with Chrome
2. Paste enough text to make the vertical scrollbar appear
3. Click on the bottom right corner to resize ckeditor
4. While resizing (do not release the click), move the pointer to the vertical scrollbar
5. Release the click with the pointer on the scroll bar
6. Move the pointer around the screen

Expected behaviour:
ckeditor stops resizing when releasing the click

Acutal behaviour:
ckeditor keeps resizing along with the mouse movements

1. Visit ​http://tests.ckeditor.dev:1030/tests/plugins/autoembed/manual/autoembed
2. Copy embeddable URL, i.e. https://twitter.com/reinmarpl/status/573118615274315776.
3. Focus editable.
4. Hold CTRL+V for a while.
5. A number of links is pasted and corresponding embed requests turn them into widgets.
6. CTRL+Z.

Expected: The last embeddable object invoked is reverted back to link.

Actual: All widgets are reverted back to the chain of links. Sometimes only some group of widgets is reverted. The point is the invocation of multiple widgets is registered within a single undo step.