Custom Query

Hi,

You can test it right here : ​http://nightly.ckeditor.com/4908/_samples/sharedspaces.html

When you click on the little arrow to collapse the shared toolbar, the editor height changes, as if the toolbar height was still calculated in the editor size.

The More Colors dialog that reappeared in CKEditor 3.1 has 8 black color buttons filling the last slots in the bottom row of color buttons. It would be nice if these "filler" buttons could be overruled by custom color buttons defined in the custom configuration.

While it's already possible to add custom colors to the default color button menu via CKEDITOR.config.colorButton_colors, one or two custom color buttons look out of place there.

I would like to add a couple of colors used in our company style as buttons in the More Colors menu, allowing me to specify the color and its name (i.e. Company green) to use in the title text.

The current behavior for the Print command is printing the contents as we see them in the editor, not as they will be in the final output. So, anchors, flash placeholders and anything else that is editor specific will be visible in the print.
We should change the print plugin to instead print the outputted document (just like preview).

Create an editor instance that spans the with of the screen.
Create a table with 100% width.
Move the mouse to the right hand side of the editor and right click on the table.
Observe the context menu correctly displays as fixed in #4594.
Open a sub-menu, e.g. Cell Properties.
Observe that the sub-menu covers the main context menu.

There are some things that should be included in the default stylesSet sample so it's more useful to learn how to use it:

• Styles for the div container: #4973
• How to apply classes

The default stylesheet might need also some adjustments to reflect those classes.

This is due to the patch in #4973, but it was a long talk that would distract from the real patch.

The patch in #4972 (v2) removes CKEDITOR.loadStylesSet and CKEDITOR.addStylesSet, so when people tries to upgrade they will get an error if they have used them.

I think that most of the people didn't know anything about CKEDITOR.loadStylesSet unless they had to fight with the lack of CKEDITOR.stylesSet ​as I did, but I think that there's lots of people using their own styles with the addStylesSet call. If they don't change the code then it will fail and they have to find out the reason.

So my question is: how do we approach backwards compatibility?

We can leave the code as is and they have to read the release notes to notice the reason of the problem. This can be more complex if they are upgrading for example from 3.0 to 3.3 with hundreds of bugs in the mean time.

We can add bold and red statements in the what's new to make it clearer, but they still have to read the docs.

We can provide also a "compatibility" plugin that it's used just to provide compatibility with older apis. This plugin could define just the old functions as mapping to the new APIs and launch a silent warning in the console (if it exists)

Something along these lines:

CKEDITOR.addStylesSet = function( name, styles ) 
{ 
    if (window.console)
        window.console("The CKEDITOR.addStylesSet function has been deprecated. Please use CKEDITOR.stylesSet.add. Read... for further info");
    CKEDITOR.stylesSet.add(name, styles)
};

This way the upgrade is easier, there's a very little overhead and people can remove the plugin when they know that everything is working OK.

Recreate Bug

Use Firefox (I tested on both Windows FF 3.5.7 and Linux FF 3.5.5) and the ​online demo and perform the following steps.

Setup

1. Click in the editor and remove all content i.e. <CTRL> + <A> then <BKSP>
2. Type or paste in: foo bar baz
3. Double-click bar and make it bold. Notice That FF on Linux selects only the word bar on double-click. Chrome and IE8 on Windows select the word plus one trailing space, which also gets the bold style.

Test 1

1. Place the cursor anywhere in the word baz then move left with the arrow key to the end of the word bar. Notice that the B button is NOT active and the status bar displays only body p.
2. Use the arrow keys to move left then right one character, returning to the same position. Notice that the B button is active and the status bar displays body p strong.
3. Use the arrow keys to move right then left one character, again returning to the same position. Notice that the B button is NOT active and the status bar displays only body p.

This inconsistency is confusing my users and is likely related to the odd behave I will describe next.

Test 2

1. Again, place the cursor to anywhere in the word baz then move left with the arrow key to the end of the word bar.
2. Backspace to the end of the word foo, completely deleting the word bar and the space between foo and bar.
3. Type <space>bar to replace the original text. Notice that the text is bold again. But rather than a STRONG tag we have a SPAN with inline style.

Unless you delete both foo and bar you cannot get rid of the littered inline style. These tests exhibit the same behavior using italic and underline, and indeed my own custom styles. This behavior is not exhibited using Chrome nor IE8. I have tested this in 3.01, 3.02 and 3.1. All exhibit the bug in FF only.

There is a bug with IE in standards mode when trying to click to the right of a control selectable node such as an image.

To replicate Set the HTML to:

<p>
	Line 1<br />
	Line 2<img src="http://www.google.com/intl/en_ALL/images/logo.gif" /></p>

I have attached a patch which fixes this problem. I dont know if it is implemented up to your standards though.

Some things that are bad about my patch are:

• Its using the CK dom range inside selection (seems a bit cyclic)
• You cannot mouse down and drag to select more
  
• I dont really understand why there are multiple selections / ranges so I have just used "getRanges()[0]"
  
• Its using setTimeout() because the selection is not ready before the event
  
• I am using a constant of 20x20 pixels to check the mouse offset - perhaps a character size calculation is needed?

There may be some other ways to fix this. Another idea is that we could insert a whitespace text node / span at the end of the block before the end of the mousedown event.

Cheers,
Scott
​http://www.synergy8.com/

When ckeditor shares a DIV with an object that has a float: left property the editor doesn't render properly.

See the attached HTML file to replicate this issue. Just copy the attached file into your samples directory and view it.

I believe the issue is due to the following CSS:

.cke_skin_kama
{
       display: block;
}

in the mainui.css file. It seems to render correctly if the css is changed to

display: inline-block;

This causes the editor to lose focus and causes the editor to lose focus, scroll somewhere offscreen and remain inaccessible due to the scrollbar having been disabled through javascript. Pressing tab will eventually circle you back around to get the editor back in focus, but not a solution by any means.

First, create a few bullet point lists items with nested lists. Next try to select a nested list and start typing to overwrite the text. Notice how the text continues at the parent list level and the multiple bullet points on the same line. It is really hard to recover from this, without deleting and recreating the lists. It seems to only be an issue in Firefox and IE6,7,8. Looking at the source, it seems that there is an orphaned <ul> tag under an empty <li> tag.

For example, the source below will cause the duplicate bullet points:

<ul>

<li>first level</li> <li>

<ul>

<li>start of 2nd level</li>

</ul>

</li>

</ul>

It is possible to get into this situation using the above steps to highlight and overwrite text.

I quote from the manual : "In-page settings can be passed to any of the editor instance creation functions, namely CKEDITOR.replace and CKEDITOR.appendTo."

​http://docs.cksource.com/CKEditor_3.x/Developers_Guide/Setting_Configurations

I use :

function ckeditorInit() {

    if (editor) /* If we already have an editor, let's destroy it first. */
    editor.destroy(true);
	CKEDITOR.replaceAll( 'rich-text', {
	    customConfig : '',
	    enterMode : CKEDITOR.ENTER_BR,
	    shiftEnterMode : CKEDITOR.ENTER_P, // Paragraphs are now made by pressing shift and enter together
	    skin : 'office2003', editor */
	    toolbar : [
		['Preview', '-', 'Cut','Copy','Paste', 'Find', 'Undo','Redo', '-','SelectAll','RemoveFormat', 'Table','HorizontalRule','PageBreak', 'ShowBlocks', '-', 'Templates', '-', 'Styles','Format', '-', 'Font','FontSize'],
		'/',
		['Bold','Italic','Underline','Strike', 'SpecialChar', 'TextColor', '-','SpellChecker', 'Scayt', '-', 'NumberedList','BulletedList','-','Outdent','Indent','Blockquote', '-', 'JustifyRight','JustifyCenter','JustifyLeft', 'JustifyBlock', '-', 'Link','Unlink','Anchor', '-', 'Image','Flash', 'Smiley', 'BGColor', '-', 'NewPage', '-', 'Source', '-', 'Maximize'],
	    ]
	}); /* End CKEDITOR replaceAll rich-text */


};

window.onload = ckeditorInit;

All the textareas are replaced with ckeditor but the config is ignored.

So, In-page settings cannot be passed to any of the editor instance creation functions. Just some of them and not CKEDITOR.replaceAll.

Please sort this out. It's what most people would expect as "Normal" Behaviour.

Thank you.

I have not found a good way to adjust the alignment of the radio buttons. The radio buttons is outputted as table cells in a single table row, that makes it impossible to use css to display them in an vertical list (Internet Explorer).

It would be great to have the following options for the radio element:

• Vertical or horizontal display of radio buttons
• Whether to display the labels above, below, in front of or behind the radio buttons.

The current definition of getContentElement, getValueOf and setValueOf requires both the pageId as well as the elementId, that means that each element is bound to the page where it has been defined, but it would allow greater flexibility for customization if those functions didn't require the pageId, store all the elements in a dialog in a new collection for example this._.allContents[elementId]

This would allow to just move one element from one page to another in the contents definition, with no need to adjust anything in the javascript code.

Are there any drawbacks?

When navigating a context menu with the keyboard, the ARROW-RIGHT executes the selected entry, just like ENTER. It should do nothing, being used just to open an eventual second level menu.

There is no visual behavior for checkboxes in dialogs when tabbing over them. We have the impression that the focus got lost.

JAWS is not reading out the accessibility help legend in dialog, it should be in VPC mode when focus reaches the legend.

Load 'Shared toolbars' sample from latest nightly.

