Custom Query

When inserting an image, width and height fields are optional.

However, when left empty, after pressing the "Ok" button, CKEditor shows two alert messages: "Width must be a number", "Height must be a number".

Introduced with [6555]. Confirmed at least in IE8. Note: "0" should not be allowed, but empty values should be still allowed.

When i run the fck editor 2.6.4 in IE9 i get a javascript error:

';' expected

It seems that in the generated minified file a d/while loop is the problem. I did a local fix in the minified version, here's a patch for the source:

Index: web/pub/fckeditor-2.6.4/editor/_source/internals/fckxhtml.js =================================================================== --- web/pub/fckeditor-2.6.4/editor/_source/internals/fckxhtml.js (revision 126316) +++ web/pub/fckeditor-2.6.4/editor/_source/internals/fckxhtml.js (working copy) @@ -446,7 +446,7 @@

} while ( htmlNode && htmlNode.nodeType == 3 && htmlNode.nodeValue.Trim().length == 0 )

• } while ( htmlNode && htmlNode.nodeName.toLowerCase() == 'li' )

+ } while ( htmlNode && htmlNode.nodeName.toLowerCase() == 'li' );

return newTarget ;

},

Blockquotes are used mainly for replying in mails, in that environment the expected behavior is that when the user press Enter inside the quoted mail, that mail is splitted so he can reply inline to each part.

The proposed patch fixed this issue.

In IE9, toolbar widths seem to have a strange fractional width, which change the horizontal width with by as much as 5-6 px (in my use case). I couldn't trace if this is due to CSS (I'm not at all familiar with the UI design) or a rendering bug.

At the very least, in that function the

	var styles = CKEDITOR.style.getStyleText( def );

isn't used, but the final part to remove the style has remained that way since it was created in [5847] that changed also the applyObjectStyle but was reverted in [5978]

So a little clean up is needed there, deciding if it must use getStyleText or def.Styles (but using a local variable), and correcting the comments.

To reproduce the defect:

1. Open Ajax sample, type some text, select it and apply some formatting to it ( for ex: bold).

2. Select the text and click on Insert/Remove Numbered list button.

3. See that a Numbered list starts with the select text as it's first list item.

4. Keep cursor at end of the list item and press Enter twice.

Expected Result:

Cursor comes out of Numbered list and a new empty paragraph starts after Numbered list.

Actual Result:

Cursor will not comes out of Numbered list no matter how many times we press Enter. every time we press Enter it is creating a new empty list item.

Same issue happens with Bulleted list

To reproduce the defect:

Scenario 1:

1. Open Ajax Sample, click on Insert/Remove Numbered list button.

2. See that Empty Numbered list starts.

3. Open Paragraph format drop down list and click on Heading 1 option.

Expected Result:

h1 tag is applied to list item and when we start typing the text, it should be in Heading 1 format.

Actual Result:

A   with h1 is created inside list item and when we start typing the text, it is in normal format.

Scenario 2:

1. Open Ajax Sample, click on Insert/Remove Numbered list button.

2. See that Empty Numbered list starts.

3. Enter some text in first list item, open Paragraph format drop down list and click on Heading 1 option.

Expected Result:

Existing text in the list should be converted to Heading 1 format and when we start typing the text, it should be in Heading 1 format.

Actual Result:

Existing text in the list is converted to Heading 1 format,cursor goes to a different line in same list item & when we start typing the text, it is in normal format.

Steps to reproduce:

1. Open the replacebycode.html sample in Chrome or Safari.
   
2. Change browser window width.
   
3. The editor toolbar and right border do not reflow and the right editor border is gone. The HTML element containing the editor seems to occupy the whole window width.

Please note this happens to the first editor instance on the page with multiple editors OR to any CKEditor instance after you switch between Source and WYSIWYG view.

Steps to reproduce the defect:

1. Open the Ajax sample.

2. Paste in the following code:

<ul>
	<li dir="rtl">
		1</li>
	<li>
		[2</li>
	<li>
		3]</li>
</ul>

3. Select the first list item & click on increase indent.

Expected: The first list item (RTL) becomes indented.

Actual: List items 2 & 3 (LTR) become indented.

Steps to reproduce the defect:

1. Open the Ajax sample in FF.

2. Paste the following code into the source view.

<ol dir="rtl">
	<li>
		list item 1
		<ol dir="ltr">
			<li>
				list item 2</li>
		</ol>
	</li>
</ol>

3. Place the cursor at the end of the last list item.

4. Press Enter twice.

Expected: The number 2 should display in the editor relating to the second list item of the outer list.

Actual: No number is displayed.

Note: If the editor is removed the number is visible but if you recreate the editor the number disappears again. Same behaviour for bulleted lists.

hi,

I make migration from the ancien .net component to the new official component

i try to change a configuration value " toolBar " on the component function (OnInit, OnLoad, etc).

        protected override void OnInit(EventArgs e)
        {
            base.OnInit(e);
            this.Toolbar = "MyToolBar";
        }
                        

but the value is not save on the config, i have always the FULL toolbar

after debug i found this code

 public CKEditorConfig config
		{
			get
			{
				if (ViewState["CKEditorConfig" + this.ClientID] == null)
					ViewState["CKEditorConfig" + this.ClientID] = new CKEditorConfig(this.BasePath);
				return (CKEditorConfig)ViewState["CKEditorConfig" + this.ClientID];
			}
			set { ViewState["CKEditorConfig" + this.ClientID] = value; }
		}

