Custom Query

Filters
 
Or
 
  
 
Columns

Show under each result:

Results (1201 - 1300 of 2591)

← 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 →
Ticket Summary Status Owner Type Priority Milestone
#9967 Fire an Event when items are selected on toolbar combo boxes and color panels confirmed New Feature Normal
Description

Currently there is no way to detect that an option has been selected from a toolbar combo box (e.g. font, format) or color panels (e.g. Text Color, Background Color). It is possible to detect other toolbar actions using afterCommandExec and the command name e.g. bold, indent, numberedlist.

Can you provide an event that would signify that a selection has been made on a combo box or color panel?

#9968 [iOS] Cannot select an image on an ipad confirmed Bug Normal
Description

Cannot seem to select an image in the editor in order to access the image dialog box so can adjust image properties. All that happens is the standard ipad select area indicators appear.

i'm using an ipad2 ios 6.0.1

#9975 Move node's methods from htmlDataProcessor to htmlParser.node confirmed Bug Normal
Description

getLast, getNext, isEmpty, isBlockBoundary, insertAfter, insertBefore, append, removeFromParent, getNodeIndex.

Thanks to that will be able to test them, so tests should be written.

#9979 magicline should show up after more html5 flow elements confirmed Bug Normal
Description

Descriptive summary :

It seems that it is impossible to leave many flow elements. Magic line simply doesn’t show when you hover cursor below/ above or between them.

This of course makes it impossible to add new content outside of these elements.

These missing flow elements are:

  • section elements: nav, section, article, aside, header, footer, hgroup
  • Grouping elements: address
  • Interactive elements: details (supported only in Chrome).
  • Form elements: fieldset
  • Interactive elements figure
  • Embedded content: audio, video

Menu isn’t supported in any browser so I wasn’t able to check it.

Rest of flow elements from this list ​http://www.w3.org/TR/html-markup/common-models.html either don’t need magic line or is supported by this plugin.

#9984 Blockquote with enterMode = ckeditor.ENTER_BR and autoParagraph = false creates <p> tags confirmed Bug Normal
Description

Using CKEDITOR 4 with <br /> for new lines instead of paragraphs doesn't work for Blockquote as it uses <p> tags with that tool.

This is my configuration: 

  // Use <br /> instead of <p> tags.
  CKEDITOR.config.enterMode = CKEDITOR.ENTER_BR;
  CKEDITOR.config.autoParagraph = false;

This is what I'm seeing in the editor: 

<body contenteditable="true" class="cke_editable cke_editable_themed cke_contents_ltr" spellcheck="false">
Test text
<blockquote>
<p>A blockquote</p>
</blockquote>
<p>
More lines<br>
Another line<br><br>
</p>
</body>
#9985 Using the Delete key causes input boxes to be deleted. confirmed Bug Normal
Description

Reproduces on the latest version of chrome (24.0.1312.52) in both OSX Mountain Lion and Windows 7.

Steps to reproduce:

  1. Go to full demo (​http://ckeditor.com/demo#full)
  1. Edit source to replace existing with the following: (this was originally manually typed into the editor) 
    <p>This is the first sentence.</p>

<p>This is a sentence with an&nbsp;<input name="input" type="text" /> in the middle.</p>
  2. Leave source view and put the cursor at the end of the first sentence.
  1. On windows, hit the DEL key or on a Mac, use FN+DELETE to get a delete rather than a backspace.

Expected:
The paragraphs should be combined, like this: 

<p>This is the first sentence.This is a sentence with an&nbsp;<input name="input" type="text" /> in the middle.</p>

Actual:
The input box and contents afterward are removed: 

<p>This is the first sentence.<span style="line-height: 1.6em;">This is a sentence with an&nbsp;</span></p>
#9996 Selection disappears from last line on getData() confirmed Bug Normal
Description
  1. open ​http://nightly.ckeditor.com/13-01-28-08-51/standard/samples/plugins/enterkey/enterkey.html
  2. select BR mode
  3. remove all text
  4. type in some text (line 1)
  5. press enter
  6. type in some more text (line 2)
  7. select all text on the second line
  8. open javascript console and run 
CKEDITOR.instances.editor1.getData()

Result: selection disappears! Expected: selection to stay where it was.

Interesting that if you select line 1 and run getData(), everything stays normal.

I was able to reproduce it in Chrome. Bug first appeared in 3.6.5, 3.6.4 works fine.

#9998 [Umbrella][Blink/Webkit] Issues related to backspace and delete keys handling confirmed Bug Normal
Description

This is an umbrella ticket for all issues related to backspace and delete keys handling caused by the following Webkit's and Blink's bugs:

Related tickets: #8438, #8439, #8440, #10359, #10515, #11483, #11592, #11415 and many others...

Cases

<span> created while joining two adjacent elements

Related issues: #9998

Target ticket: #11861

Example 1

  1. Set data 
    <p>foo</p>
<p>^bar</p>
  2. Backspace.
  3. <p>foo<span style="line-height:1.6">bar</span></p>

Example 2

  1. Set data 
    <h1>foo</h1>
<p>^bar</p>
  2. Backspace.
  3. <h1>foo<span style="font-size:13px; line-height:1.6">bar</span></h1>

<span> created while joining two adjacent elements (non-collapsed selection)

Related issues: #9998

Target ticket: #11980

Example 1

  1. Set data 
    <div>
  <p>fo[o</p>
</div>

<div>
  <p>b]ar</p>
</div>
  2. BACKSPACE
  3. Result: 
    <div>
  <p>fo<span style="line-height:1.6">^ar</span></p>
</div>
  4. Expected 
    <div>
  <p>fo^ar</p>
</div>

Attempt to preserve the visual style of the contents -> i.e. <strong> becomes <b>

Backspace (unable to remove formatting)

Related issues: #8438, #11592, #11483

  1. <p>x<strong>y^</strong></p>
  2. Backspace.
  3. Type “z”.
  4. <p>x<b>z^</b></p>

Backspace (underlined link becomes <u>)

Related issues: #11592

  1. <p><a href="http://foo">[foo]</a>bar</p>
  2. Backspace.
  3. Type "foo".
  4. <p><u>foo^</u>bar</p>
  5. Frown
  6. Grin

Note: This case uncovers the algorithm used in Webkit. The engine attempts to preserve the style no matter how ridiculous it would be – it uses <u> because it is the closest style, which corresponds visually with underlined link.

Del (unable to remove formatting)

Related issues: #8439, #8440, #12100

  1. <p>foo</p>
  2. CTRL+A.
  3. Bold.
  4. DEL.
  5. Type “bar”.
  6. <p><b>bar^</b></p>

Another example available in Blink browsers only:

  1. Switch to source mode and clear editor contents
  2. Paste this code <p><strong>test</strong></p>
  3. Press Ctrl+A
  4. Hit Backspace.

Result: Elements' path in Blink shows 'p' and when user starts typing text will be bold again. If user presses Backspace two times then whole formatting will be removed but there is no way for user to know that.

<span> created while splitting elements (DnD), extra &nbsp;

Related issues: #10515

  1. Set the following HTML 
    <p>
    <span style="font-size:28px">
        foo 
        <strong>[bar]</strong> 
        <em>bam</em> 
        bang
    </span>
</p>
  2. Drag [bar] into ba|m
  3. editor.getData().
  4. See the result. 
    <p>
    <span style="font-size:28px">
        foo &nbsp;
        <em>ba</em>
    </span>
    <strong style="font-size:28px; line-height:44.79999923706055px">bar</strong>
    <em style="font-size:28px; line-height:1.6">m</em>
    <span style="font-size:28px; line-height:1.6"> bang</span>
</p>
  5. Then WYSIWYG->Source->WYSIWYG.
  6. ACF brings the ultimate destruction. 
    <p>
    <span style="font-size:28px">foo &nbsp;<em>ba</em></span>
    <strong>bar</strong>
    <em>m</em>
    <span style="font-size:28px"> bang</span>
</p>
#10000 Magicline plugin might slow the page down confirmed Bug Normal
Description

Reported on the forum: ​http://ckeditor.com/comment/123583#comment-123583

I noticed my site was becoming slow and less responsive recently and after some profiling I traced it to this plugin. It seems that anywhere the mouse is moved triggers a mouseout event so basically the entire time the user is using a page in my application with this plugin it was causing it to be less responsive.

I traced it to this line of code inside the plugin.js file of magicline. I am using v4 of ckeditor with the div based editor.

Hide the box on mouseout if mouse leaves document.

editable.attachListener( that.inInlineMode ? doc : doc.getWindow().getFrame(), 'mouseout', function( event ) {

See attached profile results. Notice I ran it for just 2.1 seconds and the top 7 calls were running for 1.7 - 1.9s a piece which explains why it was so bogged down. These results are probably slightly misleading as well because as I move my mouse in and out of the page to click the Profiler in Firebug it would stop triggering mouse out events which means realistically as long as the user has their mouse moving on the page cycles are being consumed the entire time for no reason.

#10003 Strike through button doesnt not correctly confirmed Bug Normal
Description

Strike through doesn't work for first time for the steps given below:

Way to reproduce:

1) Enter some text and select it. Click strike through
2) Go to next line and click on strike through again to disable it
3) Enter some text again, select it and click strike through
4) Strike out doesnt work for the first time.
5) Again select the text and click strike through. It works