The first two editors editor area overlaps the border of the editor.
If you specify a very small width to the editor (like 200), the width of the editor area will be about 320px (only the border gets the specified size).
(Chrome 5.0.322.2)

If the instance is created replacing a textarea, try to make it match those dimensions by default. If the config has some dimensions set, then of course, use that, but if it's missing, then get the size of the textarea like requested in ​http://cksource.com/forums/viewtopic.php?f=11&t=17682

Text and Table Template uses "float: right" on the table, this then cannot be changed with the table properties dialog.

To reproduce

• Open an empty CKEditor instance and insert 2 page breakes.
  
• SELECT (drug the selection - not by clicking on a page break sign) the 2 page breakes signs (make sure you are not selecting any other thing - including text nodes, this is important).
  
• Insert an element on it (HR for example).

JS error is thrown (getFirst() is empty or not an object).

CKeditor's flash plugin accepts youtube URLs, if they point directly to the flash file (youtube.com/v/XXXXXXXX URLs); youtube's interface gives URLs like youtube.com/watch?v=XXXXXXXX.

This patch does a string replace on the interface so that people can paste youtube page urls.

Supplied code reeks of duct tape, but it gets the job done; I don't have 40 hours to spend learning CKeditor internals (it's already perfect, anyway!)

In api_dialog sample, there is too much space before My Custom Field in the customized link dialog. This may be related to a generic issue that needs investigation.

This behavior is confirmed in all browsers but IE.

1. Load the following HTML:

<p>This is <b>some sample</b> text.</p>

2. Select "This is some".
3. Click the Bold button.

Current results:

<p><strong>This is </strong><b><strong>some</strong> sample</b> text.</p>

Expected results:

<p><strong>This is some</strong><b> sample</b> text.</p>

With IE8 with Compatibility mode:

1. Load the following HTML:

<p>Line 1</p>
<p>Line 2</p>
<p>Line 3</p>
<p>Line 4</p>

2. Select Lines 2 and 3.

3. Click the Div Container button.

4. In the Advanced tab, set the Style field to "border: solid red".

5. Confirm the dialog.

Note that there will be extra (unwanted) space at the top padding of the created div. When moving back and forth to source, that space disappears.

If I put a background image behind my H1 tags in the editor css, that background image shows in the headings when you click the styles drop down in the editor. I'll attach a screenshot to show what I mean.

To reproduce

• Open a new editor and go to Source mode
• Write this text: <blockquote>Text</blockquote> and place the caret somewhere inside it
• Click on the blockquote button

The blockquote is not removed.

To reproduce

• Open a new editor
• Click on the B button and type some text.
• Place the caret in the beginning of the text
• Click again on the B button. Notice that the the button is not unselected.
• Type some text. notice that noe the button is unselected.

When printing the content inside of CKEditor (using the "Print" button), page breaks apart from breaking pages correctly, are also printed.