but this.ClientID is null when the affectation is make on constructor. And value change on the component life.

and after when test if (ViewState+ this.ClientID? == null) the result is true, and the ToolBar value(and other) are overwrite

why not use a private Guid, Created on the Constructor.

private Guid _guidId = new Guid();

public CKEditorControl()
        {
            base.TextMode = TextBoxMode.MultiLine;
            _guidId = Guid.NewGuid();
            
            this.config = new CKEditorConfig(this.BasePath.StartsWith("~") ? this.ResolveUrl(this.BasePath) : this.BasePath);    
        }

public CKEditorConfig config
        {
            get
            {
                if (ViewState["CKEditorConfig" + _guidId] == null)
                    ViewState["CKEditorConfig" + _guidId] = new CKEditorConfig(this.BasePath, Context);
                return (CKEditorConfig)ViewState["CKEditorConfig" + _guidId];
            }
            set { ViewState["CKEditorConfig" + _guidId] = value; }
        }

A Guid ensures is always the same ViewState is used throughout the life of the component.

Steps to reproduce the defect:

1. Open the Ajax sample in IE8.

2. Paste in the following code.

<ol>
	<li>list item 1
		<ol>
			<li>list item 2</li>
		</ol>
	</li>
	<li>list item 3</li>
</ol>

3. Place the cursor at the end of the last list item.

4. Use backspace to delete the last list item so that the cursor is now at the end of the second list item.

Result: An extra empty list item is added underneath the second list item (see screenshot).

I got the bug below using CKEditor 3.5.2 (revision 6450) on Firefox 3.6.15 in Linux Ubuntu when I tried to use horizontal rule.

To reproduce this bug, just type two paragraphs and between both, put a horizontal rule. After, try to remove the element by clicking over it and press Del. I can't do this. If i try to remove using backspace, the problem occurs too.

Thanks in advice.

1. Go to demo page
   

2. Paste some content (i.e. from MS Word) by choosing 'Edit' -> 'Paste' from the browser toolbar.
   

3. Switch to source mode and observe the garbage HTML.

Tested in WinXP (FF 3.6.15, IE8)

Seems nothing is listening to the document paste event.

Reported in ​comment:14 by jamescun

1. Open ​Rev 6593 samples in FF 3.6.15
2. Click between "You" and "are": "You |are"
3. Open the Paste dialog.
4. Leave the paste dialog empty & click ok. Result: Line break inserted between "You" and "are".

It looks like it was introduced with [6458] (?)

TypeError: Cannot call method 'match' of null

File: pastefromword/filters/default.js
Ln 1121:
Parent Method: CKEDITOR.cleanWord calling dataProcessor.toHtml

I've traced the error to the following call stack:
Ln 897: calling stylesFilter with an array, which later calls styleMigrateFilter
Ln 546: styleMigrateFilter attempts to create a CKEDITOR.htmlParser.element object with the null parameter. Since the element doesn't have a tagName, .match is called on null (core/htmlparser/element.js ln: 40) which throws an exception.

Tested on Firefox 3, and Chrome.

I've attached the example document that I was using to test pasting.

When the language direction attribute is not set explicitly then it defaults to 'ltr'. The problem occurs when we toggle between 'ltr' and 'rtl' languages. When we change it to 'rtl' for language like arabic the text direction changes to right to left. But when we change the language back to English and if we do not change the direction to 'ltr' the direction remains as 'rtl' even for English language.

Whats expected is that if we do not set the language direction or language it is defaulted to 'en' and 'ltr'. But this is not happening. This is causing a problem in our application. We do not explicitly set the direction for english. So when we move from arabic to english the text direction still appears to be 'rtl' instead of being 'ltr'.

I have attached a sample file wherein you can see this behavior. In the script i had not set the language direction to 'ltr' when language is set to 'en' as this is how it function is our application. Please check this our.

Note: For some reason, in the sample html, you have to click the radio buttons 9 corresponding to the language) twice, every time for the editor to reload. As I was testing this out in a hurry I could not find out the reason. But I don't think thats crucial for the issue in question.

1. Start with the following content and selection:

   <p>
   	&nbsp;</p>
   <p>
   	&nbsp;</p>
   <p>
   	<span style="color: #0070c0">^This should be blue.</span></p>
   

2. Press backspace to remove the middle line
3. Result:

   <p>
   	&nbsp;</p>
   <p>
   	This should be blue.</p>
   

   Note: the blue color is gone.

Steps to reproduce

1. Open replacebyclass sample
2. Clear all content and using the "Paste from Word" button paste contents from Word document attached in #7371 (​BugReport_Example.docx)
3. Switch to source mode
4. Result: an unwanted paragraph appears at the top:

   <p>
   	&nbsp;</p>
   <p>
   	This should be blue.</p>
   ...
   

The following content, which was produced and displayed fine in 2.X, makes 3.5.2 fail:

<div class=\"buttons\" style=\"float: right;\">
<ul>
    <li><a href=\"/fr/path/path\"><span>BLA BLA BLAH</span></a></li>
</ul>
</div>

I have been able to reproduce this issue in the Demo page editors, too.

More precisely, this content fails to be displayed in FF4, IE9 and Chrome 10, but is displayed fine in Safari5 (Win) and Opera11. It worked well in FF3.X last week. IE9 hangs.

FF4 shows the following Javascript error:

regular expression too complex

Note that FF4 comes with a new Regex engine.