​http://screencast.com/t/1ZEvYXj9zd

#10014 Promote better "API Changes in CKEditor 4" confirmed Task Normal
Description

I just spent a couple of minutes on trying to find this article for one user and I did it just because I remembered that something like this existed.

How about:

  1. Promoting it somewhere on the front page of docs.ckeditor.com
  1. Putting at the top of CHANGES.md something like this:

The list of relevant changes can be found in the ​API Changes page of the CKEditor 4 documentation.

?

#10017 Remove obsolete _translationstatus.txt files confirmed Task Normal
Description

I do not think it still makes a lot of sense to keep the _translationstatus.txt text files in CKEditor lang folders.

A much better overview of current translation status (with up-to-date stats) can be checked online @ ​CKEditor project page @ Transifex

We still can have a text file in the lang folder (e.g. readme.md), but with an explanation how one can contribute and with a direct link to CKEditor UI translation center, plus eventually a link to documentation about how one can contribute.

I guess providing such a file is not a requirement though.

#10023 ShowBlocks, do not outline forms/lists/tables confirmed Bug Normal
Description

Currently showblocks outlines too few blocks. It also outlines forms even when showblocks button is not active.

Edit: It seems that show blocks doesn't show tables, forms, lists and other HTML flow elements - ​http://www.w3.org/TR/html-markup/common-models.html - like: article, section, nav, aside, hgroup, header, footer, address, main, video, audio, menu, details, fieldset.

Please note that tables and firms have very similar style (dotted border) to outline.

#10026 Breaking quotes in Email replies new New Feature Normal
Description

see article in forum: Breaking block quotes in Email replies relaated topic: Ticket #7354

Short Description: Quoted parts of messages are not allways splited, if enter is pressed, e.g. "quotes with a <div>-Tag" are not splitted.

Thunderbird or any other Email client splits the quoted part of the message, if enter is pressed.

#10033 Keydown listener doesn't catch enter key confirmed Bug Normal
Description
  1. Paste in below code in replacebycode sample 
    var editor = CKEDITOR.replace( 'editor1', {

} );
			
editor.on( 'pluginsLoaded', function( evt ) {
		editor.on( 'contentDom', function( e ) {
			var editable = editor.editable();		
			var doc = editor.document;
			editable.attachListener( doc, 'keydown', function( event ){		
				console.log(event);
			});
		});
	});
  2. Run sample and Press Enter
  3. keydown listener doesn't catch it.

Problem can be reproduced in all browsers from CKEditor 4 beta

#10034 CKEditor 4 Inline editing - Clean trash after destroy confirmed Bug Normal
Description

Hi, I am using the CKEditor 4 Inline editing and as i sow its not cleanning all the trash after using destroy.

Code before i edit it: 

<div id="myId">Text text text</div>

Code after i edit it and use destroy to remove the CKEditor instance: 

<div id="myId" class="cke_focus" style="position: relative;">Text text text</div>

As we can see the class of "cke_focus" and the "position:relative" are not removed when i use the destroy.

Are there is a way to remove all the CKEditor trash?

#10041 CKEditor 4 Inline Editing - Show the toolbar on a instance without focusing it confirmed New Feature Normal
Description

Hi, When i use "CKEDITOR.inline(myId)" its applying the editor instance to the object but its applying toolbar to it if i do not focus the object.

In my case, I do not like to focus the object on startup because its scroll the page to the start/end (IE browser focus to the end of the page).

It will be very good if i will be able to show the toolbar on a instance without focusing it manually.

I also ask in some forums and there is no solutions: ​http://ckeditor.com/forums/CKEditor/CKEditor-Inline-Editing-doesnt-show-the-toolbar-on-start ​http://stackoverflow.com/questions/14724279/ckeditor-4-inline-how-to-show-the-toolbar-on-a-object-without-focusing-it

#10043 Current CKEDITOR.htmlWriter.setRules documentation incomplete confirmed Task Normal
Description

CKEDITOR.htmlWriter.setRules documentation doesn't list all available options and per instance and for every CKEDitor instance configuration examples. ​http://docs.ckeditor.com/#!/api/CKEDITOR.htmlWriter

Old documentation was more complete. ​http://docs.cksource.com/CKEditor_3.x/Developers_Guide/Output_Formatting

#10047 FF: Replacing <Enter> with <Shift+Enter> results in extra <br /> confirmed Bug Normal
Description

Firefox only (not reproducible in IE10, Chrome24):

I type a line of text followed by Enter. Whoops - I meant to type Shift+Enter, so I hit Backspace followed by Shift+Enter. What gives? Now my cursor is two 'lines' down!

Here's what I type:

"A line of text", <Enter>, <Backspace>, <Shift+Enter>

And now my cursor is two <br /> down instead of just one.

Verified in the basic demo and the nightly Using the "Enter" key in CKEditor sample, where I can see that 2 <br /> have been inserted from my final single <Shift+Enter>.

I'm running Firefox 18.0.2.

#10049 CKBuilder page could better differentiate Basic/Standard/Full plugins confirmed New Feature Normal
Description

In customizing my build I have to build up my own table of which plugins were included in each of the Basic/Standard/Full presets in order to know exactly what I'm getting.

The CKBuilder interface should use color-coding or additional icons to make it obvious which additional plugins we're getting going from Basic->Standard or from Standard->Full.

#10051 Image dialog - ratio lock is quantised confirmed Bug Normal
Description

The ratio lock is quantised, making the lock to unexpectedly unlock.

  1. Insert a 4000x3000 px image in CKEditor.
  2. Right click on the image and open the image dialiog.
  3. Ensure the ratio lock is lock, then change its width to: 330. The height automatically change to 248.
  4. Save, re-open the image dialog. The width / height are still 330x248, but the ratio lock is unlock even if the values respect the image ratio.

The calculated ratios are not equals:
originalRatio: 1333 (accurate)
thisRatio: 1330 (quantised)

I think the logic in switchLockRatio method is wrong. Instead of comparing the ratio, where one has been quantised to pixels, it would make more sense to re-calculate the height and check if the image height equals the calculated one.

The following code replacement fix this issue:

File: _source/plugins/image/dialogs/image.js
Lines: 132 to 144 

var width = dialog.getValueOf( 'info', 'txtWidth' ),
	height = dialog.getValueOf( 'info', 'txtHeight' ),
	originalRatio = oImageOriginal.$.width / oImageOriginal.$.height;
dialog.lockRatio = false; // Default: unlock ratio

if (!width && !height) {
	dialog.lockRatio = true;
} else {
	if (!isNaN(originalRatio)) {
		var ratioHeight = Math.round(width / originalRatio);
		if (ratioHeight == height) {
			dialog.lockRatio = true;
		}
	}
}
#10052 Image dialog - preview do not refresh when field is cleared confirmed Bug Normal
Description

The preview window in the image dialog do not clear the CSS attribute when the value is cleared:

  1. Insert an image in CKEditor.
  2. Right click on the image and open the image dialiog.
  3. Enter a border of 9. The preview image show a border of 9px.
  4. Clear the border field (erase the number 9 from the field). The preview image still show a border of 9px.

Affect: Border, HSpace VSpace, etc.

I notice the following in the code:
File: _source/plugins/image/dialogs/image.js
Line: 848 

if ( !isNaN( value ) )
{
	element.setStyle( 'border-width', CKEDITOR.tools.cssLength( value ) );
	element.setStyle( 'border-style', 'solid' );
}
else if ( !value && this.isChanged() )
	element.removeStyle( 'border' );

I don't understand why it check if the field has been changed before removing the style, but if I remove that check, it works as expected: 

if ( !isNaN( value ) )
{
	element.setStyle( 'border-width', CKEDITOR.tools.cssLength( value ) );
	element.setStyle( 'border-style', 'solid' );
}
else if ( !value )
	element.removeStyle( 'border' );
#10060 Disabled textarea fields are not readOnly by default confirmed Bug Normal
Description

According to CKEDITOR.config.readOnly, if not specified, it should check to see if the linked textarea has the disabled attribute. This is not currently happening.

#10061 Copy img in Chrome result in opacity: 0.9 confirmed Bug Normal
Description

I've discovered a bug when using the CKEditor 4.01 in Chrome Version 24.0.1312.57 m

For some reason when you select an image and copy paste it, it adds: style="opacity: 0.9" which makes the picture look pale.

Chrome is the only browser that I've seen this problem occur in.