This is a similar problem to issue with printing tables without a border (#731).

In some cases, CKEditor uses an alert to display messages, i.e. invalid . but, in some cases, the developer would like to use his own UI to display these messages. so, i think it can be replaced by an internal function that the developer can overwrite, or make an optional config entry with a callback that will overwrite the original alert.

<textareas> are being handled like normal HTML tags, when they should be getting the same treatment as <pre> tags.

I've done a quick look and from what I see, the offending file would be fragment.js: -Line 152: Original: "&& element.name != 'pre'" Possible replacement: " && element.name != 'pre' && element.name != 'textarea'"

-Line 191: Original: "else if ( tagName == 'pre' )"

-Line 332: Original: "if ( currentNode.name == 'pre' )"

That should theoretically fix it. I haven't worked much with it, so this is all untested, but it'd be nice if someone who has more of an idea how to work this could make a proper patch.

Browser: IE8, OS: WinXP

Occurs in demo page ( 'Little Red Riding Hood' )

When you use the up/down arrow keys to navigate through the Text, the caret will stop at the end of the first <p>-Paragraph.

The next step would be to jump into the table on the right.

Same happens at the end of the positioned table back to the second Paragraph, and also on your way back up.

When linking an image tag which is wrapped by span tag(s) the link is placed around the span tag(s) rather than the img tag. Unless the span tag has text inside it as well as the img tag, in this case the img tag will only be linked.

Example html: <a href="​http://www.google.com"><span style="font-size: 11px;"><img alt="wink" src="/ckeditor%202/plugins/smiley/images/wink_smile.gif" title="wink" /></span></a>

Expected HTML: <span style="font-size: 11px;"><a href="​http://www.google.com"><img alt="wink" src="/ckeditor%202/plugins/smiley/images/wink_smile.gif" title="wink" /></a></span>

OS: Mac OS X 10.6.2 Browser: Firefox 3.5.8

Follow up of #4555

It's not the typical "memory leak" problem present in the browsers, but it's easy to store data in a way that it isn't released after one instance is destroyed in an AJAX way, and that means more and more memory use, so it's in the end a memory leak.

In the previous bug we have cleaned up the main codeline, but there are other parts that will have to be reviewed and instead of trying to dig into every plugin we must try to find an automated way that help us diagnose where are the problems.

I can confirm that the new version fixes the img align= problem. Would it be possible/difficult to add support for replacing the style= with a CSS class for more consistent design (similar to the justifyClasses and indentClasses)? Maybe call it imageAlignClasses...

Like other form elements, a CKEditor instance should provide a way to reset back to the original contents.

And then we can hook the form onReset event to act like other elements ​http://cksource.com/forums/viewtopic.php?p=46129

To reproduce

• Open the demo or a sample in the SVN.
• Insert any form element (e.g. input, select etc.).
• Move the caret to the sides of the element.

in IE, the caret won't move back to the right. in FF, it won't move to both sides.

Related to #4711 but this is weirder: Using IE8 set this source:

<pre>Hello world!
Bla, bla, bla
</pre>

Go to the end of the first line and press enter several times, this is what it generates:

<pre>
Hello world!

B
l
a
,
 
b
l
a
,
 bla
</pre>

IE6 and IE7 seem to work correctly.

Since there is no 'OK' button in the search and replace dialog, the enter key is useless. therefore, when in search tab, the enter key should trigger the search, and in replace dialog it should trigger the replace.

When <font> tag is used in the HTML content, the font name combo displays invalid font name (Arial) instead of current font.

Steps to reproduce

• Open ​http://ckeditor.com/demo.
• Paste the following in the source mode:

  <p>
  	sdfsd sd fsd fsd f</p>
  <p>
  	<font face="Courier" size="2">test</font></p>
  

• Switch to wysiwyg mode.
• Click on the second paragraph.
• Result: the font name combo shows "Arial" instead of "Courier", which is confusing.

Confirmed in IE8 and FF 3.6.

<br> tag is lost when the block element after line break is empty.

Steps to reproduce

• Set in CKEditor configuration ignoreEmptyParagraph to false
• Paste the following in source mode:

  <div>
  	First line</div>
  <br />
  <div>
  </div>
  

• Switch to wysiwyg mode, result:

  <div>
  	First line</div>
  <div>
  	&nbsp;</div>
  

Same things happens when <p> tag is used instead of <div>.

When editor is running inside an ​modeless dialog of IE, clicking on all toolbar commands will cause the focus transferring from the dialog back to the main window, which brings a major problem for continual editing experience. See attached sample page to reproduce.

Environment

All IE versions

Reproducing Procedures

1. Load any of the sample page and fill the editor with the following contents:

   test tab order
   <hr />
   

2. Place the cursor at the beginning of first line and press 'Tab' key.

• Expected Result: The editor blurs and focus into the element in next tabIndex.
• Actual Result: The focus goes into the <hr> element.

Environment

IE8 standards, with enterMode = BR.

Reproducing Procedures

1. Load any of the sample page and fill the editor with the following contents:

   <table>
   	<tbody>
   		<tr>
   			<td>
   				text</td>
   		</tr>
   	</tbody>
   </table>
   

2. Click into the spaces after the table at the end of document.

• Expected Result: It's possible to have the cursor blinking there and start typing.
• Actual Result: Cursor stays as it is.

Reproducing Procedures

1. Load any of the sample page and fill the editor with the following contents and selection:

   <table>
   	<tbody>
   		<tr>
   			<td>
   				^text</td>
   		</tr>
   	</tbody>
   </table>
   

2. Open 'Div Container' dialog and insert a default div.
3. Right click to open context menu;

• Expected Result: Options of 'remove' and 'edit' the div appear in menu.
• Actual Result: There's no optoins to modify the inserted div.

This ​screencast in IE demonstrates that Smiley icons are displaying in a fly-in way on slow connection thus break the layout of dialog.

It's not able to move cursor after the following elements at the end of document:

• ul/ol
• form
• blockquote

The unexitable elements list should expand to consider these elements.

1. Open Ajax Sample.

2. Click the Replace icon on the editor toolbar.

3. Attempt to find & replace some text within the page content.

The user should have the option to find text before replacing it, as he/she won't always want to replace a particular occurrence of some text.

Note: Text can be found through the Find tab & replaced specifically through the Replace tab but this function should & could be simplified by adding a Find button to the Replace tab.

My clients find it confusing when the styles, listed in the "Styles" plugin, are grouped by style type.

I created a patch for "stylescombo\plugin.js"

Environment

All IE versions.

Reproducing Procedures

1. Open any of the sample page;
2. Open 'Styles' combo and click on 'Inline Quatation" and start typing a few characters;
3. Select 'Inline Quatation" again to close the style, then continue typing;

• Actual Result: The closed half of quatation is not displayed.

Environment

IE6/7

Reproducing Procedures

1. Open any of the sample page;
2. Insert a textarea with all fields filled;
3. Switch to 'Source' mode and switch back;
4. Open 'Text Field' dialog through context menu;

• Actual Result: 'Columns' field value is missing.

Reproducing Procedures

1. Open any of the sample page, load the document with the following content and selection;

   <p> [some sample] text </p>
   

2. Apply the 'Background-color' style with 'red' value.
3. Make the following selection based on the result document.

   <p><span style="background-color: rgb(255, 0, 0);">some [sample</span> text]</p>
   

4. Apply the 'Font-size' style with value '72pt';
   • Actual Result:
     1. Firefox, IE8 and Safari:
     2. IE6/7:

• Expected Result:
  

Browsers parse the contents of the style attribute and do strange things with its contents, so we should protect like we do for href and src so it's modified only when the user request it, and not by the browser.

When pasting HTML that contains lots of nested tags, IE6 throws an error: "stack overflow at line: 27".

Steps to reproduce

• Open _samples/api.html
• Copy content from attached file
• Paste it into the textarea
• Press the "Insert HTML" button

Steps to reproduce

Create new dialog without set id attribute for element or open one old dialog and remove id attribute of element:

CKEDITOR.dialog.add( 'bulletedListStyle', function( editor )
	{
		return {
			title : editor.lang.list.numberedTitle,
			minWidth : 300,
			minHeight : 50,
			contents :
			[
				{
					elements :
					[
						{
							type : 'hbox',
							widths : [ '25%', '75%' ],
							children :
							[
								{
									label: editor.lang.list.start,
									type: 'text',
									setup : function( element )
									{
										var value = element.getAttribute( 'start' ) || 1;
										value && this.setValue( value );
									},
									commit : function( element )
									{
										element.setAttribute( 'start', this.getValue() );
									}
								},
								{
									type : 'select',
									label : editor.lang.list.type,
									width: '100%',
									items :
									[
										[ editor.lang.list.style_1 , '1' ],
										[ editor.lang.list.style_a , 'a' ],
										[ editor.lang.list.style_A , 'A' ],
										[ editor.lang.list.style_i , 'i' ],
										[ editor.lang.list.style_I , 'I' ]
									],
									setup : function( element )
									{
										var value = element.getAttribute( 'type' ) || '1';
										this.setValue( value );
									},
									commit : function( element )
									{
										element.setAttribute( 'type', this.getValue() );
									}
								}
							]
						}
					]
				}
			],
			onShow: function() 
			{
				var editor = this.getParentEditor(),
					element = getListElement( editor, 'ol' );

				element && this.setupContent( element );
			},
			onOk: function()
			{
				var editor = this.getParentEditor(),
					element = getListElement( editor, 'ol' );
				
				element && this.commitContent( element );
			}
		}
	});

• Expected: all 'commit' and 'setup' method of element will execute
• Result: only the last element have that behavior

While reading #4358, I was wondering about a way to easly switch between the two list types. With the new dialog in #4358 it should be enough to switch between the tabs and pressing OK.

Context menu may became partially invisible (unusable) if it is positioned above the cursor (i.e. there is no space for context menu to be displayed below cursor).

Can be easily reproduced with SCAYT enabled - when there are SCAYT items in the menu, also may be reproduced with table menus.

Current behavior: if there is no room for the menu under the cursor it is displayed above the cursor position, and if there is no room for the whole menu above cursor position it becomes partially invisible

Expected behavior: if there is no room for the menu under cursor position then the room above cursor position should be evaluated - if there is a room for the menu - menu should be displayed above cursor position, if there is no room for the menu, it should be displayed strating from the top of the document visible part (i.e. top: 0).

Steps to reproduce

• Open the demo.
• Put the caret inside one of the table's cells.
• Click 'body' in the elements path.

Notice not only the body is selected, but also 'h1' and 'image'.
Tested with the demo and the current trunk with IE 8.

Like MS Word, the first option in the Styles combo should be "Remove format" as it's clearly related to the Styles so it's easier to find for new users, then people can opt to remove the button from the toolbar.

It would be nice if I can view unprintable characters of used formatting in CKeditor. Is it possible to make button for display and hide such unprintable characters?

When someone copies a hex value for a custom color without initial hash character (e.g. cc31e2) and then use it as a custom color, CKEditor will ignore this color and insert an empty <span> tag.

Confirmed in IE8 and FF 3.6.

<p>
	Sample <span>foo</span> bar.</p>

Steps to reproduce

• Select some text
• Press "Text Color" button
• Press "More Color"
• in the right corner paste cc31e2 and press "Ok"

People usually don't know anything about html tags, they just want things to work, so as long as it's possible we should try to think like a new user and how the expect the editor to behave.

In order to end a PRE-formatted paragraph I think that it's easy to expect that just selecting "Normal" in the format dropdown should start a new paragraph and get out of the pre if the caret is at the start of a new line. This would work in a similar way to the Bold button: you press to start marking things as bold and when you press it again you end the effect.

1. Open 5606.html​ on FF.
2. Focus editor.
3. Switch to source mode.

<p><br />
<? $stuff='stuff'; ?></p>

All browsers autoparagraph protected source. FF additionally adds <br>.

Environment

IE, enterMode = CKEDITOR.ENTER_BR;

Reproducing Procedures

1. Open any of the sample page and click on 'New Page' to clear all contents;
2. Click on 'Insert Horizontal Line' 3 times to insert 3 <hr>;
3. Undo once to revert one <hr>
4. Strart typing some text
   • Actual Result: The inserted text appears after the first <hr>;
   • Expected Result: The inserted text appears after the second <hr>;

This issue is partly related to #5434. On a 64-bit server, the fix for #5434 works but on a 32-bit server a different error occurs.

The issue is that 32-bit IE does not appear to support IPv6 addresses correctly, especially when getting document.domain. IE truncates the ip address when it reaches the first colon ":" in the address. This causes "access denied" errors when new IFrames are opened by CKEditor.

On (32-bit) IE:

document.domain = "[fe80:"
window.location.hostname = "fe80::fe80:fe80:fe80:fe80" 

A work-around is required for this IE bug.

Follow up from #4231:

If the editor has been set so it can be resized only vertically, instead of having the grabber at the right-bottom corner it should be displayed at the bottom with a different picture. Same applies for horizontal-only resize.

Putting contentEditable="false" on table doesn't prevent the table content from been editable, this works in all other browsers.

CKeditor 3.2.1 : IE 7.x only - Unable to set font name and font size

I can reproduce the problem in CKeditor demo site:

• Blank the editor's textarea
• Type one word, for example "WORD"
• select the word typed previously and apply font name "arial" and immediately select font size "12"
• put the cursor at the end of "WORD"
• With the cursor at the end of the word, select font name "Tahoma" and immediately select font size 18

=> You will notice that the font is Arial again. The font name "Tahoma" disappeared when you select the font size, paragraph format or styles.

This's a continuation of #5111, where it's not able to navigate the toolbar in JAWS with Arrow keys, it shouldn't be a problem if JAWS is in PC cursor mode.

Provide a plugin to detect desktop file dropping into editor, for those support browsers.
The detected file info could be used to upload the file immediately or perform any possible customization.

Edit: Yes that is most correct. Editor should have at least a hook for implementing drag&drop into editor. What I mean is that if editor is connected with file uploader (like CKFinder) it should prepare image information that uploader can use to upload file. After file is uploaded, uploader should send new image path and editor should apply it.

Environment

IE

Reproducing Procedures

1. Open any of the sample page and clear all content with 'New Page';
2. Create a empty bulleted list in place.
3. Open 'Format' combo and select the 'Normal' paragraph format.
   • Actual Result: There's an extra blank line above the paragraph created, but as soon as we start typing, the empty line vanishes.

1. Load the following content in editor;

   <p>
   	wrongspell</p>
   

   • Expected Result: The word is red-marked in wysiwyg mode.
2. Open 'Replace' dialog and replace the word with "right spell", then close the dialog.
   • Expected Result: The red underline is removed.
   • Actual Result: The red underline is still in place.

1. Load the following content with selection in editor;

   <p>paragraph</p>
   <table>
   	<tbody>
   		<tr>
   			<td>
   				[cell]</td>
   		</tr>
   	</tbody>
   </table>
   

2. Open context menu on the selection, select 'Column' -> 'Delete Columns'.
   • Expect Result: The entire table is removed.
3. Click on 'Undo' once to revert the deletion.
   • Actual Result: the table is brought back but the cursor is not blinking inside the paragraph instead in the cell.
   • Expected Result: both content and selectoin should match the original state.

1. Load the following content with selection in editor;

   <table>
   	<tbody>
   		<tr>
   			<td>
   				some [mispelled] word</td>
   		</tr>
   	</tbody>
   </table>
   

2. Open context menu on the selection, make sure the selection overlaps exactly with the underline marker.
3. Click on either 'Delete Cell'/'Delete Column'/'Delete Row';
   • Actual Result: Error thrown and there's on deletion happens.

Reproducible in all browsers, error stack in Firefox:

row is undefined
http://ckeditor.t/projects/3.3.x/_source/plugins/tabletools/plugin.js
Line 181

Environment

IE, enterMode=BR

Reproducing Procedures

1. Load the following content and selection in editor;

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

2. Press 'Enter' key twice to move out of the list.
   • Actual Result: The new paragraph is not created and cursor is blinking at wrong place.
   • Expected Result: A new paragraph is created after the list.

fixlineends should handle special keywords, for example to not add a BOM character to a .js file or force different line endings.

For example in .js file we could add:

// FIXLINEENDS_NOBOM

to remove BOM from that file (.js files should have BOM character).

This is important as BOM is causing problems when creating the API documentation ([5526]).

Steps to reproduce

• Load a sample with enterMode br
• Click on one of the list icons.
• Create two list items and press ENTER twice to finish the list.

Notice that the caret is getting back to the last item, thus it is impossible to create sequential lists.

Version 3.3 added support for preserving heading structure when creating a list item. Would be nice to also preserve structure when creating list items from paragraphs.

In IE6 there is a 8MB memory leak every time an instance of CKEditor is created.

Confirmed using Process Explorer, after creating & destroying CKEditor 10 times (using AJAX sample), memory usage (private bytes) jumped from 9MB to 90MB (tested on IE6.0.3790.1830 @ Win2003/SP1, also reported by user using IE6 6.0.2900.2180).

I have attached a dump from IE Sieve.

Confirmed in FF 3.0.19.

When using CKEditor in fullPage mode, an error is thrown when <title> tag is not present.

Steps to reproduce:

• In Ajax sample add

  var config = {fullPage : true};
  

in createEditor() function.

• Open ajax sample, press "Create Editor".
• Press "Source" button or "Remove Editor".

Result:

element.children[0] is undefined
http://192.168.1.126/bugtest/ckeditor/_source/plugins/htmldataprocessor/plugin.js
Line 198

196  title : function( element )
197 {
198     element.children[ 0 ].value = element.attributes[ '_cke_title' ];
199 } 

Browser/Os: In Chrome on Ubuntu Linux. Bug does not occur in Firefox, not sure about other browsers/operating systems. Steps to reproduce: 1) Go into Full screen mode. 2) Open a dialog (seems like any dialog, but for sure the image dialog produces bug). 3) Exit full screen mode.