Steps:

1. Go to www.ckeditor.com/demo
2. Set focus inside the editor body
3. Click on the "Source" button to enter HTML source mode

Result: Source mode is displayed, but the textarea does not have focus
Expected: Focus set to the textarea

Furthermore, once you're in source mode if you then:

4. Click on the "Source" button again to return to WYSIWYG mode

Result: WYSIWYG mode is displayed, but the editor does not have focus
Expected: Focus set to the editor

This feels like a bug to me, but after searching for similar cases and reading over previous related bugs that have been closed, I'm not so sure.

My expected/desired behavior is also related to ticket #3709 (cursor position and/or the current selection should be remain intact when switching between editor modes).

The current style system is based on defining which element is going to be used for that "Style", but there are times when it's desirable to have a Style that can be applied to a set of elements.

This means of course, that in this situation the element must exist, it won't be created and so it's behavior is more along the current STYLE_OBJECT system

The patch enhances the style system to allow an "elements" property in the style definition to specify the set of allowed tags, and also allows to specify the type of the style to override the default behavior so even if a single element like "p" is specified it can work in this way instead of having the default "block" behavior.

​http://www.pierewaaienscheveningen.nl/

This is about the fck editor build in the Website Baker CMS. Everytime I have to make a change in the Homepage of this site, I have to renew the code of the Addthis script:

<!-- AddThis Button BEGIN -->
<div class="addthis_toolbox addthis_default_style ">
<a href="http://www.addthis.com/bookmark.php?v=250&amp;username=pierewaaienscheveningen" class="addthis_button_compact">Share</a>
<span class="addthis_separator">|</span>
<a class="addthis_button_facebook"></a>
<a class="addthis_button_hyves"></a>
<a class="addthis_button_twitter"></a>
<a class="addthis_button_email"></a>
</div>
<script type="text/javascript">var addthis_config = {"data_track_clickback":true};</script>
<script type="text/javascript" src="http://s7.addthis.com/js/250/addthis_widget.js#username=pierewaaienscheveningen"></script>
<!-- AddThis Button END -->

Everytime the WYSIWYG feature is used, the code will be changed to:

<!-- AddThis Button BEGIN -->
                        <div class="addthis_toolbox addthis_default_style "><a class="addthis_button_compact" href="http://www.addthis.com/bookmark.php?v=250&amp;username=pierewaaienscheveningen">Share</a> <span class="addthis_separator">|</span></div>
                        <script type="text/javascript">var addthis_config = {"data_track_clickback":true};</script><script type="text/javascript" src="http://s7.addthis.com/js/250/addthis_widget.js#username=pierewaaienscheveningen"></script><!-- AddThis Button END -->

And that does not work as intended! Hope you can help!

If multiple editor instances are active on one page, the last/lowest instance automatically receives focus upon load. This happens only in Firefox (tested with V3.6.15 but not IEs) and only if all text areas are empty. This happens despite of CKEDITOR.startupFocus being set to 'false' explicitely. This is likely related to bug #4472 which was supposedly fixed in V3.4.1. I've attached a test page.

Clicking the Omega (Special Characters) symbol, (​http://screencast.com/t/LzmUDUbfl), selecting a character, and then clicking Ok, only to reopening CKEditor causes it never load properly (​http://screencast.com/t/z4RGIXE5Bzm)

Chrome spits on an error:

Uncaught TypeError: Object #<Object> has no method 'indexOf' : ckeditor.js:9

Using IE9 and doing copy/paste. The system hangs. Sends ie in to light background and locks the IE application. Can do nothing but go to task manager and kill ie.

Hello,

I have complete Lithuanian translation files for CKFinder.

Complete Lithuanian translation files for CKFinder.

Steps to reproduce the defect:

1. Open the skins sample in Firefox.

2. Open the Table Properties dialog in the V2 skin.

3. Drag the dialog to the right of the screen (as far right as possible).

Result: The dialog shrinks in width (see screenshot). This happens for all dialogs when dragged to the right of the screen. It also happens in the office skin.

The problem was introduced in 3.5 in changeset ​6118 when the getSize function in the dialog/plugin.js was changed from

getSize : function()
{
	if ( !this._.updateSize )
		return this._.size;
	var element = this._.element.getFirst();
	var size = this._.size = { width : element.$.offsetWidth || 0, height : element.$.offsetHeight || 0};

	// If either the offsetWidth or offsetHeight is 0, the element isn't visible.
	this._.updateSize = !size.width || !size.height;

	return size;
}

to

getSize : function()
{
	var element = this._.element.getFirst();
	return { width : element.$.offsetWidth || 0, height : element.$.offsetHeight || 0};
}

The problem can be resolved by using the old getSize function.

Drag&drop doesn't create Undo images, so they can't be restored correctly.

The patch fixes the problem (as best as I can).

In Firefox all the situations seems to be correct now. In IE8, drag&drop from external sources don't create the initial image. In Webkit the final image isn't created. Opera doesn't support Drag&drop

As in #7131 we introduced list item properties pasted from MS-WORD, a corresponding dialog should be provided to make these properties "manageable" within the editor, "List Properties" dialog right now is only capable of managing the list root, it could be extended to support this case.

1. Paste the following code

   <ul>
   	<li>
   		Copy form word</li>
   	<li>
   		copy from word 3 level lists</li>
   	<li>
   		klawiszeTemty kt&oacute;re powtarzają się i warto najpierw zrobić search.</li>
   </ul>
   <p>
   	<strong>Temty kt&oacute;re powtarzają się i warto najpierw zrobić search.</strong></p>
   <p>
   	&nbsp;</p>
   <ul style="list-style-type: decimal;">
   	<li>
   		Copy form word</li>
   	<li>
   		copy from word 3 level lists</li>
   	<li>
   		klawisze
   		<div>
   			<strong>Temty kt&oacute;re powtarzają się i warto najpierw zrobić search.</strong></div>
   		<div>
   			&nbsp;</div>
   	</li>
   	<li>
   		Copy form word</li>
   	<li>
   		copy from word 3 level lists</li>
   	<li>
   		klawiszeTemty kt&oacute;re powtarzają się i warto najpierw zrobić search.<br />
   		<br />
   		&bull; Copy form word<br />
   		&bull; copy from word 3 level lists<br />
   		&bull; klawisze<br />
   		<p>
   			&nbsp;</p>
   	</li>
   </ul>
   

2. Go to wysiwyg mode.
3. Search for word "copy"
4. You will se that editor doesn't scroll to show matching words.

This dopesn't work under FF3.6 in WIN7 and XP since CKEditor version 3.0.1

In CKEditor version 3.0 "cycle search" doesn't work at all. When editor reaches the end of the document it doesn't come back up and displays the dialog - no occurrences found.

NOW THE FUN PART: The text I have pasted is big enough for scrolling, but try to double the text by pasting it once more. Now everything works. Magic :)