#10065 [Webkit] Changing "Object Styles" of an image does not work properly confirmed Bug Normal
Description

The "Object Styles" of an image with a link cannot be changed.

Go to ​http://ckeditor.com/demo

  • Click on the image of the rocket.
  • Click on the Style drop-down menu and click "Styled image (left)". This works fine.
  • Now attach a link to the image (select it, click the link button on the toolbar and add a link).
  • Now try to change the object style ("Styled image (left)" or "Styled image (right)"). Nothing happens. The style is not changed / applied.
#10066 Feature to make it possible to hook into elementspath events confirmed New Feature Normal
Description

I need some way to hook into events for elementspath items. I imagine it would not be hard to add some kind of event that happens when elementspath is rightclicked.

This would make it possible to build a custom elementspath context menu - which I need and which stops me from upgrading.

This could be done my modifying the current elementspath\plugin.js and adding to the path item template something like oncontextmenu="HoweverCKEHandlesEvents(event, currentElementsPath, index)" and then defining an onContextMenu handler exactly like onClick is defined; 

var onClickHanlder = CKEDITOR.tools.addFunction( onClick );

// imitate onclick and build a contextmenu handling system like it...
function onContextMenu(elementIndex, event) {
	editor.focus();
	var element = editor._.elementsPath.list[elementIndex];
	var data = {};
	data.event = event;
	data.element = element;
	editor.execCommand('elementspathContextmenuForElement', data);
}

var onContextMenuHanlder = CKEDITOR.tools.addFunction(onContextMenu);

---

This would make it possible to create plugins that Greatly extend the usage of elementspath and is really not that very difficult to implement and wouldn't take that link I think. What do you think? This is a must have for me but I Really do not want to create my own "fork" of CKE to get this done. How quickly could this make it into the main trunk?

#10078 BBCode: No line-breaks when pasting paragraphed text confirmed Bug Normal
Description

When pasting a list of items from notepad/textedit in the bbcode editor line breaks are not preserved, this seems to be a chrome specific issue. Can be recreated in the demo section.

To recreate: 1.) Make a list of a few items in notepad/textedit "one two three"

2.) Copy, then paste into a blank bbcode ckeditor ​http://ckeditor.com/demo#bbcode

3.) Click source, see there are no line breaks "onetwothree", on some versions of chrome no text will be seen until hitting source.

I believe this is related to the br mode of the browser, it seems that chrome is inserting <p> tags when copied directly to the editor

#10085 Plugin divarea causes baseHref not working confirmed Bug Normal
Description

This worked fine in 4.0. I've only tried it with the minified ckeditor.js, so it's possible it might be a bug in that.

Please look into this as the use of baseHref is critical for my client's application.

Thanks.

Edit:
*Problem can be reporduced from CKEditor 4.0.1
*To reproduce open attached ckeditor-bug package and test 4.0.1 sample page. It turns out that when divarea is used baseHref doesn't work.
Remove the plugin and it works again: 

 $(document).ready(function() {
        $('#make-editor').click(function(){
            CKEDITOR.replace('testeditor', {
		removePlugins : 'divarea'
		});
        });
});

Please see comment:3 for detailed TC

#10092 Focus lost on IE when reopening in an iframe with a jQuery onload listener confirmed Bug Normal
Description

CKEditor doesn't receive the focus with a mouse click in a quite specific case. The editor area seems to only be refocusable by clicking on a toolbar button.

  • the editor has been loaded in an iframe
  • there is an overlay behind the iframe
  • the iframe has any jQuery .on('load') binding
  • the iframe has already been created and destroyed
  • on IE8-10.

See the test case here:

Download the test case here: ​http://venko.net/naapuri/tyo/ckeditor/focus.zip

#10093 IE9 inline/destroy multiple times causes return key to repeat that many times confirmed Bug Normal
Description

In IE9 if you call inline and destroy multiple times on the same div then you hit the enter key in the editor you get as many paragraph tags as you inline/destroyed the editor.

Doesn't appear to happen in IE7, IE8 or Firefox (sorry, can't test webkit browsers where I am).

Attached is file with simple script to recreate.

#10096 [IE10] Error javascript (Browser mode IE 10) confirmed Bug Normal
Description

To reproduce:

Press New Table Toolbar button

Result: Unable to get property '0' of undefined or null reference

Line: 463 URI: ckeditor/ckeditor.js

Problem can be reproduced in IE10 (Browser mode IE 10) from CKEditor 4.0.1

#10101 afterUndo and afterRedo are fired on the commands new Alfonso Martínez de Lizarrondo Bug Normal
Description

There are two events "afterUndo" and "afterRedo" that seem like good ways to be notified about when an undo/redo operation is performed but they are trickier to use because they aren't fired on the editor but on the commands themselves: 

			var undoCommand = editor.addCommand( 'undo',
				{
					exec : function()
					{
						if ( undoManager.undo() )
						{
							editor.selectionChange();
							this.fire( 'afterUndo' );
						}
					},
					state : CKEDITOR.TRISTATE_DISABLED,
					canUndo : false
				});

			var redoCommand = editor.addCommand( 'redo',
				{
					exec : function()
					{
						if ( undoManager.redo() )
						{
							editor.selectionChange();
							this.fire( 'afterRedo' );
						}
					},
					state : CKEDITOR.TRISTATE_DISABLED,
					canUndo : false
				});

if instead of "this.fire" the code is "editor.fire" then they can be used in a normal way.