At this point, the editor will extend all the way to the right of the screen, instead of going back to the size it was originally.

It seems to me that if a dialog was opened for the first time before going into full screen mode, then the bug does not occur.

Checkbox and radiobutton do not get focus or context menu on FF 3.6.3 and are not editable because of this.

Fake element replacement would be feasable to fix this since elements have basically two appearances selected and unselected.

In #4056 I posted form_changes.zip which uses fake elements for this. Allthough changes are dated and should be rewritten I see no harm done in this approach.

I got this code from #4647:

1. Go to ckeditor.com/demo in IE8 and paste the following into the source:

<p style="width:200px"><img src="http://dev.fckeditor.net/chrome/site/logos.gif" /></p>

2. Right click the image to open the context menu or select the image and click the image button.
3. Image Properties option is not there or it loads the dialog with empty fields.

Note that the first time I right click the image I don't get the "Image Properties" option. However, if I then right-click on the image again OR if I left click the image and then click the Image button, I get the "Image Properties" option and the dialog loads correctly.

It's not able to block certain keystrokes in IE with 'CKEDITOR.config.blockedKeystrokes', e.g. F5.

In Google Chrome 5.0.375.70 (Windows 7) adding :first-letter style to content.css leads to incorect editor behavior.

If you set cursor to any position inside an existing paragraph and press a character, the character is inserted not in the cursor position but at the previous position.