NOTE : I HAVEN'T TESTED IT ON FF4 YET.

If an object is selected (an image or a table), the alignment buttons should act on that specific object, not the paragraph. At least the align left and right buttons.

These buttons should simply apply/remove the proper float style to the object.

Not sure if it has somehting to do with #5156, because it started happening in 3.3. Confimed in Safari 5.0.4 @ Mac.

Steps to reproduce

1. Open Link dialog
2. URL field gets focus
3. Press the "Esc" key to close the dialog
4. Click "Link" button again
5. Result: URL field is not active, cursor is still blinking in the editing area.

To reproduce the defect:

1. Open Ajax Sample, with out focus in editor body, click on Insert/Remove Numbered list button.

Expected Result:

See that Empty Numbered list starts.

Actual Result:

2 empty paragraphs are created one with a span

<p>
	<span id="cke_bm_101S" style="display: none; ">&nbsp;</span></p>
<p>
	&nbsp;</p>

With IE9 standard mode:

1. Load any of the sample page;
2. Put cursor at the end of block and press one shift enter;
3. Press shift enter again;
   • Expected Result: Cursor moved to the next line;
   • Actual Result: Cursor stays at the same line.

With IE9 standard mode:

1. Load the editor with the following input:

   <ol>
   	<li>
   		item1</li>
   	<li>
   		&nbsp;</li>
   	<li>
   		item2</li>
   </ol>
   

2. Try to move cursor into the second list item;

• Actual Result: It's impossible to access the middle list item

Confirmed with IE8/9 on Win7 at least, ALT + F11 wasn't bring focus to elements path bar button.

1) Paste the following text: "This is some sample text. You are using"

2) Align text to right.

3) Click button 'rtl'

4) Place the cursor after word "using" and start typing or press space and start typing or press enter and start typing.

5) The elements are created with attribute 'rtl' but when I write the text inside them it's 'ltr'.

Is this normal behavoiur of CKEditor?

Till CKEditor version 3.4.1 when you align sample text to right and press 'rtl' 'ltr' buttons nothing happens.

On Webkit, applying an inline style on a collapsed selection (sounds weird huh? ;)) makes the cusror to disappear.

• Open a sample.
• Without giving focus to the editor, click on one of the list buttons.

Note that a list is not created and a bookmark is being added to the document.

Load the following content and selection:

<table border="0" style="width: 150px;">
	<caption style="border:solid 1px black">
		Table^ title</caption>
	<tbody>
		<tr>
			<td>
				&nbsp;</td>
			<td>
				&nbsp;</td>
		</tr>
	</tbody>
</table>

• Right click and select table properties.
• Click OK.

JS error is thrown.
Not a recent regression.

I am getting this error when trying to edit an article in joomla 1.6 Warning: Invalid argument supplied for foreach() in /home/wwwsecon/public_html/plugins/editors/ckeditor/ckeditor.php on line 320

1. Load editor with the following source and selection:

   <ol>
   	<li>
   		ltr</li>
   </ol>
   <p dir="rtl">
   rtl^</p>
   

2. Click "Numbered List" button;

• Actual Result:

  <ol dir="rtl">
  	<li>
  		ltr</li>
  	<li>
  		rtl</li>
  </ol>
  

• Expected:

  <ol>
  	<li>
  		ltr</li>
  	<li dir="rtl">
  		rtl</li>
  </ol>
  

In Quirks mode, Last line of SCAYT dropdown menu is placed wrong, see attached screen

Environment

IE7,8,9

TC

• Load ui_language sample
• select any language
• Select Styles dropdown with mouse

Expected

List contains default styles

Actual

Empty Dropdown list shows, JS error is thrown

Message: Object doesn't support this property or method
Line: 82
Char: 395
Code: 0
URI: http://rev.ckeditor.com/ckeditor/trunk/6553/ckeditor.js

Looks like introduced with r6548

• Load the following in source mode:

  <p>
  	One</p>
  <pre>
  Two
  </pre>
  <p>
  	Three</p>
  