I think that this is just a typo that no one has realized so far (after all these events aren't documented in any way)

I don't think that anyone is using the current events so it shouldn't be a problem to correct them.

I guess that you won't bother about fixing this in 3.6 so I'll create a patch just for 4.0

#10104 Add CKBuilder to Maven confirmed New Feature Normal
Description

A request added at the community forum: ​http://ckeditor.com/comment/124098#comment-124098

Is there any way to integrate ckbuilder into a maven based build process?

Is there a chance to have the ckbuilder in maven central?

#10114 Invalid cursor position for non-editable element confirmed Bug Normal
Description

When non-editable element is the last one in the paragraph and cursor is right after this non-editable element, the cursor is aligned to the right edge of the editor, instead of beeing just after the last word.

Sample content for editor:

<p>editable

<span contenteditable="false">non editable</span>

</p>

#10126 [Opera] Arrows in colorbuttons are displaced in RTL editor confirmed Bug Normal
Description

#10127 [Opera] Error thrown when applying list on incorrectly focused editor confirmed Bug Normal
Description
  1. Open inlinebycode sample.
  2. Without focusing editor first, click inside editor area, but not near the text, so editor is focused, but cursor isn't blinking.
  3. Click list button.
  4. Error is thrown. 
Uncaught exception: TypeError: Cannot convert 'path' to object

Error thrown at line 70, column 2 in <anonymous function: refresh>(editor, path) in http://localhost/cksource/ckeditor-dev/core/command.js:
    if ( this.context && !path.isContextFor( this.context ) )
called from line 416, column 3 in <anonymous function: exec>(editor) in http://localhost/cksource/ckeditor-dev/plugins/list/plugin.js:
    this.refresh(editor, editor.elementPath());
called via Function.prototype.call() from line 52, column 2 in <anonymous function: exec>(data) in http://localhost/cksource/ckeditor-dev/core/command.js:
    return ( commandDefinition.exec.call( this, editor, data ) !== false );
called from line 531, column 5 in <anonymous function: execCommand>(commandName, data) in http://localhost/cksource/ckeditor-dev/core/editor.js:
    eventData.returnValue = command.exec(eventData.commandData);
called from line 79, column 4 in <anonymous function: click>(editor) in http://localhost/cksource/ckeditor-dev/plugins/button/plugin.js:
    editor.execCommand( definition.command );
called from line 135, column 5 in <anonymous function: execute>() in http://localhost/cksource/ckeditor-dev/plugins/button/plugin.js:
    this.button.click( editor );
called from line 182, column 4 in <anonymous function>() in http://localhost/cksource/ckeditor-dev/plugins/button/plugin.js:
    instance.execute();
called via Function.prototype.apply() from line 569, column 4 in <anonymous function: addFunction>() in http://localhost/cksource/ckeditor-dev/core/tools.js:
    return fn.apply(scope || this, arguments);
called via Function.prototype.apply() from line 595, column 3 in <anonymous function: callFunction>(ref) in http://localhost/cksource/ckeditor-dev/core/tools.js:
    return fn && fn.apply(window, Array.prototype.slice.call(arguments, 1));
called from line 1, column 0 in <anonymous function>(event) in http://localhost/cksource/ckeditor-dev/samples/inlinebycode.html:
    CKEDITOR.tools.callFunction(121, this);
#10134 Window crashes when inserting HR into selected TR. confirmed Bug Normal
Description
  1. Open ​http://ckeditor4.t/ckeditor/samples/replacebycode.html
  2. Scroll to the table (on the right).
  3. Put caret in some cell.
  4. Use elementspath, click TR.
  5. Insert HR via toolbar.
  6. Infinite loop. Window is broken.

Tested in Chrome 25, IE10.

#10142 CKBuilder: Closure Compiler thinks native is a reserved keyword confirmed Bug Normal
Description

It it impossible to build CKEditor after this change: ​a2136ce (native is a reserved keyword). I remember that it happened at least once in the past. We renamed the "native" variable to "nativ".

After a quick research it turned out we may get rid of this issue forever by using this piece of code: 

options.setLanguageIn( CompilerOptions.LanguageMode.ECMASCRIPT5 );

native was a future reserved word in ECMAScript v3. Closure Compiler follows ES4, which also lists native as a reserved keyword.

However, I believe it will be quite unsafe to switch CC to ES5 mode at this moment, because this time we may not notice code that will be incompatible with IE7.

#10147 Editable#setup called twice during wysiwygarea initialization confirmed Task Normal
Description

It is first time called in editable's constructor, second in onDomReady inside wysiwygarea plugin.

This may not be a wrong behaviour, though it looks like one. We should investigate whether after the first call every changes (new listeners, etc) are cleaned up before second call. This maybe happen thanks to clean up done by setData which is done in onLoad function inside 'wysiwyg' mode (not class) constructor.

If everything is ok, appropriate docs should be written to avoid confusion in the future. Why editable#setup has to be called for the first time and why for the second time?

#10150 Copy styles available in styles and formats combos from contents.css to samples/styles.css assigned Olek Nowodziński Bug Normal
Description
  1. Open inlineall sample.
  2. Focus one of editors.
  3. Apply marker style from stylescombo.
  4. Style is applied, but it's not visible in the inline editor contents.

It works on themed editor, because marker style is defined in contents.css.

Other TC:

  1. Open inlineall sample.
  2. Open format combo.
  3. Apply header 1 style.
  4. Open combo again and compare how header 1 looks in combo and in contents of the editor. They have different font styles.
#10153 Style combo update issue when inline editor programmatically created and focused confirmed Bug Normal
Description

When programmatically creating an inline editor and giving it focus, the style combo doesn't reflect the selected style of the cursor. Worse, in Chrome, when you move the cursor/selection while in the same HTML element, the combo still doesn't update. It's not until you switch to a new element that the combo box starts updating.

May be easier to understand with this repro. Using the HTML/JS below, do the following in Chrome (I also attached a zip file):

  1. Click the "Start Editing" button
  2. Note that the cursor is on the H1, but the styles combo says "Styles" and not "Heading 1."
  3. Click & highlight elsewhere within the H1.
  4. Note that the styles combo still is stuck on "Styles."

I've been unable to find any workaround for this bug.

Repro HTML:

<html>

<head>

<script src="ckeditor/ckeditor.js"></script>

<script>

function onEdit() {

var div = document.getElementById('div'); div.setAttribute('contenteditable', 'true'); var editor = CKEDITOR.inline(div); div.focus();

}

</script> </head> <body> <div id="div" style="margin-top: 300px">

<h1>Header</h1> <p>Lorum Ipsum etc. etc.</p>

</div> <button onclick="onEdit()">Start Editing</button> </body>

</html>

Repro config.js:

CKEDITOR.editorConfig = function( config ) {

config.toolbar = [

{

name: 'style', items: Styles?

}

];

config.stylesSet = [

{

name: 'Heading 1', element: 'h1'

}, {

name: 'Paragraph', element: 'p'

}

];

};

#10156 [Webkit] Cut&pasting text adds inline wrappers confirmed Bug Normal
Description

This is basically the same as #9754 except that it requires config.contentsCss.

Steps:

  1. Add problem.css to the samples folder
  2. In the replacebycode.html sample change the JS to CKEDITOR.replace( 'editor1', { contentsCss: 'problem.css' } );
  3. Viewing the sample page, cut a word from the now red colored text and paste it back into the text.

Expected result: The source should be unchanged.

Actual result: The pasted text is now wrapped by <span style="color: rgb(255, 0, 0);">...</span>.

Notes:

  • Pasting using the Paste from Word command has the desired effect. (Removes styles but doesn't remove a link for example.)
  • Is it possible to make Cmd+V, Ctrl+V paste as if Paste from Word? I know of config.forcePasteAsPlainText but that would remove links, as well as styles.
#10159 Sourcedialog and Sourcearea plugins - better interoperability needed? confirmed Bug Normal
Description

The new sourcedialog plugin introduced a new button name for a feature that already exists - editing source code. I think it is wrong because of various reasons.

1. Do not use different button names

(i) Noone would like to have enabled both "Source" buttons in the toolbar at the same time. The only purpose of both plugins is to edit source code (that's why even the buttons looks the same).

Why user may want to use both plugins? Let's look at the case of Drupal: there is a framed editor in the administration area and inline editor in another place. So it needs both plugins (unless the sourcedialog will be used in each case, however some may prefer the sourcearea plugin and use sourcedialog just as a fallback solution.

(ii) Once user realize that he has two buttons in some samples (after downloading a custom package), he'll have to start digging how to change the toolbar configuration to get rid of one button. And then, deal everywhere with toolbar configurations or removePlugins/extraPlugins and so on.

* Unless we configure online builder to allow adding just one plugin of this kind.

2. sourcedialog could be shipped in std/full package

Currently, because of a "toolbar conflict" between both plugins, we cannot have both enabled in default packages (at least without adding code to other samples to handle both plugins at once). If we ensure that both plugins can be enabled at once and that duplicated buttons will not appear, that would be cool, because for example the inline/sharedspace samples could show this button.

There is so many samples, that it is still hard to say whether CKEditor supports source editing in inline mode, by looking at promoted inline samples at the top where the "Source" button is missing.

This is the main reason why I think it would be better if plugins could somehow cooperate, to be able to use source editing in inline mode without any extra effort. If enabling the "Source" button in inline environment would require from user 0 additional steps that would be quite awesome.

3. sourcedialog for inline editing sourcearea for framed mode?

If both plugins are enabled, we could make the plugins a bit more smart: let the sourcearea plugin work in framed environment and enable the source dialog plugin in environment not supported by sourcearea. Again, a single button name would be used for the toolbar configuration.

#10161 Docprops plugin requires colordialog confirmed Bug Normal
Description

Found by Olek. It looks like the docprops is missing colordialog entry in the "requires" property. The "Choose" buttons in "Design" tab are supposed to launch colordialog.

#10169 Editor context menu unavailable when opened outside the body element of the content area confirmed Bug Normal
Description

I added an Screen shot to make it more simple to understand the ticket.

There is an section where you can enter text at the CKEditor, in the following called content section. If you enter a small text at the content section and leave out enough unfilled place in the content section you can't paste text inside the editor. If you right click in the free place of the content section the browsers default context-menu will be opened but not the context menu of the CKEditor. While the text selection cursor will be changed, the context menu keeps missing.

#10175 Label Style should have float:none. confirmed Bug Normal
Description

Continuation of ticket #7943.

It has been fixed in CKE 3.x but can be reproduced in 4.x branch in both skins.

To reproduce:
Add styles on page: 

<style type="text/css">
    label,select,.ui-select-menu { float: left; margin-right: 10px; font-size:1.3em; height:26px; line-height:26px; font-size:1em; }
</style>

Open dialog - labels are floated.

#10176 Sourcedialog Resizing confirmed Bug Normal
Description

The dialog itself resizes but the textarea remains the same size.

It's not a big deal, but since the textarea is the only element within the dialog, it would make sense for them to resize together.

#10178 [Webkit] : Magic line is too aggressive in enter mode BR confirmed Bug Normal
Description

To reproduce:

  1. Open enter key sample and set mode to BR
  2. Press new page button
  3. Insert two div containers
  4. You should have div div p in elements path now - try clicking inside content area

Result: Elements path shows only body like there is nothing in editor.

In 4.0 beta - You had to source and back to see the code again plus magic line above divs.

From 4.0 - You also have to switch to source and back to see the code again but magic line behaves different.
It shows up in first line and once you click on it, elements path shows only body. This gives you false impression that whole code wasn't created. Since cursor is now in correct place you may want to create your containers again. If you do that and switch to source you will get two sets of divs.
First set for some reason gets hidden.

Problem can be reproduced only in Webkit.

#10180 [iOS] iPad issue with Dropdown Menus new Bug Normal
Description

I am trying to solve an issue I have on an iPad with the dropdown menus (font, style, spellcheck, etc). On a project I am working on it is impossible to click on the options in the menu with the iPad.

The issue can be demonstrated with the following jsfiddle links:

​http://jsfiddle.net/QXFcM/6/ works correctly on the iPad. You are able to select the options as you would expect.

​http://jsfiddle.net/QXFcM/6/embedded/result does not work correctly. You cannot select any of the options from the menu.

I originally posted this to the forums at ​http://ckeditor.com/forums/Support/iPad-issue-with-Dropdown-Menus but have not gotten any replies.

#10185 Placeholder is not configurable confirmed New Feature Normal
Description

At the moment it is not possible to change the placeholder that is used. So I made it optional configurable via the editor config. Default is still content ?. Due to placeholder is a core plugin i thought its right to report the patch here. If I'm wrong please tell me where to contribute.

#10188 Append template option to templates plugin confirmed New Feature Normal
Description

Proposal for templates plugin expansion for adding an option to append selected template into editor data.

Sometimes and especially with responsive web design it would be very helpful if templates plugin would have an option to append selected template into editor HTML instead of just replacing whole content with a template or inserting template into cursor position.

Quick prototype solution below.

templates/dialogs/templates.js 

function insertTemplate( html ) {
	var dialog = CKEDITOR.dialog.getCurrent(),
		replaceType = dialog.getValueOf( 'selectTpl', 'chkInsertOpt' );

	switch(replaceType)
	{
		case 'append':
			
			editor.fire( 'saveSnapshot' );
			// Everything should happen after the document is loaded (#4073).
			
			var currentHTML = editor.getData( );
			
			editor.setData( currentHTML + html, function() {
				dialog.hide();

				// Place the cursor at the first editable place.
				var range = editor.createRange();
				range.moveToElementEditStart( editor.editable() );
				range.select();
				setTimeout( function() {
					editor.fire( 'saveSnapshot' );
				}, 0 );

			} );
			
			break;
		case 'replace':
			
			editor.fire( 'saveSnapshot' );
			// Everything should happen after the document is loaded (#4073).
			editor.setData( html, function() {
				dialog.hide();

				// Place the cursor at the first editable place.
				var range = editor.createRange();
				range.moveToElementEditStart( editor.editable() );
				range.select();
				setTimeout( function() {
					editor.fire( 'saveSnapshot' );
				}, 0 );

			} );
			
			break;
			
		case 'in-place':
		default:
			
			editor.insertHtml( html );
			dialog.hide();
			
			break;
	}
}

templates/dialogs/templates.js dialog definition. 

{
	id: 'chkInsertOpt',
	type: 'select',
	label: lang.insertOption,
	style: 'width:100%',
	'default': config.templates_replaceContent,
	items: [
	[ 'append' ],
	[ 'in-place' ],
	[ 'replace' ]
	]
}
#10197 CKE 4.x Inline Can't disable object resizing confirmed Bug Normal
Description
  1. Put below code into inlinebycode sample 
    CKEDITOR.disableAutoInline = true;

var editor = CKEDITOR.inline( 'editable', {
	disableObjectResizing : true
});
  2. You can also try putting disableObjectResizing into config.js

It is still possible to resize images with inline editor in Firefox or IE (Please do not confuse with #9317).

#10199 CKFinder Bug - Image Properties Link Tab Browse Server Button confirmed Bug Normal
Description

When you are on the Image Properties dialog and click the Link Tab and then the Browse Server button, it is applying the image-only filter to the results - it should allow for the Files folder to be visible, as well as PDF/other files (same filter as when you click Browse Server from the Insert Link dialog).

Please take a look and adjust the filter as necessary.

Thanks!

#10201 Table properties have no width when opening dialog after resizing table confirmed Bug Normal
Description
  1. Create new table with default width 500.
  2. Switch to wysiwyg mode and resize it with FF native handles.
  3. Right click on the table and select Table properties.

Result: Width has no value assigned.

This is happening because when using Firefox native resize handles, Firefox operates on attributes while CKEditor operates on styles what results in two widths (#9287).
Issue #9287 no longer occurs in CKE 4.1 RC but one does and can be reproduced from CKEditor 4.0 in Firefox browser.

#10204 Introduce editor.htmlProcessor for input HTML processing confirmed New Feature Normal
Description

There are two main input sources:

  • data loaded to editor by setData(),
  • and html inserted into editor when pasting or directly by insertHtml().

Currently input data which comes from both of these sources is processed by the same data processor - htmlDataProcessor.

However, the idea of data processors is to be able to handle different data formats - e.g. markdown and bbcode.

Because of the current architecture this is clearly impossible to replace editor.dataProcessor with e.g. markdownDataProcessor, because pasted data will be processed as... a markdown input.

Therefore I'm proposing to introduce second property - the editor.htmlProcessor. There will be two properties now:

  • editor.dataProcessor - used for 1st kind of input (setData()),
  • editor.htmlProcessor - used for 2nd kind of input (pasted, inserted HTML).

This is very simple fix - we need to set editor.htmlProcessor = editor.dataProcessor and use htmlProcessor instead of dataProcessor in editable.setData.

If one want to handle different input format he has to replace editor.dataProcessor with his own dataProcessor, what will not break pasting/inserting HTML like it does now.

#10207 about plugin (link to target='_blank') confirmed Bug Normal
Description

The about dialog contains 4 links to various pages. But they all target to the same page you are working on. So maybe a target="_blank" is better?

#10209 [FF] Odd cursor position after deleting single bullet point confirmed Bug Normal
Description

In Firefox 19, the cursor moves to the far right after deleting unordered list that contains a single list item.

Steps to reproduce:
1) Open Firefox
2) Load CKEditor with the following HTML source: 

<p>A line of text</p>
<ul>
  <li>First bullet point</li>
</ul>

3) While CKEditor is in WYSIWYG mode, select "First bullet point" as in the first screenshot.
4) Press delete or backspace on the keyboard.
5) The cursor ends up on the same line as the remaining paragraph, but oddly at the far right of the editing area (second screenshot, circled in red).