If you try to set cursor at the beginning of an existing paragraph, it's set after the first letter instead. If you try to press backwards key to move the cursor via keyboard, the script hangs.

In IE 8 that works fine.

• Place CKEditor 3.3.1 inside a DIV or other element
• Editing works fine
• Hide the CKEditor by changing that enclosing DIV's "display" CSS properly to "none".
• Show the CKEditor again later by changing the CSS "display" back to "block".

The result is that the CKEditor works OK, but the cursor is never displayed. If you type ... you may see your results, but without any visible cursor.

You can get the cursor back by changing the CKEditor mode to "source" and then back to "wysiwyg". This is not a very user friendly Work Around.

Issue observed in FireFox 3.6 on Mac and Windows XP. Have not tried it in other browsers yet.

This issue did not exist with CKEditor 3.1.

Many customers and ourselves have noticed that CKEditor has a much slower response to keyboard input when SCAYT is enabled. The speed seems to be vary over an editing session.

Anecdotally, it seems to be worse if you are far geographically from the SCAYT server.

We would expect SCAYT to be as unobtrusive as possible. I.e. perhaps not doing anything until the user has paused in editing for at least a second or so, and not blocking on any AJAX calls.

This uses CKE 3.1.1 and the latest SCAYT back-end release from last week.

1. Set option CKEDITOR.config.fullpage="true" for the sample page.
   