• Switch to wysiwyg mode
• Click on "Two"
• Result: an empty line is added below "Two"

Found in Firefox.

• Use this source code:

  <p>
  	<img alt="" src="http://dev.ckeditor.com/chrome/site/logo-ckeditor-dev.png" /></p>
  

• Right click on the image, open image properties.
• Set width field to 0. Hit ENTER and confirm the alert.
• The width field's content is now selected. Hit BACKSPACE, ENTER.

The dialog closes and the image width and height are set to 0.

Paste the following code and switch to wysiwyg mode. You can always create such list your self in CKEditor.

<ol>
	<li>
		i1
		<ol>
			<li>
				i11</li>
			<li>
				i12</li>
			<li>
				i13</li>
		</ol>
	</li>
	<li>
		i2</li>
	<li>
		i3</li>
	<li>
		i4</li>
</ol>

Place the cursor behind (or before) 'i11' element. Now toggle between "Decrease Indent" and "Increase Indent" (in that order) few times. You will see that line breaks (br tags) are inserted after 'i11' element every time you press a button.

This happens in FF browsers only. It has first showed up in CKEditor version 3.5.1.

1. Load editor with the following content and selection:

   <ol>
   	<li>
   		item1^
   		<ol>
   			<li>
   				item2</li>
   		</ol>
   	</li>
   </ol>
   

• Actual Result: Unexpected new line is opened after the nested list.

In IE, Paste into CKEditor:

<ol dir="rtl">
        <li>
                This.
                <ol dir="rtl">
                        <li>
                                This is CKEditor.
                                <ol dir="rtl">
                                        <li>
                                                This is some CKEditor.^</li>
                                </ol>
                        </li>
                </ol>
        </li>
</ol>

Set caret at the end of last line by mouse

first case

• press shift and right cursor key under my IE, the dot, at the end becomes first selected, then the "r"

Is it expected?

second case

• set caret in the middle of last line, and then move it to the end of line by pressing cursor keys.

caret jumps in similar way to selection in first case.

• Enter the following code into source:

  	<img alt="" src="http://dev.ckeditor.com/chrome/site/logo-ckeditor-dev.png" /></p>
  

• Switch the WYSIWYG.

Contents disappear.

Steps to reproduce the defect:

1. Open the Ajax sample in Firefox.

2. Open the iframe dialog.

3. Enter the following URL ​http://www.youtube.com/embed/rWTY8xtiZjc & click OK.

4. Click on the iframe placeholder to select the iframe.

5. Right click on one of the iframe placeholder resize handles (when the resize arrows are black) to display the context menu.

6. Choose iframe from the context menu to display the iframe dialog.

Result: The iframe dialog is displayed but all of the text fields in the dialog are empty.

Steps to reproduce the defect:

1. Open the Ajax sample in IE.

2. Insert a Flash/Image/Iframe with width 1000px & height 1000px.

3. Click on the Flash/Image/Iframe placeholder to select it (make sure the scrollbar is right down to the end of the editor).

4. Now try to click on any of the buttons in the toolbar.

Result: The toolbar buttons are disabled because the Flash/Image/Iframe placeholder has the focus.

Steps to reproduce the defect:

1. Open the Ajax sample in IE6.

2. Open a dialog that contains a drop down menu (e.g. Table Properties)

3. Drag the dialog back & forth from the left to the right of the screen.

Result: See that the dialog becomes distorted briefly due to the dropdown menus in the dialog.

Without focus in the editing area select some paragraph format, like "Heading 2". When typing text, the selected format is not used.

If you check the source code, you'll see that an empty header was created, but the text has not been entered inside of it.

<p>
	Sample text</p>
<h2>
	&nbsp;</h2>

Steps to reproduce the defect:

1. Open the Ajax sample.

2. Copy the bulleted list from the sample doc.

3. Paste the bulleted list into the editor.

Expected: The bulleted list is pasted into the editor.

Actual: A numbered list is pasted into the editor.

This is a regression in 3.5.3

Steps to reproduce the defect:

1. Open the Ajax sample in Firefox.

2. Copy the indented numbered list from the sample doc.

3. Paste it into the editor.

Result: The 3rd list item is not properly indented (see screenshot).

Note: The numbered list pastes as expected in IE.

Using Chrome 10.0.648.151 for Linux one is able to double-click a checkbox (or right-click and go to checkbox properties). In the properties dialog you can check or uncheck 'Selected'. This does not change the state of the checkbox from checked to unchecked or vice versa.

Using Firefox 3.5.15 for Linux the checkbox properties dialog box is unavailable. Neither double-clicking or right-click produces any events.

Steps to reproduce:

1. Set CKEDITOR.config.enterMode to CKEDITOR.ENTER_BR
2. Open up CKEDITOR on IE9
3. Type a sentence, press enter twice, type another sentence.

Expected result: Sentence 1

Sentence 2

Actual Result: Sentence 1 Sentence 2

The break tags are correctly being created by the editor, but it seems that IE9 is ignoring them.

How to fix: Open up enterkey\plugin.js

Ln# 273-274:

   if ( !CKEDITOR.env.ie )
      doc.createText( '\ufeff' ).insertAfter( lineBreak );

+ Add +

   else {
      doc.createElement( ' ' ).insertAfter( lineBreak );
   }

With editor using enterMode BR:

1. load editor with the following content:

   <p>line1<br />line2<br />line3</p>
   