This was discovered on Windows 7 64-bit using CKEditor 4.0.2. I was not able to reproduce the behavior in IE9 or Chrome25.

#10214 Find/Raplace doesn't match everything. confirmed Bug Normal
Description

Steps To Recreate:

  1. Open replacebycode sample
  2. Switch to source and pase: 
    <p>This is an example [<a href="http://bbc.co.uk">click</a>] of linked text.</p>
  3. Copy the whole line and paste 4 or so times into the page.
  4. Click the Find/Replace button, and select the replace tab.
  5. In the "Find what:" type "[click]". Do not type anything into the "Replace with:" text box.
  6. Click "Replace All".

Result: The find and replace tool only replaces one match when you choose "replace all".

1-5. The same as above

  1. Click "Replace" - the text gets a highlight.
  2. Click "Replace" again -- the text is replaced (with nothing).
  3. Click Cancel.

Result: JS error os thrown: Message: TypeError: startNode is null
Line : 1469
URI: ckeditor4/core/dom/range.js

Problem can be reproduced in all Browsers from CKEditor 3.1.1 in both CKE 3.x and 4.x (v4).

#10215 Object Styles matching is perhaps to aggressive confirmed Bug Normal
Description

Problem can be reproduced from CKEditor 3.2.1 in both CKE 3.x and 4.x (v4).

To reproduce:

  1. Paste below code into source mode 
    <ul style="list-style-type:square;">
	<li>one</li>
	<li>two
	<ul>
		<li>three</li>
	</ul>
	</li>
</ul>
  2. Switch to wysiwyg and apply style for list (from styles dropdown) to outer list.
  3. Click on inner list and notice that list style is selected although inner list doesn't really have it.

I think the problem is checkActive method in core/styles.js which checks all elements from elements path for inline and object styles: 

case CKEDITOR.STYLE_OBJECT:
case CKEDITOR.STYLE_INLINE:
	var elements = elementPath.elements;
	for ( var i = 0, element; i < elements.length; i++ ) {...
#10220 Docs for basePath need improvement confirmed Bug Normal
Description

Based on #10187.

The only way to change base path is using GLOBAL variable - CKEDITOR_BASEPATH. After setting this value, CKEditor.basePath == CKEDITOR_BASEPATH and CKEDITOR.plugins.basePath == CKEDITOR_BASEPATH +'/plugins'.
Properties basePath and plugins.baePath are read-only and changing them will not have any result. If you look at the ticket #10187 you will see that not everyone knows that.

As discussed with @fredck I propose:

  1. Marking them in docs as read-only values
  2. Changing text for plugins.basePath from "The base directory containing all resources." to something like "Path to directory containing all plugins". First one may suggest that this is in fact path to CKEditor directory.
#10226 Add extra <p> tag in IE in enter mode is <br> confirmed Bug Normal
Description

1) select text 2) bold it 3) type anything 4) issue -> add p tag

#10227 Magicline should carefully register commands, keystrokes and listeners confirmed Bug Normal
Description