2. Load the sample page.
   

3. Click on the "Source" button to view source HTML code
   

--> Body doesn't contain any SCAYT classes.

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

5. Click on the "Save" icon to view the result
   

Actual result:

<body class="scayt-enabled">

Expected result:

<body>

TinyMCE and other enterprise wiki wysiwyg editors by default ship keybindings for changing a line into headline (h1, h2, h3 are accessed using CTRL+1, CTRL+2, ...).

In order to make CKEditor work the same way I had to go into the plugins I wanted to add keybindings for, and add this line of code

editor.addCommand(tag, new CKEDITOR.styleCommand(style));

to enable h1..h6 and CTRL+0 for p (turn headline back to normal paragraph).

1. Go to CK editor demo page.
   
2. Clear editor content
   
3. Type a word
   
4. Select and copy the word
   
5. Paste it several times on the same line.
   
6. Hit Ctrl-Z to undo.
   
7. Javascript errors should be generated. Probably related to range code.
   

Tested on Windows, Firefox 3.6.3
Editor version : 3.3.1

Steps to reproduce:

1. Start with a editor clean of any HTML.
2. Insert a DIV using the DIV Container toolbar-button.
3. When cursor is inside the DIV, try to move the cursor outside the DIV for further writing. That dosent work.

This happens only of the DIV is the last element in the editor. If there is a span or paragraph below the DIV, you are able to mouseclick/press key down to move the cursor in position outside and below the DIV.