2. Have the first two lines fully selected (by mouse drag to expand selection)
3. Click on any block format command, e.g. Indentation.

• Actual Result: Last line is also affected;
• Expected: Only selected lines are affected.

Load the XHTML output sample.[br] Select a word an apply for example the Comic font to it. Now select Times New Roman

Expected results:[br] A single span that makes the text appear in Times New Roman.

Actual results:[br] Two nested spans with and the text is shown in Comic.

This is the way that the font styles are defined for that sample:

font_style :
{
		element		: 'span',
		attributes		: { 'class' : '#(family)' },
		overrides	: [ { element : 'span', attributes : { 'class' : /^Font(?:Comic|Courier|Times)$/ } } ]
},

As the "element" is the same one that it's used in the overrides definition it seems to be cause of the problems.

The patch fixes this problem, but there might be better ways to fix it if it was planned in a different way (and there might be other places to review)

To reproduce the defect:

1. Open Ajax sample, Copy Numbered/Bulleted list from the attached doc and open Paste from Word dialog.

2. Paste the content in to the dialog and press OK button.

Expected Result:

Numbered/Bulleted list is pasted properly.

Actual Result:

we are getting an Unresponsive Script Error

Environment

MS Word 2007; WinXP IE7

Steps to reproduce

• In Word, create bulleted list with 3 items
• Select all and copy to clipboard
• In CKEditor clear content
• Paste selection using PasteFromWord button

Actual result

Under IE7, additional paragraph is added before pasted list

<p>
	&nbsp;</p>
<ul>
	<li>
		Test</li>
	<li>
		Test</li>
	<li>
		Test</li>
</ul>

On opera 11.01 windows 7. I use sample (eg replacebyclass.html). Steps to reproduce:

• clear all text in sample
• click Bold icon
• type some text
• try to click bold one more time -> in my case text is bold forever (I can't unbold it)

When I click enter I can make 'unbold' text but only once in new line. I tested this issue to version 3.5 (not below) .

Not sure if we can actually do anything with it.

The following code (without the opening tag to start the unordered list):

<p>
	Text above list</p>

	<li>
		One</li>
	<li>
		Two</li>
	<li>
		Three</li>
</ul>
<p>
	Text below list</p>
<p>
	Another paragraph below</p>

is changed into:

<p>
	Text above list</p>
<p>
	Text below list</p>
<p>
	Another paragraph below</p>
<ul>
	<li>
		One</li>
	<li>
		Two</li>
	<li>
		Three</li>
</ul>

• Load replacebyclass sample, clear all contents
• Select font size "12"
• Select font size "16"
• Type some text
• Result:

  <span style="font-size: 12px;"><span style="font-size: 16px;">Some text</span></span>
  

• Expected result:

  <span style="font-size: 16px;">Some text</span>
  

Note: it is a bit different case than the one reported in #2203

Exactly the same behaviour like #7368. Tested on Opera 11.01 Win 7. I have tested this in versions 3.5.2, 3.5.1 & 3.5 and there was all ok.

• Open the sample.
• Click on the list button.
• Click outside the editor chrome.
• Click the new page button.

The selectionChange event is not being fired (visually: toolbar is not refreshed).

Paste the following code:

<form action="formAct" id="myid" method="get" name="form">
	<select name="myselect" size="1"><option selected="selected" value="val1">opt1</option><option value="val2">opt2</option><option value="val3">opt3</option></select></form>
<p>
	<select name="myselect1" size="1"><option selected="selected" value="val1">opt1</option><option value="val2">opt2</option><option value="val3">opt3</option></select></p>

1. Place cursor after select (any select) and press enter.

Result: New list with two elements is created.

2. Now place cursor after second select and press enter

Result: New list with one element is created.

You can create as many selects as there are options in the first one select. See attachment.

Tested on opera 11.01 Win 7. Shortcuts from ​http://docs.cksource.com/CKEditor_3.x/Users_Guide/Keyboard_Shortcuts in Accessibility section like (alt+0, alt+f10 , alt+f11 and others) doesn't work.

• Open a sample configured as enterMode BR, click "new page".
• Open the format combo and chose "heading 1".

Result: nothing happens.

From version 3.3.2 of CKEditor, when you create a form and than input filed, than input field is created with break line (br) element above it (see code below). This prevents input from being deleted.

{{{<form action="formAct" id="myid" method="get" name="form">

<br /> <input name="opt1" type="radio" value="opt1Val" />&nbsp;</form>}}}

I have a page with 2 instances of CKEditor and I'm using the MediaEmbed (​http://www.fluidbyte.net/embed-youtube-vimeo-etc-into-ckeditor) plugin to add video to the content. I add the MediaEmbed button to the tool bar of the first instance only, but when trying to add media to the content, it gets inserted in the second instance. Debugging reveals that inside the plugin the "editor" variable which is passed to the plugin is the wrong (second) instance and the "currentInstance" property of the CKEDITOR object is null.

While using the method CKEDIOTR.setData end of the line <br> element gets removed in firefox.

config.enterMode = CKEDITOR.ENTER_BR, Firefox 3.6.13, Linux OS.

I've the following code:

<div class="main-holder main-holder-2">

HEADER
TEXT

</div>

When i now select the HEADER and press H3 style the code gets:

<div class="main-holder main-holder-2">

</div>
<h3 class="main-holder main-holder-2">HEADER</h3>
<div class="main-holder main-holder-2">
TEXT
</div>

But that's def. wrong / bad it should be:

<div class="main-holder main-holder-2">
<h3>HEADER</h3>
TEXT
</div>

Greets Stefan

If i click the SOURCE-botton and insert <p style="color: red;>Red</p> (missing the last ") and then click SOURCE again to go back to normal edit-view; CK will go grey and stop working.