This code is executed multiple times when switching between modes. It is hard to predict whether this is safe or not, therefore it should be fixed. 

	// Editor commands for accessing difficult focus spaces.
	editor.addCommand( 'accessPreviousSpace', accessFocusSpaceCmd( that ) );
	editor.addCommand( 'accessNextSpace', accessFocusSpaceCmd( that, true ) );

	editor.setKeystroke( [
		[ config.magicline_keystrokePrevious, 'accessPreviousSpace' ],
		[ config.magicline_keystrokeNext, 'accessNextSpace' ]
	] );

	// Revert magicline hot node on undo/redo.
	editor.on( 'loadSnapshot', function( event ) {
		var elements = doc.getElementsByTag( that.enterBehavior ),
			element;

		for ( var i = elements.count(); i--; ) {
			if ( ( element = elements.getItem( i ) ).hasAttribute( 'data-cke-magicline-hot' ) ) {
				// Restore hotNode
				that.hotNode = element;
				// Restore last access direction
				that.lastCmdDirection = element.getAttribute( 'data-cke-magicline-dir' ) === 'true' ? true : false;
				break;
			}
		}
	} );
#10228 Alert for samples when Basic or Standard packages were downloaded assigned Olek Nowodziński Bug Normal
Description

CKEditor needs a dedicated alert for samples that would explain the users why the content they see is limited if they downloaded the Basic or Standard editor packages.

The alert should use the editor#dataFiltered event.

#10239 Tabletools: add ability to set scope in cell attributes dialog new New Feature Normal
Description

Hello,

I was very happy to see how well CKE handles the creation of tables, and its half-automated way of creating table headers, which are essential for table accessibility, esp. for screen readers.

While CKE does a good job in "guessing" the correct directionality of table header scope, it doesn't get them right all the time, and also does not automatically create scope="colgroup" or scope="rowgroup" on merged header cells.

Thus, it would be great if you could add a dropdown to the cell attributes dialog that allows the user to specify the correct scope of a header cell (row|col|rowgroup|colgroup) without having to switch to code view, which is often overwhelming for "mere" content editors.

This dropdown would be perfectly placed following the dropdown with which one can set a cell as data or header cell.

Hoping you can take this option into consideration for the next release, and thanking you in advance!

#10247 [Webkit] Enter key before sub list is not right assigned Olek Nowodziński Bug Normal
Description 
<ol>
	<li>item1</li>
	<li>&nbsp;
	<ol>
		<li>item2</li>
	</ol>
	</li>
</ol>

Place caret at the beginning of second list item (nbsp) and press enter.

Result: 

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

Note that the caret was moved up to the item 1.

I was able to reproduce this on Chrome, but not on FF.

#10252 disableObjectResizing=true in full screen mode confirmed Bug Normal
Description

This is my first bug report. I am attempting to se disableObjectResizing=true and it seems to be working fine, except when clicking to full screen mode / maximize. Once you do that, users can click the image and resize, even if disableObjectResizing is true.