A similar bug has been reported before: #994

Summary: When a DIV Container is the only element in the editor, you are unable to get the cursor outside the DIV if you want to type something else. A workaround is to type something first, then move the cursor above the text and insert a DIV there - that makes the DIV the second last element. But we can't relay on our clients to know this workaround.

Tip: I added a border which made it easier to see the DIV and understand the issue.

Tested and fails in: Firefox 3.5.10 Opera 10.10 Chrome Google Chrome 5+ IE 6, 7, 8.

It looks like the 'html' element is not properly handled in IE7.

Users will use 'html' UI elements when the content that will be put there is not yet known, so it is impossible to set a fixed with for it in advance.

Once the HTML content of such 'html' element is set, the dialog should expand correctly to show the contents.

It works this way in FF, Safari and Ie6 / IE8 (in IE8 mode), however it doesn't work for me in IE7-like browser (IE7 and IE8 in IE7 mode). When I add content to the 'html' element, it is not shown correctly (it is truncated at the bottom).

I have attached a sample where this bug can be easily reproduced.

Enter at the end of DT element should create new DD element and vice versa.

Styles plugin loses the attributes of PRE blocks in mergePre and toPre functions.

The link plugin is not registering a selected link when a <br> is inside an <a>, which the editor sometimes creates by itself.

To replicate: ​http://nightly.ckeditor.com/5649/_samples/replacebyclass.html

• press Enter at end of line
• click the Link button
• create a link (e.g. www.google.com)
• click OK
• click link button again, it will work
• click OK
• click link button again, there will be nothing in the URL field
• you also then can't right click the link and edit it.

If that doesn't reproduce the problem, try adding a <br> to the <a> element manually, e.g. using Firebug.

Firefox 3.6.6, Mac OS X 10.6.3

When in high contrast mode, navigating with the keyboard to a menu button and using spacebar to open the menu panel, it works correctly the first time. If you navigate back to the toolbar and open the menu again the majority of times the panel displays for a few milliseconds, then focus goes to the content area. You can see this behaviour by using the SCAYT menu button.

Steps to reporduce:

1. Turn on high contrast mode.

2. Load an instance of the editor.

3. Tab to the content area.

4. Press alt+F10 to give focus to the toolbar.

5. Navigate to the SCAYT menu and press spacebar to open the menu.

6. Press the down arrow to select the first menu item.

7. Press Esc to close the menu and return focus to the content area.

8. Repeat steps 4 and 5 to open the menu again.

The menu will only display for a few milliseconds, in more cases then not.

When I create a list <ol> elment under a <p> element

• The example is shown as follow:

  <p>
  	test</p>
  <ol>
  	<li>
  		li</li>
  	<li>
  		li</li>
  </ol>
  

• when I press delete after the last character of 'test', the ckeditor will prduce a bug, the example is shown as follow:

  <p>
  	test</p>
  <p>
  	<li>
  		li</li>
  </p>
  <p>
  	<li>
  		li</li>
  </p>
  

And my IE version is IE8.0.6001.18702, Os is Windows XP + SP3, ckediotr version is 3.3.1

However when I use win7 + IE8.0.7600.16385 to test this example, the bug doesn't appear!

I am looking forward your reply!

Best regards,

Littlepower

Step to reproduce:

1. Select more than 2 lines with inserted numberic list format
2. click on insert horizontal line
3. Hit javascript error

Select the sentences >click on background color >select more color

click on cancel>click on cancel again

Click on <OK> ( Notice it show selected color )

it does not change background color in the editor

Enter !@#$ and click on enter , notice it shown underline.

Happens on IE only