It's a case that only happens to me with IE7 (with Firefox, IE8 and Chrome works well). I use Ckeditor 3.5.2.

If I edit a link for the second time, I get a javascript error referencing a loss of focus. Same happens if I edit an image a second time. The javascript error is the same, but on another line. Mistakes are the file "_source / plugins / dialogui / plugin.js", lines 1039 and 1021:

Line 1039

// GECKO BUG: setTimeout() is needed to workaround invisible selections.
setTimeout( function()
{
     var e = me.getInputElement();
          if ( e )
	  {
	       e.$.focus(); //************* ERROR ****************//
	       e.$.select();
	  }
}, 0 );

Line 1021

// GECKO BUG: setTimeout() is needed to workaround invisible selections.
setTimeout( function()
{
     var element = me.getInputElement();
     element && element.$.focus(); //************* ERROR ****************//
}, 0 );

This error is not limited only to enter a second time in the editing of an image or link, but also occurs on any button on the display an editing window. If this is the second time the page stop working in the background (no javascript error, but inaccessible), see the special characters button or the smiley button for example.

1. Open replace by code sample.
2. Place the cursor before word "This"
3. Create number or bullet list
4. The cursor should be still before word "This". If it's not, place it again in front
5. Press enter

Result: You will have two-item list, but you won't be able to get to empty element with arrow keys. It's possible only with mouse

Same thing happens, if you create this element in the middle of the list.

Using setData does not fire the selectionChange event. It could be confirmed using the api sample.

To reproduce the defect:

1. Open CK Editor and insert a table with default values.

2. Keep cursor in one of table cells, and open context menu.

3. Delete the current Row by slecting Row - > Delete Rows.

Expected Result:

Current Table Row is deleted with out showing any errors.

Actual Result:

Current Table Row is deleted but an Error is shown in status bar and the Error is Invalid argument

In Internet Explorer 8 positioning the cursor on the beginning of a non empty paragraph and clicking on a basic style button (bold or italic) does not toggle the style button in the toolbar. The style is applied and when typing you will see the style in the text area. The problem only arises in non-empty paragraphs.

To reproduce the defect:

1. Copy and paste a table from the attached word doc using paste from word dialog.

2. keep cursor in one of table cells and open Table Properties dialog by clicking on Table Properties section in context menu.

3. Press OK button on the dialog with out changing any properties.

Expected Result:

Table Properties dialog is closed, and table remains the same.

Actual Result:

Table Properties dialog is closed, but a new empty table with 3 rows and 2 cols is added.

4. Open Table Properties dialog again from context menu, and try to click OK Button to close the dialog.

Expected Result:

Table Properties dialog is closed, and table remains the same.

Actual Result:

Table Properties dialog not closed and we it will show an error on status bar and Error is Invalid argument

I get "getFirst() is null or not an object" error by the following steps:

1. Open ​http://ckeditor.com/demo/ on IE8
2. Delete all of the stuff in the editor
3. Type Ctrl+A
4. Type Enter

Happens in IE6 & IE7

• Load ​rev 6478 sample
• Press "New Page"
• Switch to source
• Paste the following code:

  <p>
  	&nbsp;</p>
  <table border="1" cellpadding="0" cellspacing="0">
  	<tbody>
  		<tr>
  			<td style="width: 197px">
  				<p>
  					Ewrew</p>
  			</td>
  			<td style="width: 197px">
  				<p>
  					435</p>
  			</td>
  			<td style="width: 197px">
  				<p>
  					43543543</p>
  			</td>
  		</tr>
  		<tr>
  			<td style="width: 197px">
  				<p>
  					Werw</p>
  			</td>
  			<td style="width: 197px">
  				<p>
  					435</p>
  			</td>
  			<td style="width: 197px">
  				<p>
  					click here</p>
  			</td>
  		</tr>
  		<tr>
  			<td style="width: 197px">
  				<p>
  					435</p>
  			</td>
  			<td style="width: 197px">
  				<p>
  					435435</p>
  			</td>
  			<td style="width: 197px">
  				<p>
  					ertretretre</p>
  			</td>
  		</tr>
  	</tbody>
  </table>
  