You can test here: ​http://www.elementalblend.com/_prodev/ckeditor4/

  • using firefox 19.0 and mac osx 10.8.2
  • click the image button and put in an absolute image link
  • try to resize (which shouldn't work)
  • click maximize button
  • click image to resize in full screen mode, and now it works (even though disableObjectResizing=true)

Here is a screen shot: ​http://i.imgur.com/kUHHm6e.png

Thanks, Brandon

#10253 double alert in dialog confirmed Bug Normal
Description

in a Dialog, if we press Enter key instead of click 'OK' button, onOK will be trigger double times. for example, if we show image dialog, and leave url field empty. the press Enter key, the alert will show double times.

#10261 Walker documentation seems to be outdated confirmed Bug Normal
Description

​http://docs.ckeditor.com/#!/api/CKEDITOR.dom.walker

It says that walker may split nodes and the examples shows such a case. However, neither unit tests nor my manual tests proved that.

Source: ​http://stackoverflow.com/questions/15613082/ckeditor-walker-get-selection-only

#10269 Issue with ckeditor 4.0.2 running in a jquery dialog confirmed Bug Normal
Description

I posted the following in the forums on 3/13/2013:

We've been running ckeditor 3.6.4 in a jquery (1.7.2) dialog for the last year. We recently had to update to jquery version 1.9.1 and jquery ui version 1.10.1. In the process we also upgraded to ckeditor 4.0.2. We're still using frame editing so we wouldn't have to change the existing code.

We're now seeing the following issue with the ckeditor dropdowns:

In FireFox, if you click a dropdown, then click it again to close it, the dropdown remains but the option values are gone. Subsequently, the same dropdown or any other dropdown also open with no options displayed. In Chrome the behavior is virtually the same, just triggers differently - click the dropdown to open, click it to close, then click it again and it exhibits the same behavior. Using FireBug we get the following error: "TypeError: a.contentWindow is null", referencing line 324 in ckeditor.js. (standard version).

We also downloaded the source code (4.02 full version), created our own build, and got a different error:

"TypeError: $.contentWindow is null - return selected.$;"

I'm attaching a file that recreates the issue. It requires ckeditor 4.0.2 standard to be in the same directory to run. The file should also be renamed to 'index.html'.

Please let me know what we're doing wrong, or if this is a legitimate bug. We have a major launch this month and would like to get this resolved as quickly as possible.

Thanks in advance to anyone who helps.

Please have a look at comment:6 for possible solution and comment:11 for more details.

#10273 Finish integrating dialogs with Advanced Content Filter confirmed Task Normal
Description

Some dialogs are still not compatible - not allowed fields are displayed.

Already compatible:

  • image
  • link
  • div
  • iframe
  • flash

Incompatible:

  • tabletools
  • forms
  • liststyle
  • ?
#10279 4.0.x Autogrow in IE8/9 causes cursor to 'jump' confirmed Bug Normal
Description

When the autogrow plugin is enabled we're noticing an issue in IE8/9.

Steps to reproduce:

  1. Ensure autogrow plugin is enabled.
  2. Place cursor in editor and hit enter to make the editor grow

Once the cursor reaches the end of the available editor, every time the enter key is pressed the cursor moves to the next line, then immediately jumps back to its position in the prior line.

I know this isn't a ton to go on, and we also found it difficult to reproduce under certain circumstances. We did notice that this was an issue when IE went into compatibility mode, such as with an intranet site.

#10289 Incorrect del behaviour in table cell with list inside confirmed Bug Normal
Description

Paste this HTML to the editor: 

<table border="1" cellpadding="1" cellspacing="1" style="width:500px">
	<tbody>
		<tr>
			<td>
			<ul>
				<li>foo</li>
				<li>bar</li>
			</ul>
			</td>
			<td>OOO</td>
		</tr>
	</tbody>
</table>

Place caret after "bar". Press delete.

Expected: nothing happened

Actual: "OO" moved from second table cell to the first one and error on the console.

Reproducible on all browsers.

#10290 Language code with locale handling confirmed Bug Normal
Description

ckeditor does not handle language code as 'it-IT' or 'en-US', 'en-GB' etc etc

it should fall back to 'it', 'en' or whatever instead it does not recognize these language coeds

#10305 Strange jumps to the top in Firefox upon dialog closing (testcase included) confirmed Bug Normal
Description

Affects Firefox 20.0 (the latest version atm). Does not affect WebKit-based browsers.

Reproducable: Always.

Steps to reproduce:

  1. Open full-featured CKEditor, for example, ​http://ckeditor.com/demo#full
  2. Switch to source mode.
  3. Paste this code: 
    <p>Foo.</p>
<p>Foo.</p>
<p>Foo.</p>
<p>Foo.</p>
<p>Foo.</p>
<p>Foo.</p>
<p>Foo.</p>
<p>Foo.</p>
<p>Foo.</p>
<p>Foo.</p>
<p>Foo.</p>
<p>Foo.</p>
<p>Foo.</p>
<p>Foo.</p>
<p>Foo.</p>
<p>Foo.</p>
<p>Foo.</p>
<p>Foo.</p>
<div style="float:right">
<p><img alt="" src="http://b.cksource.com/a/1/img/sample.jpg" style="height: 200px;" /></p>
<p><img alt="" src="http://b.cksource.com/a/1/img/sample.jpg" style="height: 200px;" /></p>
<p><img alt="" src="http://b.cksource.com/a/1/img/sample.jpg" style="height: 200px;" /></p>
<p><img alt="" src="http://b.cksource.com/a/1/img/sample.jpg" style="height: 200px;" /></p>
</div>
  4. Switch to WYSiWYG mode.
  5. Scroll to the bottom, make sure you don't see any of the «Foo» labels in the viewport.
  6. Double-click on the last image. A dialog will appear.
  7. Press «OK» to apply and close the dialog.

What actually happens: The CKEditor viewport is instantly scrolled to top (jumps).

What should happen: The viewport should remain as it is. If the height is changed after editing the image, the top border of the visible area should remain the same, and the viewport should extend to the bottom (as it does when everything is fine).

Additional notes: Clicking the last image is not requred, you can use either one. This depends on if the text at the left (the «Foo» lines) is in the viewport. If you can still see the text (not scrolled down or the viewport is large enough), then the bug won't trigger.

#10306 Anchor allows spaces in id attribute. confirmed Bug Normal
Description

According to ​http://www.whatwg.org/specs/web-apps/current-work/multipage/elements.html#the-id-attribute - there can't be any spaces in id attribute.

If you put in editor code like: 

<p><a href="#first anchor">#first anchor</a></p>
<p>Lots of P</p>
<p><a id="first anchor" name="first anchor"></a></p>

it will work in e.g. Chrome or IE9. If you press preview button then click on link at the top you will see it works and browsers properly do the "jump" to the bottom.

If you put this code in HTML5 Validator you will get 2 messages:

  • "Bad value first anchor for attribute id on element a: An ID must not contain whitespace."
  • "Bad value #first anchor for attribute href on element a: Whitespace in fragment component. Use %20 in place of spaces."

I think that in this case first message is the one we should focus on and not allow entering spaces into anchor.

Problem can be reproduced from CKEditor 3.6.5 rev. [7618] when id for anchor was introduced.

I have written "if you paste such code" but same code is produced when you create anchor with dialog and then create link to anchor with another dialog.

#10312 Checkbox styling within a Dialog IE applied multiple times. confirmed Bug Normal
Description

When i add styling to a checkbox within a dialog, It is applied multiple times, To the text and to the Checkbox.

This happend in Internet Explorer 7

#10317 Double-click to initiate inline edit mode confirmed New Feature Normal
Description

There have been discussions about this in the forums with some plausible work-arounds but all seem a bit like a hack.

​http://ckeditor.com/forums/CKEditor/CKEditor-v4-Double-Click-to-Initiate-Editor

Would it be possible to add a configuration to override the default signle-click event to initiate inline edit mode?

#10319 Link Properties Dialog should search both Encoded and UTF-8 values confirmed Bug Normal
Description

Background:
It looks that new browsers handle UTF-8 characters in URLs. They show utf-8 characters to user but in background use encoded values to gets right URL.
​http://stackoverflow.com/questions/2742852/unicode-characters-in-urls
​http://stackoverflow.com/questions/6625035/utf-8-characters-in-urls[[BR]] ​http://en.wikipedia.org/wiki/Internationalized_domain_name
It also looks like that old or very old browsers handle encoded values only.

Real-life use case:
It is possible that application has to handle many types of browsers including old ones thus link created in CKEditor e.g. <a href="#első">line1</a> may get transformed to <a href="#els%C5%91">line1</a>. Href value may end up in URL thus it gets encoded on server-side

CKEditor issue: Beacuse of the above, code loaded into editor may look like: 

<p><a href="#els%C5%91">line1</a></p>
<p>many lines...</p>
<p><a id="első" name="első">line101</a></p>

If you doble-click on link, link properties dialog will show but value első will not be shown as selected in dropdown.

I think editor should look for both UTF-8 and encoded value (encodeURIComponent perhaps).
The same thing should be done when pressing ok in link properties dialog. It should be checked if current value is első or els%C5%91, next new value should be compared with current one and if they match (probably only if they match) current one should be left. Such check should prevent changing els%C5%91 to első but if application does such transformation on request basis then perhaps this isn't necessary in editor.

Problem can be reproduced from CKEditor 3.0 in both CKE 3.x and CKE 4.x (v4)

#10322 support for creating accessible data tables confirmed New Feature Normal
Description

We miss better support for creating accessible data tables i CKEditor. Would you consider distinguish between data and layout tables? (which are processed differently by assistive technologies such as screen readers). A Screen Reader Preview that helps authors further optimize content for accessibility would also be nice. Both functions are available in XStandard editor (We use this right now, but would like to shift to CKEditor)

#10326 Maximize button removes input[name=style] from form confirmed Bug Normal
Description

When clicking the maximize button if there is an element with the name of "style" it will be removed from the DOM.

After discovering the issue I downloaded the latest Zip from the website and modified the replacebycode.html (attached) file to confirm.

#10327 Under any circumstances filter should not be applied twice to one node assigned Piotrek Koszuliński Bug Normal
#10331 inline css images not displayed in Chrome confirmed Bug Normal
Description

I've noticed that background images in chrome are not displayed in CKEditor 4.1 if you enter them as absolute paths:

<div style="background-image: url('/images/test.jpg') => won't display on Chrome

BUT:

<div style="background-image: url('http:www.myserver.com/images/test.jpg') will display on Chrome.

It works on Firefox and Safari

AND: It also works on Chrome using CKEditor 3.x...

#10336 Filter should understand complex styles confirmed New Feature Normal
Description

Currently it won't create rules from following styles: 

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

fontSize_sizes: 'Smaller/FontSmaller;Larger/FontLarger;8pt/FontSmall;14pt/FontBig;Double Size/FontDouble',
fontSize_style: {
	element: 'span',
	attributes: { 'class': '#(size)' },
	overrides: [
		{
			element: 'span',
			attributes: {
				'class': /^Font(?:Smaller|Larger|Small|Big|Double)$/
			}
		}
	]
}

Perhaps this can be a part of #9991.

#10340 DTD caching makes it impossible to modify DTD even before first editor creation confirmed Bug Normal
Description 
CKEDITOR.dtd.customtag = { em:1 };
CKEDITOR.dtd.$block.customtag = 1;
CKEDITOR.dtd.body.customtag = 1;

Load: 

<customtag>foo</customtag>

It will be transformed to: 

<customtag></customtag>

<p>foo</p>

I think that we don't have to rewrite all code which caches DTD objects because that would ruin performance or increase complexity in some places. Therefore we can introduce event CKEDITOR#dtd fired when DTD is created and allowing its modifications.

Based on: ​http://stackoverflow.com/questions/16066556/ckeditor-how-to-allow-for-inserthtmlcustomtag-myattr-value-customtag

EDIT: The problem is DTD caching makes it impossible to extend it after loading CKEditor source.

#10344 build-config.js invalid to rebuild confirmed Bug Normal
Description

I noticed a problem using build-config.js to rebuild ckeditor.

I Always get an 'Invalid File' dialog: 'The uploaded file was invalid.'

So rebuilding according to a previous version of CKEditor is not possible.

Even with the content of the 'build-config.js' that is shown in the help e.g. ​https://github.com/ckeditor/ckeditor-dev/blob/master/dev/builder/build-config.js is not valid either!

#10351 Opera: Replace Textarea Elements by Class Name sample - CKEditor loses focus confirmed Bug Normal
Description

Steps:

Open sample attached (Replace Textarea Elements by Class Name) in Opera. Click at very beginning of the editor area.

Expected Result:

Console logs 'editor has focus'

Actual Result:

No logs in the console

By following the same steps in FF

Actual Result:

Console logs 'editor has focus'

Also, it is possible to change the CKEditor version in the sample from 4.1 (svn) to 3.6.6. Then we will get desired behaviour in all browsers.

To change the CKeditor version:

Uncomment this line <script src="​http://rev.ckeditor.com/ckeditor/trunk/7695/ckeditor.js"></script> to enable version 3.6.6 in the sample provided.

Comment out this line <script src="​http://nightly.ckeditor.com/13-04-19-07-42/full/ckeditor.js"></script> to disable version 4.1 in the sample provided.

#10355 Remove link-tag in Image-Dialog when Link is empty confirmed Bug Normal
Description

Currently when an Image nested within a link is edited and the link-url becomes empty the image afterwards still is wrapped in an empty <a> - Tag what maybe is not what should happen.

Reproduced on the demo-page, also on nightly, gives something like: 

<a href=""><img alt="Saturn V carrying Apollo 11" class="right" src="assets/sample.jpg" /></a>

I changed this behaviour in plugins/image/dialogs/image.js, ca. Line 397ff by removing the linkElement: 

	//Remove Link, Image exists.
	else if ( this.linkEditMode && !this.addLink ) {
		editor.getSelection().selectElement( this.linkElement );
		editor.insertElement( this.imageElement );
		this.linkElement.remove(true); // added
	}
#10356 "learn mode" for ACF confirmed New Feature Normal
Description

Many people will disable ACF because it modifies whatever content exists upon initial load if the classes, attributes, styles... haven't been added to the allowedContent.

A simple example is the stylesheet parser: you have been forced to make it disable the ACF in order to have things work.

Instead, the ACF could have an optional "learn mode" that adds whatever exists in the initial source to the allowedContent for that instance (maybe even configurable to work only on elements/classes/attributes/styles)

So now the stylesheet parser doesn't need to completely disable ACF: the first run of ACF sees some classes that aren't included in the ACF, so it adds those classes. Then the stylesheet is loaded and adds all the desired classes for the content. Nothing has been lost. The user then tries to paste something from an external page and the ACF is run and cleans up removing anything undesired. If the user copies and pastes something inside the editor, then all that content now is allowed and no filtering happens. everyone is happy.

#10358 [IE]:Editor removes &nbsp; if after &shy; confirmed Bug Normal
Description
  1. Go to any page in editor and clear editor contents (new page command)
  2. Switch to source and insert <p>&shy;&nbsp;</p>
  3. Switch to WYSIWYG and to Source again

Result: <p>&shy;</p> non-breaking spaced is removed.

Problem occurs in all versions of IE from CKEditor 4.0 (it works in CKE 3.0 - 4.0 beta).

#10359 [Webkit] Extra &nbsp; and inline styles when copying and pasting aligned text confirmed Bug Normal
Description

Follow the cases below. Set initial data on the editor, then C-a, C-c, C-v and get data.

  • Tested in latest Chrome (26.0.1410.63), Webkit 537.31
  • Note that &nbsp;s are displayed always regardless of the alignment.
  • Related ticket: #9998

replacebycode (extra &nbsp;, inline style on <a>)

Initial data: 

<p style="text-align:right">This is some <strong>sample text</strong>. You are using <a href="http://ckeditor.com/">CKEditor</a>.</p>

After copy and paste: 

<p style="text-align:right">This is some&nbsp;<strong>sample&nbsp;text</strong>. You are using&nbsp;<a href="http://ckeditor.com/" style="text-align: right;">CKEditor</a>.</p>

outputhtml (extra &nbsp;)

Initial data: 

<p align="right">This is some <b>sample text</b>. You are using <a href="http://ckeditor.com/">CKEditor</a>.</p>

After copy and paste: 

<p align="right">This is some&nbsp;<b>sample text</b>. You are using&nbsp;<a href="http://ckeditor.com/">CKEditor</a>.</p>
#10362 Introduce editor#feature event confirmed New Feature Normal
Description

Based on: ​http://ckeditor.com/forums/CKEditor/How-to-remove-attributes-which-are-removed-from-dialog-windows

It is not possible to modify feature's properties in convenient way or in some cases at all. Editor#feature should be fired when feature is registered by filter#addFeature, but before its rules/transformations/etc are registered.

This should be possible: 

CKEDITOR.replace( 'editor1', {
	on: {
		feature: function( evt ) {
			if ( evt.data.name == 'table' )
				evt.data.allowedContent = 'table[border,summary];caption tbody thead tfoot;th td tr[scope]';
		}
	}
} );
#10369 IE9 ENTER_BR problem with AUTOGROW plugin confirmed Bug Normal
Description

Steps to reproduce:

  1. Download CKEditor by CKBuilder with default settings, but
  • Just adding Autogrow plugin to selected
  1. Launching /ckeditor/samples/plugins/enterkey/enterkey.html
  2. Choosing "When Enter is pressed : Break the line with a <BR>"
  3. Put cursor at the end of text
  4. Press "Enter"
  • Expected Result: Cursor moved to the next line;
  • Actual Result: Cursor stays at the same line
#10371 Insert HTML space bug IE confirmed Bug Normal
Description

if you insert spaces in the overrided insertHTML text mode, it gives a error. This happens in Internet Explorer 7

code: 

CKEDITOR.instances["yourInstance"].insertHtml(" ", 'text');

ERROR: 

  'isBlock' is null or isn't a object  ckeditor.js, Line 287 Character 56
#10372 Paste as plain text do not work confirmed New Feature Normal
Description

The bug is reproducible at the demo page: ​http://ckeditor.com/demo#full.

Copy any <br>-splitted text to clipboard. I use text from the thunderbird mail client. The source code of an email shows something like this: 

<br/>
  TEXTTEXT  TEXTTEXT  TEXTTEXT  TEXTTEXT<br/>
<br/>
  TEXTTEXT  TEXTTEXT  TEXTTEXT  TEXTTEXT  TEXTTEXT<br/>
<br/>
<br/>

So when I copy the text to the clipboard(not the source code). When I paste it to CKEditor with Ctrl + V, and open source code, i see the same <br> formatting.

When I use Ctrl + Shift + V to paste the text, the source code shows <p>-formatted: 

<p>
  TEXTTEXT  TEXTTEXT  TEXTTEXT  TEXTTEXT
</p>
<p>
  TEXTTEXT  TEXTTEXT  TEXTTEXT  TEXTTEXT  TEXTTEXT
</p>

So I expect the button "Paste as plain text" and pastetext plugin would work the same way, but it looks like it doesn't.

I tried to set config.forcePasteAsPlainText = true; but the behaviour is the same.

I allowed javascript to access the clipboard, but the "Paste as plain text" button works the same way as regular Paste button.

I expected that setting forcePasteAsPlainText to true will force Ctrl + V to work like Ctrl + Shift + V, but it doesn't.

We use latest CKEditor release 4.1.1.

Tested at Latest FF, Chrome and Opera.

#10375 table cell horizontal split error confirmed Bug Normal
Description

The far right of the back of the cell in the table to insert a new cell, horizontal split the cell error

#10378 IE: empty lines removed from list on copy/paste review_failed Piotr Jasiun Bug Normal
Description
  1. Clear editor contents with new page
  2. Press Bulleted list button
  3. Type test
  4. Press Shift+Enter
  5. Press Enter
  6. Repeat steps 3-5 few times.
  7. Press Ctrl+A and Ctrl+C
  8. Clear editor contents with New Page command
  9. Press Ctrl+V

Result: &nbsp; and br tags are lost thus new lines get removed.

Problem can be reproduced from CKEditor 3.5.3 rev [6624].

Other tickets caused by this revision: #8743, #8140

#10383 IE10 crashes after deleting certain content assigned Piotr Jasiun Bug Normal
Description
  • Open source code view
  • Insert the attached html code snippet
  • Switch back to wysiwyg view
  • Select all content
  • Press 'Del' key

IE10 crashes, IE9 is fine. This is also reproducible with the current demo.

#10388 Deleting a sub bullet, deletes entire bullet list confirmed Bug Normal
Description

Steps to reproduce:

1) Create a Bullet list