• Switch to wysiwyg
• Click with RBM on cell with value "click here" (DON'T CLICK WITH LBM) and choose cell -> cell properties...
• Result: the error is displayed:

'null' is null or not an object

 function getSelectedCells( selection )
 {
      // Walker will try to split text nodes, which will make the current selection
      // invalid. So save bookmarks before doing anything.
     var bookmarks = selection.createBookmarks(); // <-------

     tabletools/plugin.js, line 14

We have a two step validation/cleaning process for HTML. When the user presses our save button, we call 'updateElement' on the editor, send the data to our server, clean and return it, then call 'setData' on the editor. If the user disables SCAYT before saving it becomes enabled after the 'setData' call.

Steps to reproduce:

1. Write some data in the editor
   
2. Disable the SCAYT plugin via the toolbar button
   
3. Call 'updateElement' on the editor
   
4. Call 'setData' on the editor (doesn't matter what data you use)
   
5. SCAYT plugin is enabled

Expected result:

1. Write some data in the editor
   
2. Disable the SCAYT plugin via the toolbar button
   
3. Call 'updateElement' on the editor
   
4. Call 'setData' on the editor (doesn't matter what data you use)
   
5. SCAYT plugin is disabled

When the parent container changes style from display:none to display:block, the cursor is not visible and if you press backspace, ALL previous content is deleted.

I created a very simple html to show the problem, to reproduce:

-extract ckeditor zip on any directory -put the attached file "sample.html" just before the ckeditor folder example:

/sample.html /ckeditor/(ckeditor.js and all other ck files)

-open sample.html -Write something in the editor -Press the "hide Editor" button -Press the "show Editor" button *notice the cursor is lost, and you can't know your current position* -Write something else -Press backspace key *All content is lost

With a small window width (300px) some toolbar group is not wrapping onto the new line, Firefox is the only browser that works fine in this sense.

As I understand you want to have read-only version of CKEditor. Currently we are working on a new feature that might be what you are looking for. Please refer to #1376 for more details.

If that is something that suits your needs than maybe you could participate in that discussion and give us your feedback on how you think this should work?

Waiting for your comments.

After moving an image inside of editing area (e.g. a smiley), a meta tag is added to the source, even if CKEditor is not running in "Full Page" mode.

• Load replacebyclass sample
• Insert a smiley at the end of first line
• Move it with a mouse to the beginning
• Click "Source" button
• Result: a meta tag is added to the source

  <p>
  	<meta content="text/html;charset=UTF-8" http-equiv="Content-Type" />
  </p>
  

• Note: when switching to source mode and back, an empty paragraph is being added constantly.

There is no JS error in the error console.

Caused by [6453]

Steps to reproduce:

1. Minimal document (HTML):
   

   <ul>
   	<li>
   		a</li>
   </ul>
   

2. Place the cursor before "a".
   
3. Press backspace.

Expected behaviour:

The list should be removed and the document should contain "a".

Actual behaviour:

Nothing happens - the list remains in the document.

Tested on:

Windows 7, 64-bit, FF 3.6.16, FF4, Chrome 10.0.648.204, Opera 11.01

Additional notes:

• works fine on IE 9.0.8112.16421
• the list can have arbitrary number of items to simulate the bug
• using toolbar icon for outdenting works fine

Using Chrome, go to source and paste:

<ul>
	<li>
		This is some <strong>sample text</strong>. You are using <a href="http://ckeditor.com/">CKEditor. 
</ul>

Expected:

<ul>
	<li>
		This is some <strong>sample text</strong>. You are using <a href="http://ckeditor.com/">CKEditor.</a></li>
</ul>

Result:

<ul>
	<li>
		This is some <strong>sample text</strong>. You are using <a href="http://ckeditor.com/">CKEditor. </a></li>
</ul>
<p>
	<a href="http://ckeditor.com/"> </a></p>

Not a recent regression. Didn't try Safari.

I'm not sure if this is a design choice or a bug. Selecting text and pressing the enter key does not clear text.

Steps to reproduce:

1. Type a few lines of text within the editor
   
2. Select the text
   
3. Press the enter key
   

Browsers tested, IE8/9, Chrome

When you want to add a link to an anchor, the down arrow button of the empty drop-down list By Anchor Id contains a minor visual glitch: it does not extend to the bottom border of the input field.

After the list is populated (note that it is now only possible to add an anchor ID by editing the source), the button is displayed correctly.

Please note that in earlier versions (CKEditor 3 up till 3.5.2) the glitch was a bit different as the whole input field had a smaller height.

Steps to reproduce

1. Create an anchor in the document, by default it will only contain the name attribute and no id.
2. Open the Link dialog window, select Link to anchor in the text as the Link Type.

Reported in ​http://cksource.com/forums/viewtopic.php?t=21915

editor.Toolbar="CHSummaryEditor"

is changed by CKEditor control into

"toolbar" : [["CHSummaryEditor"]]

The following customisation works in page without masterpage but it does not work when the page is in a masterpage. any help?:

<ckeditor:ckeditorcontrol ID="CKEDetails" runat="server" 
Toolbar="Source|-|NewPage|Cut|Copy|Paste|PasteText|PasteFromWord|-|Bold|Italic|Underline|Strike
|-|Subscript|Superscript|NumberedList|BulletedList|-|Outdent|Indent|-|JustifyLeft|JustifyCenter|JustifyRight|JustifyBlock|-|Link|Unlink|-|Table|HorizontalRule|Styles|Format|Font|FontSize|TextColor" 
Width="500px"
></ckeditor:ckeditorcontrol>

Reported in ​http://cksource.com/forums/viewtopic.php?t=21839

I've just installed CKEditor.NET and i'm using asp.NET with VB. The Editor is showing and working fine however all my attempt to configure it are not taken in account.

More information is available in the original bug report: ​http://cksource.com/forums/viewtopic.php?t=21780

PHP works in attributes, but when open dialog it shows

{cke_protcted_0} 

Example: make a form with

action="<?php echo $_SERVER['REQUEST_URI']; ?>"

source is right, forms dialog value shows protected.

Bug or new feature request, I don't know. Please change it.

I write a plugin to show placeholder icons for js and php code. For that I use

return new CKEDITOR.htmlParser.element( 'img', attributes );

This works with js code. But php source is rewrite from

<?PHP echo 'test'; ?>

to

<?php echo=""></?php>

I can post more code when needed.