2) Add a Sub bullet

3) Select the sub bullet and press backspace

Entire bullet list gets removed

#10391 Image upload in data-uri base64 doesn't work with large files confirmed Bug Normal
Description

Hello,

I am trying to use ckeditor's image plugin and I have encountered some issues with large images.

First of all, I have a java servlet that is mapped to the filebrowserUploadUrl. This servlet doesn't save the file locally, it only reads the binary data, converts it to base64 and create a "data-uri" link then calls the right function of ckeditor.

Anyway, it works well with small images (<500ko) and for bigger ones I have got a red X image and : "Failed to load resource" in my Javascript console.

It is definitly not a server issue, the post request works fine, I can get the data back and call ckeditor internal function. The error seems to occur in element.js line 19 : 

setAttribute : (function()
    {
	var standard = function( name, value )
            {
		this.$.setAttribute( name, value ); //here!
		return this;
	    };
//...

I have tried to debug in Chrome and the value contains the right data-uri (however I can't see it entirely..)

I have also another error. Sometimes (I really don't understand how), The image is well read and displayed in the dialog. And when I try to add it (by pressing the "Ok" button), I get an alert "Image source URL is missing.".

It seems that this last error happens in more often in Firefox and the first one in chrome..

If I try to insert the same images as data-uri in javascript outside CKeditor, it works perfectly.

This never happens with small images.

I hope this is clear enough. Thanks in advance,

#10392 Dropdowns don't scroll to top when opened. confirmed Bug Normal
Description

To reproduce:

  1. Open font menu, scroll down half way and then click away (don't select anything) so it closes
  2. Now click the size menu.

Result: It starts scrolled down rather than back at the top.

This is happening because we use 1 float panel for dropdowns and don't reset on every open.
To be honest I have never considered this as a bug. Very common situation is when you try to select something from e.g. styles dropdown and you didn't click what you wanted. If you open dropdown again you will see dropdown in same area where you have finished. You don't have to scroll all the way down what may be frustrating to some (especially when you didn’t click what you wanted for the second time).

Sure when opening new dropdown user would expect it to be at the top. Since we are using one float panel we can either live with it or perhaps remember last dropdown opened and if current dropdown is different scroll it up.

For those who want to have float panel always scrolled up please use below code: 

var editor = CKEDITOR.replace( 'editor1', {} );	
editor.on( 'panelShow', function( ev )	{				
	ev.data._.iframe.$.contentWindow.scrollTo(0,0);				
});

Another solution would be adding onOpen methods to below plugins (e.g. after onRender): \ckeditor4-git\plugins\format\plugin.js
\ckeditor4-git\plugins\font\plugin.js
\ckeditor4-git\plugins\stylescombo\plugin.js 

onOpen : function(){
this._.panel._.iframe.$.contentWindow.scrollTo(0,0);
},
← 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 →
Note: See TracQuery for help on using queries.
© 2003 – 2022, CKSource sp. z o.o. sp.k. All rights reserved. | Terms of use | Privacy policy