Custom Query

Filters
 
Or
 
  
 
Columns

Show under each result:

Results (1501 - 1600 of 2591)

← 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 →
Ticket Summary Status Owner Type Priority Milestone
#11624 toolbarGroups - impossible to remove subgroup that has the same name as group confirmed Bug Normal
Description

Default configuration: 

CKEDITOR.replace( 'editor1', {
	toolbarGroups : [
		{ name: 'document',    groups: [ 'mode', 'document', 'doctools' ] },
		{ name: 'clipboard',   groups: [ 'clipboard', 'undo' ] }, // <----
		{ name: 'editing',     groups: [ 'find', 'selection', 'spellchecker' ] },
		{ name: 'forms' },
		'/',
		{ name: 'basicstyles', groups: [ 'basicstyles', 'cleanup' ] },
		{ name: 'paragraph',   groups: [ 'list', 'indent', 'blocks', 'align', 'bidi' ] },
		{ name: 'links' },
		{ name: 'insert' },
		'/',
		{ name: 'styles' },
		{ name: 'colors' },
		{ name: 'tools' },
		{ name: 'others' },
		{ name: 'about' }
	]
} );

Undo subgroup removed (undo/redo buttons are gone): 

CKEDITOR.replace( 'editor1', {
	toolbarGroups : [
		{ name: 'document',    groups: [ 'mode', 'document', 'doctools' ] },
		{ name: 'clipboard',   groups: [ 'clipboard' ] }, // <----
		{ name: 'editing',     groups: [ 'find', 'selection', 'spellchecker' ] },
		{ name: 'forms' },
		'/',
		{ name: 'basicstyles', groups: [ 'basicstyles', 'cleanup' ] },
		{ name: 'paragraph',   groups: [ 'list', 'indent', 'blocks', 'align', 'bidi' ] },
		{ name: 'links' },
		{ name: 'insert' },
		'/',
		{ name: 'styles' },
		{ name: 'colors' },
		{ name: 'tools' },
		{ name: 'others' },
		{ name: 'about' }
	]
} );

Clipboard subgroup removed (clipboard buttons are still available): 

CKEDITOR.replace( 'editor1', {
	toolbarGroups : [
		{ name: 'document',    groups: [ 'mode', 'document', 'doctools' ] },
		{ name: 'clipboard',   groups: [ 'undo' ] }, // <----
		{ name: 'editing',     groups: [ 'find', 'selection', 'spellchecker' ] },
		{ name: 'forms' },
		'/',
		{ name: 'basicstyles', groups: [ 'basicstyles', 'cleanup' ] },
		{ name: 'paragraph',   groups: [ 'list', 'indent', 'blocks', 'align', 'bidi' ] },
		{ name: 'links' },
		{ name: 'insert' },
		'/',
		{ name: 'styles' },
		{ name: 'colors' },
		{ name: 'tools' },
		{ name: 'others' },
		{ name: 'about' }
	]
} );
#11638 Inserted text blinks after multiple insertions using ctrl-v confirmed Bug Normal
Description

After multiple insertions some text using ctrl-v, inserted text blinks in the left of the screen. It is reproduced with presented html structure and css.

Browser: Chrome, 33.0

Html structure:

<body>
    <div class="container">
        <div class="cke-holder">
            <div class="toolbar-ckeditor" id="toolbar">
                <!-- This div will handle top toolbars -->
            </div>
            <div id="ck-field" class="inline-ckeditor-default"></div>
        </div>
    </div>
</body>

Div with id="ck-field" initialized as inline ckeditor.
css: 

body {min-width: 960px;}
.container {width: 960px;margin: 0 auto;position: relative;}

This can be fixed by adding "position: relative;" style for div initialized as ckeditor.

#11639 Image2's resizer is displayed far from image if image has a margin confirmed Bug Low
Description

Add to contents styles: 

img { margin: 10px }

And the resizer will be displayed 10px from its correct position. Can we do anything about it other than resetting that style?

I think that the only solution is to change the resizer position if margin was discovered, but that's not even close to a clean solution.

On the other hand, if I set padding instead of margin, then resizer also isn't displayed over the image. But in this case it's still displayed over the widget outline (so it looks good), because the outline is also pushed from image. So theoretically we could add border or ~0px padding to widget wrapper, so it'd be pushed from image if it has margin, but then we would break margins collapsing between image and surrounding elements, what's not acceptable.

#11655 editor.config.toolbarLocation='bottom' does not work confirmed New Feature Normal
Description

Due to the implementation with the "floating" toolbar (depending on the scroll position/available space of the page), it is not possible to fix the location of the toolbar. It'd be great to give a preferred location of the toolbar ("put it to the bottom unless there is no space").

Please see my attached plugins/floatingspace/plugin.js with my temporary workaround to get an idea of location of the problem.

#11668 delete key behavior is WRONG!! confirmed Bug Normal
Description

EDIT:

  1. Insert below code into editor: 
    <p>test <strong>test <em>test </em>test</strong> test test</p>
  2. Switch to wysiwyg and put cursor behind italic element. Please note that element path needs to show body p strong. If you are seeing em move curosor with arrows to get out of em element. 
    test test test ^test test test
  3. Start deleting text with delete key

Result: once whole bold text is deleted, the italic element gets deleted (except for the first letter)

hi all; I newly found key delete works wrong!!! for example, we have an bold element in a paragraph and in the bold element there is an italic word. if we put cursor after italic element exactly, and begin pressing delete key, the characters of bold text node which is after italic element, delete one by one. when the text node deleted and italic element was placed in end of bold element and cursor was placed in bold element and exactly after italic, pressing delete key will caused all of italic characters except one char, be remove!!! that is WRONG!!!

  1. http://upload7.ir/imgs/2014-03/22805295702919488848.png
  2. http://upload7.ir/imgs/2014-03/03656317128125272710.png
  3. http://upload7.ir/imgs/2014-03/11671183618270822326.png
  4. http://upload7.ir/imgs/2014-03/21996411578075141160.png

True : In these circumstances, characters of next text node from next element must be remove. I checked this, from 4.3.3 to 3.0 versions and all of them have this bug. please solve it. Very Thanks.

#11671 [IE 8-10] inserttext space as first call on editor problem confirmed Bug Normal
Description

please take a look at the following JSFiddle: ​http://jsfiddle.net/5zx3B/1/ in IE10

click on button: "space"

click on button: "letter A"

result: "A"

expected result: " A"

Take an empty editor. If you call insertText(" ") the space is displayed. If you call insertText("A") the space is removed and the "A" is added.

#11673 Builder ignores languages set for plugins confirmed Bug Normal
Description

CKBuilder ignores languages set in CKBUILDER_CONFIG when handles plugins. All language files gets into package.

#11674 Tableresize - allow percent-based values confirmed New Feature Normal
Description

It would be nice if tableresize allowed to set column widths in percent instead of pixels. Of course, it should be configurable.

#11679 Color buttons can't be customized confirmed Bug Normal
Description
  1. Insert below code on page and into contents.css 
    var editor = CKEDITOR.replace( 'editor1', {		
				//allowedContent : true,
				colorButton_colors : 'highlight_yellow/FFFF00',
				colorButton_backStyle : {
					element : 'span',
					attributes : {
						'class' : '#(color)'
					}
				}
});


//in contents.css
.highlight_yellow{
	background-color: #FFFF00;
}
  2. Refresh the cache.
  3. Highlight the word and change its color to Yellow.

Results:

  1. If you use custom color for which translation is not defined in language file e.g. 'highlight_yellow/FFFF00' then you will see green square in color button dropdown panel but its label is 'FFFF00'. Maybe I'm wrong here but I would expect label to be displayed as 'highlight_yellow'.
  1. The 'highlight_yellow' is used for styling text you select but it has no effect. Such style <span style="color:highlight_yellow">item 2</span> is not recognized thus it looks like text is not styled.
  1. With ACF enabled, once you switch to source and back, span will be removed and it shouldn't be. This part is reproducible from CKEditor 4.1.
  1. You can't change this color to automatic with such configuration. This part can be reproduced from CKEditor 3.0.
#11686 [Webkit] "Delete Columns" sometimes removes two columns instead of one confirmed Bug Normal
Description
  • Create 3x3 table and type any char in the first cell
  • Double click to the right of the char (the char should not be selected, but cell will. See video: ​http://www.screencast.com/t/XIGCstjiDti ). Or you can use Shift+Right arrow key.
  • Right click -> Column -> Delete Columns

Outcome: the first and the second columns will be deleted
Expected: only the first column deleted

#11687 [FF] Caret position reset when clicking editable confirmed Bug Normal
Description

Clicking text in editable moves caret to very beginning, which is extremely annoying if you want i.e. select something in order to bold it.

  1. open sample with image2 plugin (/samples/plugins/image2/image2.html)
  2. put caret at following position Roll out of |Saturn V on launch pad in first widget caption
    note first selection is OK
  3. move caret to other element (by clicking), i.e. "Apollo 11" headline
  4. repeat 2nd step

Expected result:
Caret should be placed in of |Saturn

Current result:
Caret is placed at very beginning of the caption.

additional info:

  1. Bug occurs only if focus is moved from within the editor. That means, if you'll blur editor by i.e. clicking at sample headline "CKEditor Samples » New Image plugin", and then you'll click at editable - bug does not occur.
#11690 Placeholder with forbidden characters confirmed Bug Normal
Description
  1. Open placeholder sample (plugins/placeholder/samples/placeholder.html).
  2. In the source mode enter: 
    <p>This is a [[sample ]placeholder]].</p>
  3. Switch back to the WYSIWYG mode.

Result: there is no placeholder.

  1. In the source mode enter: 
    <p>This is a [[sample >placeholder]].</p>
  2. Switch back to the WYSIWYG mode.

Result: there is a placeholder.

  1. Open placeholder dialog.
  2. Press "Ok".

Result: Error message:

The placeholder can not be empty and can not contain any of following characters: [, ], <, >

On the one hand it is not possible to upcast placeholder with ']' character, because it is forbidden, but on the other we can do it with '>'.

In my opinion it should not be possible to upcast placeholder with forbidden character but also it would be good if user would be informed that the placeholder contains forbidden character.

#11691 [IE8] Can not expand selection when caret is at the end of a inline element confirmed Bug Normal
Description

checked only on IE8, but this issue might be also present in other IE versions

  1. open any sample with CKEditor (i.e. samples/replacebyclass.html)
  2. using "Source" button set content to following markup: 
    <p>aa bb<u> cc </u>dd ee</p>
  3. go back to wysiwyg mode
  4. place caret in following position aa bb cc |dd ee
  5. press ctrl + shift + right arrow

Expected result:
Selection should be expanded to contain dd string.

Current result:
Selection is not expanded.

additional info:

  1. i've tested it with bold, italic - they have the same issue too
#11692 [IE9-10] Home and end buttons in inputs move cursor to the wrong possition confirmed Bug Normal
Description
  1. Open Replace by Class sample.
  2. Open Link dialog.
  3. Paste long link into URL field (longer than text field width) like: 
    ckeditor.com#Lorem-ipsum-dolor-sit-amet-consectetur-adipiscing-elit-Nam-vel-lectus-varius
  4. press "Home" button

Result: there is a space between cursor and text.

The same issue for 'end' button. I was able to reproduce it on IE9 and IE10 with all of the text fields. On IE8, IE11 and Chrome everything is fine.

#11696 undo link scrolls to top of page confirmed Bug Normal
Description
  1. Go to full featured DEMO page (I used IE9).
  2. Select the whole content and scroll to bottom of the page and paste. This is just to let enough content to demonstrate the scrolling.
  3. Insert a picture to the bottom of the page.
  4. Select the picture and press link tool. So give it a link eg. ​http://google.com. The link appears on the picture. > OK
  5. Press the undo button. The page scrolls to the top.

Not an issue if you put a link on a word at the bottom of the page but using a picture for a link.

#11700 Bringing accessibility support for widgets confirmed Bug Normal
Description

We should think about providing good a11y for widget.

Currently screen readers treats every widget as the end of an element. We need to do far better than that.

The most important requirements i see at the moment are:

  • labels for widgets
  • labels for editables

labels for widgets

  • Should be context sensitive, by that i mean they should be diffrent for image, placeholder, mathjax and codesnippet
  • Label should be also possible to fetch some information from widget instance itself, i.e. image should be have its alt attribute included into label
  • Should be specified in widget definition

labels for editables

Here i have no clear conception as of yet, because you're only able to access editable using the tab key, but it iterates from the very beginning of the document, rather than current caret position.

Currently 2 solutions come to my mind:

Solution 1

  • leave current keyboard as it is
  • upon getting into editable, user should be informed what widget editable belongs to

Solution 2

  • Allow only to enter into editable (with tab key) only when widget is focused
  • We don't need to inform our end-user what widget he's in (that reduces extra time spent on listening). The only one information he will need is the name of editable itself
  • Important implementation detail would be to allow focus cycling inside widget
#11701 Richcombo breaks on HTML entities in HTML formatted values confirmed Bug Normal
Description
  • configure an editor with latest strinsert addon from ​https://github.com/57u/custom-dropdown-ckeditor4/commit/32bb75a5fc5129722ddd32637e8639be2735cbb5 (​http://ckeditor.com/addon/strinsert is not up to date) using following settings: 
    config.extraPlugins = 'strinsert';
config.strinsert_strings = [{'name': 'test', 'value': 'html entities like &lt;, &gt and &amp; wont work'}];
  • open the editor, select 'test' from the 'Insert' dropdown in the toolbar and you'll get a JavaScript error like:
    • Chrome: 
      Uncaught TypeError: Cannot call method 'hasClass' of null ckeditor.js:746
Uncaught TypeError: Cannot call method 'hasClass' of null ckeditor.js:746
CKEDITOR.ui.listBlock.CKEDITOR.tools.createClass.proto.isMarked ckeditor.js:746
CKEDITOR.ui.listBlock.CKEDITOR.tools.createClass.proto.toggle ckeditor.js:744
(anonymous function) ckeditor.js:742
(anonymous function) ckeditor.js:27
CKEDITOR.tools.callFunction ckeditor.js:27
onclick
    • IE: 
      Error: Unable to get property 'hasClass' of undefined or null reference

Edit: Please see comment:5 for quick summary :)

#11708 Magic-Line showing in email confirmed Bug Normal
Description

Magic line is a great tool for creating new paragraphs into spaces where normally would be impossible to reach.But we have found an issue associated to it.

Issue: We have been using CKeditor for designing emails. We have seen some red line remain in final email we design. This issue we have seen to be happening from IE 10. Steps to reproduce:

  1. Place your cursor such that Magic-Line is visible in ckeditor.
  2. Press Ctrl+A then Ctrl+C.
  3. Paste the content you will see that Magic-Line is also copied with other content.

Please see attachment.

Any help will be admirable. thanks

#11720 Method insertElement causes error in IE if editable hasn't been yet focused confirmed Bug Normal
Description

Insert below code into replacebycode sapmle: 

editor.on('instanceReady', function(){
	var elem = new CKEDITOR.dom.element( 'pre' );
	editor.insertElement(elem); //error in IE
	//editor.editable().append(elem); //works
});

When you load the page you will get:
Message: 'undefined' is empty or not an object
Line: 299
URI: /ckeditor4_git/core/editable.js

Problem can be reproduced from CKEditor 4.3 in all versions of IE.

#11721 [iOS] Dialog in the wrong position confirmed Bug Normal
Description

Tested on the Safari (537.51.2) on iOS 7.1 (iPad).

Dialogs are in the correct position as long as I'm not using zoom.

#11729 [iOS] Magicline does not work confirmed Bug Normal
Description

Tested with Safari (537.51.2) on iOS 7.1 (iPad).

Magicline does not work on Safari on iOS.

It could works as it works on Chrome on Android so show the magicline when cursor is just before or after the position of the magicline. Or magicline could be shown when user tap on the magicline position. There are solutions.

#11730 [iOS] Editor is scrolling when command is execute confirmed Bug Normal
Description

Tested with Safari (537.51.2) on iOS 7.1 (iPad), CKEditor 4.3.4.

When I apply any command page scoll down so toolbar is above the viewport.

#11731 [iOS] Dialogs move selection to the begging of the document confirmed Bug Normal
Description

Tested with Safari (537.51.2) on iOS 7.1 (iPad), CKEditor 4.3.4.

When I try to insert a content using dialog (link, special character, smiley) cursor is moved to the begging of the document.

#11732 [iOS] It's not possible to switch to source and back confirmed Bug Normal
Description

Tested with Safari (537.51.2) on iOS 7.1 (iPad), CKEditor 4.3.4.

  1. Open "Replace by Class" sample.
  2. Press "Source" button.
  3. Press "Source" button again.

Expected: editor will switch back to the wysiwyg mode.

Result: nothing happens.

I can go back to the wysiwyg mode if I move the focus to the source textarea.

#11733 [iOS] It is not possible to open Image2 edit dialog. confirmed Bug Normal
Description

Tested with Safari (537.51.2) on iOS 7.1 (iPad), CKEditor 4.3.4.

  1. One Image2 sample.
  2. Tap on the image.
  3. Press image button in the toolbar.

Expected: Image edit dialog will be shown.

Result: Empty dialog is shown.

#11734 [iOS] Native context ballon options does not work with ACF and undo confirmed Bug Normal
Description

Tested with Safari (537.51.2) on iOS 7.1 (iPad), CKEditor 4.3.4.

Using build-in format tools I'm able to execute commands which are forbidden by ACF (see attachment). Also modification made by this options are on recorded by undo manager.

#11735 Adding a tab support inside codesnippet dialog confirmed New Feature Normal
Description

Follow-up of #11480.

We should add some support for tab key inside code-edit textarea of codesnippet dialog. We need to handle following keys:

  1. Tab - should insert a tabulation character
  2. ctrl + . (period) - should move focus to next focusable item in dialog

Note: that shift + tab should remain untouched.

We're considering 2 solutions:

  1. Handling it completly by ourself (leftovers are in ​t/11480b branch)
  2. Use external library, like ​Behave.js
  3. Or use full code editor.
#11743 Exception flood caused by calling nativeSelection.createRange() confirmed Bug Normal
Description

Steps to reproduce in Google Chrome, but should be relevant for any browser: Open browser builtin Developer Tools, set "pause on exception" option. You get a series of debugger pauses every time you try to focus on CKEditor.

This issue makes a debugging difficult. Additionally, this is a waist of processor ticks :) .

#11745 Maximize should use position:fixed instead of changing entire page styling confirmed Task Normal
Description

Follow up of ​http://dev.ckeditor.com/ticket/8587#comment:8

Using position:fixed for the editor chrome may be beneficial for us because it will need less code, there won't be a problem with input names, it may be faster and more stable. Additionally, we may check if there are other ways to remove scrolls from the viewport which will not cause so much damage underneath the editor.

#11750 Iframe Dialog Scrollbar problem. confirmed Bug Normal
Description

Problem can be reproduced in IE8-10 (works in IE11).

  1. Add attached insertVariable plugin (best in CKEditor dev)
  2. Add below code in HTML page: 
    CKEDITOR.replace( 'editor1', {
	extraPlugins: 'insertVariable'
});
  3. Load the page and open insertVariable Dialog.

Result: in IE8-10 there is always vertical scrollbar. This is happening because below TD element exceeds size of wrapper div: 

<td class="cke_dialog_ui_vbox_child" role="presentation" style="width: 100%; height: 100%;">

<div class="cke_dialog_ui_vbox cke_dialog_page_contents" id="cke_111_uiElement" role="tabpanel" aria-hidden="false" aria-labelledby="cke_iframe_112" style="width: 100%; height: 100%;" name="iframe">

This scrolbarr is not a big issue with small content but you you use large contant you will get double vertical scrollbar (Please see attached iframeDialog.png)

One solution to this problem is adding below rule in ckeditor/skins/moono/dialog_ie.css: 

.cke_dialog_page_contents {
	overflow:hidden;
}

The above rule solves the problem (please note that kama skin uses this class in dialog_iequirks.css) but if someone uses ony iframe in his dialog it would be nice idea to to have paddings removed on cke_dialog_contents_body. Perhaps we could introduce separate class for iframeDialog and 'contents body' or simply modify both?

#11755 Styles dropdown not updated after object style change confirmed Bug Normal
Description
  1. Editor styles.js so Styled image names are shorter (to make them visible in the button.
  2. Load <p><img alt="" src="assets/sample.jpg" /></p>.
  3. Select image and apply one of the styles.
  4. See that the styles dropdown still shows "Styles" instead of the applied style name.
  5. Deselect and select image again - styles dropdown shows the applied style name.

Reproduced on master and major. Checked Firefox and Chrome.

#11759 Introduce a "split button" UI component confirmed New Feature Normal
Description

It would be suit color buttons, paste button, table button etc.

#11761 Event system dies along with the last editor being destroyed confirmed Bug Normal
Description

I stumbled upon this issue while developing sample for #11480.

This issue makes CKEditor events API quite useless without editor instance, especially if the editor is destroyed in the callback.

See ​jsFiddle.

How to reproduce?

  1. Create editor instance.
  2. Attach event listener on a DOM element outside of the editor. Not related to editable, etc.
  3. See that event works.
  4. Destroy the editor(s).

Actual:

  1. Attached listener is not fired anymore.

Expected:

  1. Attached listener is fired since the object in DOM is still there.

Quick research:

#11762 Styles are not applied in Internet Explorer if backspace is the last key pressed confirmed Bug Normal
Description

Steps to Reproduce

  1. Open the example document in the nightly test page ​http://nightly.ckeditor.com/14-04-03-06-05/standard/samples/replacebyclass.html
  2. Place your cursor in some text that is not styled, for example the end of surface in "lunar surface"
  3. Use the backspace key to delete the words "lunar surface"
  4. Hit Ctrl+B to enable bold
  5. Type the word "moon"

Expected result

...to step onto the moon...

Actual result

...to step onto the moon...

This happens with bold, italic, and underline and occurs with both the keyboard shortcuts and the toolbar buttons. It also occurs if you use multiple backspace presses or highlight the text you want to remove and press backspace once.

Browser

I can reproduce this on Internet Explorer 9, 10, and 11, it does occur on Internet Explorer 8 or Chrome/Firefox

#11765 Editor does not show in divreplace sample, when clicked between paragraphs. confirmed Bug Normal
Description

Div is not replaced with editor when you'll click inbetween paragraphs. It's minor issue but can mess up user UX, if he'll click in such place for the very first time.

  1. open divreplace sample (​divreplace)
  2. click on space between two paragraphs

Expected result:
Editor should be created.

Current result:
Nothing happens.

additional info:

The reason for this issue is iterating element.parentNode from the very first iteration. If you click in position given above, the ev.target
ev.srcElement will point to div (with class editable). Therefore first iteration will check against body, rather than div. Code is located in samples/divreplace.html.
#11767 Table Tools spanning not updated confirmed Bug Normal
Description

Testcase:

Use IE or FF to be able to easily visualize the issue, in the source its broken for all browsers thogh

  1. Go on ​http://ckeditor.com/demo
  2. Create a table with 4x4
  3. select Cell 1x1 and set row-spanning to 10
  4. You now have a broken spanning since it is not validated

A more common use-case for normal users would be

  1. Go on ​http://ckeditor.com/demo
  2. Create a table with 4x4
  3. 3. select Cell 1x1 and set row-spanning to 10
  4. Delete a row -> spanning is not set to 9, instead of getting recoundet and fixed

Basically there is no sense in not validating the data since this is an editor for a user, not developer. In a table with a lot of rows you can easily count wrong and end up having a broken table.

Solution:

a) caluculate max span for col/rowspan before entering the dialog and offer a select-widget rather then an input field b) validate on save

You could also reuse such a table validator: ​http://jsfiddle.net/UkV35/8/

#11768 Merging table-cells using the cell-properties dialog is completely broken confirmed Bug Normal
Description

Reproduce by:

  1. ​http://ckeditor.com/demo#standard
  2. insert a 3x3 table
  3. select cel 1x1
  4. right click -> cell -> cell properties
  5. input 2 in "merge row cells"
  6. submit the dialog

Problem: Everytime you do this, no matter which values you choose for col/rowspan, the complete table-layout is broken, respectivly an extra broken column is inserted every time.

#11770 Destroying an editor while a widget is selected causes JS error in IE confirmed Bug Normal
Description

I am using IE 11 and an inline editor. If you add a widget, such as the demo simplebox widget, and while it is selected you destroy the editor (such as the user clicking a cancel button that removes the editor function) there is a JS error.

The JS error occurs while trying to remove expand attributes. In plugin.js on line 211 it calls contents.replace( /\s+data-cke-expando=".*?"/g, ) but in this situation contents is a Boolean value equal to true so it throws the exception that there is no replace function for that value.

It appears that the code expects editor the getSnapshot to return a string but a Boolean is being returned instead. Looking at event.js it appears true is returned when an event call returns undefined.

#11771 Introduce styleableElement in widget API confirmed Bug Normal
Description

At the moment (#11297) to change the way widgets are styled, developer must override widget#add/has/getClasses, which is not quite straightforward.

The idea is to introduce widget#styleableElement property which, referring to the right element, can be re-used in widget#add/has/getClass and reduce the complexity of customisation.

Problems:

  • How to set it? Via widgetDef (selector, method...)? Via widget.setStyleableElement? By overriding widget.getStyleableElement?
  • How to alter it? delete widget.styleableElement? widget.setStyleableElement? By overriding widget.getStyleableElement?
  • How to optimise it? Cache? How to override cached element?
#11772 [Inline] Format drop down shows that selection is in a div when image2 is focused confirmed Bug Normal
Description
  1. Open plugins/image2/dev/image2.html
  2. Focus captioned image. In framed editor format shows that none of the formats is used, in inline editor format shows "Normal (DIV)".
#11773 Hierarchy request script error confirmed Bug Normal
Description
  1. Using IE11 go to the full featured DEMO page and make a new (empty) page.
  2. Open the attached page and copy/paste its content to the DEMO page.
  3. Find the following list item: "1. this list". This is the third line from the bottom.
  4. Place the caret to the end of this line and press DELETE.

RESULT: You get Hierarchy request script error.

Note: IE9 will not throw error but the cursor will be dropped to an unexpected place.

#11775 Codesnippet plugin should allow to specify custom path for highlight.js confirmed New Feature Normal
Description

Plugin should allow to use other highlight.js path than the default one. It will allow developer to:

  • link it against more convenient path, where it's already used for website front-end - thus he might update highlight.js in one place
  • link it to other cdn
#11778 IE11: The xml object loaded with Ajax plugin fails to find children confirmed Bug Normal
Description

Create a "test.xml" file with something like this: 

<?xml version="1.0" encoding="utf-8" ?><Templates><Template>content</Template></Templates>

Add this to a page with CKEditor: 

CKEDITOR.on('instanceReady', function(e) {
	CKEDITOR.ajax.loadXml( "test.xml", function(oXml) {
		var child = oXml.selectSingleNode( 'Templates' );
		if (!child)
			alert("Failed, the Templates node hasn't been found");
		else
			alert("XML successful");

	});
});

Now when the page is loaded IE11 will state that the child hasn't been found This can be prevented by using the XML code found in CKFinder (in theory it was added for Android, but it turns out that it also works here).

#11779 Scrolling issues in modern browsers when document is in Quirks Mode confirmed Bug Normal
Description

Steps to Reproduce

Standards mode Example: ​http://jsfiddle.net/SdL9u

Quirks mode Example: ​http://jsfiddle.net/SdL9u/1/

  1. Open the sample pages linked above in Internet Explorer 11, Firefox, or Chrome
  2. Click the Scroll to Paragraph X buttons above the editor to scroll to the three paragraphs with ids

Expected Result

The paragraph named in the button is scrolled into view when the button is clicked

Actual Result

In standards mode it works as expected, but when the editor's document is in Quirks mode scrolling only works in the forward direction on IE 11 and Firefox, and does not work at all in Chrome.

Browser

I can reproduce this in Internet Explorer 11, Firefox, and Chrome.

Internet Explorer 9 and 10 work correctly.

#11786 [IE8] codesnippetgeshi does not print new lines correctly confirmed Bug Normal
Description

As in a ticket title.

#11787 Umbrella ticket for Problems with Asian input confirmed Bug Normal
Description

It seems that we have couple of issues that concern Asian languages and input methods:

  1. #8854, #10414, #13575 - IE, Enter_BR result in invalid input. Related ticket - #12391
  2. #12597 - Blink, Enter_BR result in invalid imput
  3. #9332, #9749 - startupFocus:true in Firefox, not possible to type Japanese chars
  4. #11663, #12415 - IE11, autogrow result in invalid input
  5. #11601 - Firfox space is increased on copy/paste
  6. #8913 - full width spaces.
  7. #11764 - not yet confirmed.
  1. #12910 - IE11, Korean; Issue may be associated with string size.
  2. #13043 - IE11, Korean; Inline styles are not maintained when typing.

#11794 [UX] Apollo image has class instead of style confirmed Bug Normal
Description
  1. Open replace by class sample.
  2. Click on "Apollo" image.
  3. Open image dialog.

Image is align to right, but "Alignment" is "<not set>". It is because we use class instead of inline style to align this image.

#11795 [FF] Ctrl+backspace inside table removes too much stuff confirmed Bug Normal
Description
  1. open any sample with CKEditor (i.e. samples/replacebyclass.html)
  2. use following source code 
    <table border="1" cellpadding="1" cellspacing="1" style="width:500px">
	<tbody>
		<tr>
			<td>&nbsp;</td>
			<td>&nbsp;</td>
			<td>&nbsp;</td>
		</tr>
		<tr>
			<td>&nbsp;</td>
			<td>&nbsp;</td>
			<td>&nbsp;</td>
		</tr>
		<tr>
			<td>&nbsp;</td>
			<td>&nbsp;</td>
			<td>&nbsp;</td>
		</tr>
	</tbody>
</table>

<p>&nbsp;</p>
  3. switch back to wysiwyg mode
  4. place cursor in very last cell (9th)
  5. press ctrl + backspace twice

Expected result:
Caret should not move outside of the cell?

Current result:
Leaves table with one column and two rows

additional info:

  1. it acts weird with ctrl+backspace in many other cases with tables
#11800 Missing integration of anchor and image2 confirmed Bug Normal
Description

Following #11341, it's not possible to create an anchor (with Anchor button) out of image2 widget (image) or, at least, the feature is buggy in most cases. Since it's possible to create linked images, users would expect to do the same with anchors.

There are two solutions: We can either completely disable the feature but, quite frankly, it would not make much sense since linking already works or we can simply enable it.

Special case of #11963.

#11802 Margin is set on list item when creating list from indented paragraph confirmed Bug Normal
Description
  1. Open editor with no content
  2. Focus editor
  3. Click few times "Increase indent"
  4. Click "Insert/Remove ordered list"
  5. Switch to source mode: 
<ul>
	<li style="margin-left: 120px;">Foo</li>
	<li>Bar</li>
</ul>

Expected: 

<ul style="margin-left: 120px;">
	<li>Foo</li>
	<li>Bar</li>
</ul>

Reasoning

Paragraph's margin should be moved to ol/li when creating list, because this is where indent/outdent buttons add/remove margin when you have a caret in the first item of a list.

However, I'm not sure what if we're creating list out of few paragraphs when each have different margin. I think that in such case it's best to remove those margins and "normalize" the situation. Otherwise, we'd have to go crazy and e.g. create sublists based on indentation of following paragraphs, but that would be a waste of time and we don't know if user wanted to do that anyway.

Alternatively, we can simply remove all margins when creating list, because none of the solutions seem to be 100% correct. Everyone can have different idea about how that should work.

#11806 [IE-all] Creating placeholder in anchor and click drag handler load page which URL is set in anchor confirmed Bug Normal
Description
  1. Open placeholder sample ​http://presets.ckeditor.dev/4.4.0/full-all/ckeditor/samples/plugins/placeholder/placeholder.html
  2. Put caret in anchor "CKEditor"
  3. Create some placeholder in caret position by clicking placeholder button located in toolbar
  4. Hover over newly created placeholder
  5. Click placeholder drag handler

Actual result: In WYSIWYG area there is loaded page with URL set in related anchor tag.

#11808 [IE] It's possible to enter code snippet's (non-editable content's?) body by up/down keys confirmed Bug Normal
Description
  1. Open code snippet sample.
  2. Focus editor.
  3. Start navigating with up/down keys.

Reproduced in IE9 and IE11. I set version to 4.3, because since then we support widgets.

#11810 [IE] Widgets drag container allows to put text in it confirmed Bug Normal
Description

I've noticed it in IE8 and IE9. Didn't reproduce it with IE11 though.

Chromium and FF seems to be untouched by this issue.

  1. open any sample with CKEditor and widgets (i.e. samples/plugins/codesnippet/codesnippet.html)
  2. move your mouse to top border of the widget
  3. click left mouse button

Expected result:
I think that nothing really should happen, i mean - focus should remain on widget.

Current result:
Caret goes into a drag handler.

additional info:

  1. In IE9 caret sometimes went to the last widget on the page
  2. I tried to reproduce it with captioned image2 / mathjax but none of these succeed, is it possible that it's codesnippet issue?
#11817 Magic line does not display properly confirmed Bug Normal
Description

Browsers: All

  1. Open magic line sample: ​http://presets.ckeditor.dev/4.4.0/full-all/ckeditor/samples/plugins/magicline/magicline.html
  2. Set following content into editor

<p></p> <hr /> <hr />

  1. Try to insert line between hr tags

Actual result: Magic line is not displayed in proper position

Please note: when add more hr tag, then magic lines are displayed between some of them.

#11821 [IE] Wrong behaviour after hit Ctrl + Backspace on selected text in CKEditor confirmed Bug Normal
Description

It's a browser bug, but we can handle it in CKEditor.

​https://connect.microsoft.com/IE/feedback/details/852659/ie11-wrong-behaviour-after-hit-ctrl-backspace-on-selected-text-in-contenteditable

  1. Open CKEditor sample
  2. Select some word
  3. Hit Ctrl + Backspace

Actual result: Word is not deleted but previous does.

#11824 sourcedialog / codesnippet textarea enhancement confirmed New Feature Normal
Description

Current implementation is kind of static, once the dialog is inited it will store textarea sizes, and if one will resize browser / dialog, it's not being resized with it.

Because of that reason I've decided to hide resizer in codesnippet plugin, because it does not give expected results.

Things to solve:

  • Make textarea responsive, it should be shrinked/expanded along with dialog (both width and height ideally)
  • Put some effort so that these two implementations will share the same solution - let's keep it DRY
  • Put dialog resizer back to codesnippet plugin
#11825 [IE] Clicking on image2 in readonly mode causes error confirmed Bug Normal
Description

Part of: #12134.

  1. Open image2 dev sample.
  2. Make editor readonly.
  3. Click on any image.

IE8: 

  Could not complete the operation due to error 800a025e.
  selection.js, line 1834 character 6

IE11: 

SCRIPT16389: Unspecified error.
File: selection.js, Line: 1937, Column: 6

Since 4.3, I was not able to reproduce it on 4.3beta.

#11828 Content Templates Improvement confirmed New Feature Normal
Description

Templates are very useful, but if a site has a large number of templates or very complex templates, the load on the editor can get rather large. I would suggest an option to provide a URL as an alternative to the html variable. I have set this up in my own tests and this works well. In my example, it expects the return data to all be JSON arrays with the actual HTML in the "result" param, but it could also take raw HTML instead if needed.

In the template plugin, around line 50 there is currently this line...

item.on('click', function(){

I moved the insertTemplate call into an if statement, so now it reads this way...

if(template.url){

var hmtlData = ; var ajaxOptions = {

async : false, url : template.url, success : function(data, status, xhr){

htmlData = data.result;

}

}; jQuery.ajax(ajaxOptions); insertTemplate( htmlData );

}else{

insertTemplate( template.html );

}

Basically, if the chosen template has a URL, it will load via ajax (async off) and populates, rather than using the HTML attached. This also solves other issues, like needing to encode templates inside JS files or not being able to access HTML based templates from the server.

#11829 List element disappear after making indentation confirmed Bug Normal
Description

Browsers: all

  1. Load editor with following content:

<ol>

<li><h1>One</h1></li> <li>Two</li>

</ol>

  1. Put caret right after "One" word
  2. Click "Increase indent" button located in toolbar

Actual result: list item with word "Two" disappear

#11832 Remove "Dependent (Netscape)" from popup options confirmed Bug Normal
Description
  1. Open link dialog.
  2. Open "Target" tab.
  3. Select <popup window>

This is pretty old part of the Editor, especially "Dependent (Netscape)" option. We should remove this option since we do not support Netscape for a while.

#11835 [Opera] Dialog box is displayed behind flash content confirmed Bug Normal
Description

Tested in Opera 20.0.1387.91, Windows 7 64-bit, Flash Player 13.0.0.182

  1. Open sample ​http://ckeditor.dev/plugins/htmlwriter/samples/outputforflash.html
  2. Open link dialog (by hitting Ctrl + L when editor focused or by clicking appropriate button)

Actual Result: Dialog box is behind flash content http://dev.ckeditor.com/raw-attachment/ticket/11835/usun.PNG

#11836 [IE8-10] Special container style is not fully removed confirmed Bug Normal
Description
  1. Open Replace by class sample.
  2. Select a paragraph.
  3. Apply "Special container" style.
  4. Select "Special container" style again to remove to.

Result: border and padding is removed but background is still grey.

On Chrome background is also removed.

Since 4.0.

#11837 [IE8-10]: Delete key removes too much confirmed Bug Normal
Description
  1. Open replacebycode.html sample
  2. Put cursor e.g in header Apo^llo 11
  3. Press Shift+End to select text
  4. Press Delete

Problem: Paragraph below is joined with deleted Header. If you do that with mouse or try same steps in modern browsers, only header will be deleted but no joining will occur.

This problem occurs in native contenteditable used in IE8-11 but in CKEditor this doesn't work in IE8-10. For some reason this issue works in IE11 in CKEditor. Perhaps there is something that can be done in IE8-10?

NOTE: When you do the same with mouse it will work.

#11840 [IE11] Editable fixDom causes selection to lose direction confirmed Bug Normal
Description

Steps to reproduce

  1. Open the nightly demo page in IE11 ​http://nightly.ckeditor.com/14-04-16-06-05/standard/samples/replacebyclass.html
  2. Place your cursor at the end of the document
  3. Hold the Shift key down and tap the Up Arrow key repeatedly

Expected Result The selection expands one line at a time until you have the whole document selection

Actual Result Each time your cursor enters a new block that does not contain a filler the fixDom method reselects the current selection, when this happens the direction of the selection is lost and Shift+Up starts moving the end of the selection upwards instead of the beginning.

System This affects IE11 on both Windows 8.1 and Windows 7

Notes The steps to reproduce here are just for the convenience of reproducing it easily, I don't think we have anyone that actually selects large amounts of text holding shift and tapping the up arrow key. The scenarios we're getting from our customers when they run into this is that they usually hold Shift and Up to span multiple paragraphs and then without releasing Shift tap right or left arrow to fine tune their selection, this results in the end of their selection moving left and right if fixDom has been called due to the selection change.

#11842 Invalid element given in a parameter for contextMenu.addListener listener. confirmed Bug Normal
Description

There seems to be an issue with context menu in readonly mode. It does not pass correct argument to funciton registered with editor.contextMenu.addListener() (see ​reference) on right click. Elements path seems to be invalid as well.

Issue might be OS dependent - tested with FF28, Chrome, IE8 all @Win7.

  1. change your editor config (i.e. by changing config.js) to contain: config.readOnly = true;
  2. open any sample with CKEditor (i.e. samples/replacebyclass.html)
  3. execute following code in order to register a simple listener: 
    CKEDITOR.instances.editor1.contextMenu.addListener( function( el, sel ) {
	console.log( 'clicked tag ', el && el.getName() );
} );
  4. right click directly on any link inside the editor

Expected result:
Editor should log something like: 

clicked tag  a

Current result:
It uses totally different tag, and output is like: 

clicked tag  h1

FF28@Win7 gives even a body as a parameter.

additional info:

  1. PK did already a quick tests and says that everything seems to be ok at Mac
  2. for me it seems that selection does not reposition itself in readonly mode or sth
#11843 [IE8-11] selectionChange event outputs invalid element when Ie is in Compatibility mode. confirmed Bug Normal
Description
  1. Put attached api2.html sample into samples folder
  2. Open sample in IE browser
  3. This sample contains code for selectionChange: 
    selectionChange : function(evt){
	var htmlElement = evt.data.selection.getStartElement();
	console.log(htmlElement.getText());
}
  4. Set compatibility mode for current page e.g. Top Menu -> Tools -> "Compatibility View"
  5. Click on "Set Editor Contents" button
  6. Click on "Objective".
  7. IE will report 
    Main Title
Objective
  8. Scroll Down and double-click on "Description". You may need to double-click few times.
  9. Editor will report: 
    Main Title
Description

This doesn't happen in standard IE mode. Whenever you click on element, only its description gets reported.

Problem is reproducible in CKE 4.0 beta.

#11853 Fail to correctly copy and paster the Upper Roman list confirmed Bug Normal
Description

It’s an issue that deals with insert list and changing the list to Upper Roman or any other within the list and copying the list and pasting it again. This issue happens in IE8 and IE9 only. FireFox works correct Recreate Steps:

  1. Created a list and indented the list (Highlighting all at once)
  2. Highlighted all at once and Selected Insert Number list
  3. Highlighted all and right clicked and selected Number List Properties and changed number list to Upper Roman.
  4. The list updated and gave Upper Roman.
  5. I highlighted the list and did a Control-C and copied the list.
  6. I double spaced to give space
  7. I did a Control-V to pasted the list.

Expected Results: To copy the same Upper Roman list. Actual Results: Pasted the Orginal Number list. (Please see the attachment)

#11864 Paragraph added with magicline under div keeps nbsp confirmed Bug Normal
Description

If you use magicline to create a paragraph under a div the paragraph keeps the nbsp after typing content.

Open demo, remove all content and in source view type: <div>DIV</div>

Leave source view and use magicline to insert a paragraph under the div. Type some text - the nbsp remains at the end of the paragraph. Also clicking away from paragraph (before typing) and back to it doesn't remove the nbsp (text starts after it).

Appears viewing source helps fix the issue.

#11865 HTML parser and writer do not process HTML entities in attributes like browsers confirmed Bug Normal
Description

htmlParser and htmlWriter should handle HTML entities in attributes the same way browsers do natively. See test case in corresponding branch to notify the difference.

#11873 flash and iframe resizing confirmed Bug Normal
Description

Object resizing for flash and iframe fake objects don't work. It's reproducable on full featured demo. After resizing fake object is set to 82x82 pixel. On double click it shows the right values in dialog width and height fields.

#11875 Image plugin ACF: disallowed height also removes width setting confirmed New Feature Normal
Description

I want to remove table and image height settings completely. I only want widths set, so I set an ACF rule like so: img(*)[*]{width};table(*)[*]{width};

Now the height setting is gone from image properties, but so is the width. The table properties dialog shows only the width setting as expected.

I expected it to remove the height and lock ratio button, but leave the width and reset button. I'm a support client - should I have reported it through email instead of creating a ticket?

#11876 [FF] Remove -moz- prefix from box-sizing property confirmed Task Normal
Description

Since version 29 (​https://developer.mozilla.org/docs/Mozilla/Firefox/Releases/29), Firefox supports box-sizing without the vendor prefix. Let's keep things clean and save some bytes here and there.

#11877 Walker executes guard on node twice in specific situation confirmed Bug Normal
Description

Range: <blockquote><p>y[</p>z</blockquote>]

Walker going forwards will execute guard twice on <p> and on "z". This may be a track of some bigger issue, but next() and previous() return nodes correctly, so only performance is affected.

The same bug occurs when going backwards with mirrored range.

#11878 [IE11]: Applying different numbers to list doesn’t work. confirmed Bug Normal
Description

Variation of #11853.

Problem can be reproduced in IE9-11

  1. Open replacebycode.html sample and clean editor contents with New Page command.
  2. Create numbered list (e.g. with 4 elements).
  3. Select List with Ctrl+A
  4. Right-click on list and open List Properties Dialog
  5. Change style to upper Roman

Result: In IE11 it is impossible to change numbering.

#11879 (Chrome) Table: Insert Column After confirmed Bug Normal
Description
  1. Load the editor with following content: 
<table border="1" cellpadding="1" cellspacing="1" style="width:500px">
	<tbody>
		<tr>
			<td>1</td>
			<td>2</td>
			<td>3</td>
		</tr>
		<tr>
			<td>a</td>
			<td>b</td>
			<td>c</td>
		</tr>
	</tbody>
</table>

<p>&nbsp;</p>
  1. Select the entire second cell ('b') on second column by double clicking it. Make sure the entire cell is selected.
  2. Right click and choose Column > Insert Column After

Expected result

A new column is added between the existing second and third columns

Actual result

A new column is added after the third column e.g. as the last column in the table

Tested in Chrome

#11902 Math plugin doesn't work when used in inline editor created on Paragraph. confirmed Bug Normal
Description

Edit: Please see comment:3[[BR]] Math plugin is disabled when inline editor is created on paragraph.

When using the inline editor on phrased content​https://developer.mozilla.org/en-US/docs/Web/Guide/HTML/Content_categories#Phrasing_content the option to add math with the Mathematical Content plugin is disabled.

According to the specs the <math> element is considered valid phrased content. I guess there is some unnecessary content filtering going on that prevents me from using the plugin? If so, is there a list where I can check which elements/plugins are allowed for each content type (flow or phrased)?

#11906 Parser does not accept incorrect HTML tag with excessive quotes confirmed Bug Normal
Description

See comment:2 for TC. Note that this HTML is not fixed by Firefox nor Chrome when we pass content through fixBin.

#11914 FF: Default values for Horizontal Alignment & Vertical Aalignment not shown on Cell Properties dialog confirmed Bug Normal
Description

To reproduce the defect:

  1. Open any CK Sample and insert a Table with default values
  1. Open Cell Properties dialog by clicking on Cell - > Cell Properties option in Context menu

Issue: see that Default values <not set> not shown in Horizontal Alignment & Vertical Alignment combo boxes

Tested on FF27, FF28 & FF29

#11923 [Blink] Caret is rendered at wrong location when editable region ends with a non-editable region confirmed Bug Normal
Description

Simplified TC:

  1. Open ​http://ckeditor.com/demo#widgets
  2. Put the cursor in 3rd line, after the first mathematical formula (The Cauchy-Schwarz Inequality)
  3. The cursor is at the very end, it should be placed straight after the formula (a widget).

This happens because a non-editable element is at the end.

Original TC:

Configuration: Plugins: image2 enterMode: ENTER_P

Steps to Reproduce in Chrome Or Safari:

  1. Open up plugins/image2/dev/image2.html
  2. Select an image and remove caption alignment styles
  3. Make sure the image is the only item on the line
  4. Click to the right of the image

You will see the cursor appears on the right side of the screen, at the end of the P tag.

Expected Result: Cursor should be next to the image.

I reproduced this from the source code repository. I originally found it in my implementation. When I add images by upload, this does not occur. I have to save the document and then load it up again to reproduce it. This does not seem to be a problem while in BR mode but I haven't totally verified that.

#11938 Webkit Blink: Image2 Cursor not available when Image is only content in editor confirmed Bug Normal
Description

This is variation of: #11923

  1. Open image2 sample.
  2. Clear editor contents.
  3. Paste below code: 
    <p><img alt="Saturn V" src="assets/image1.jpg" style="float:right" width="200" /></p>

or 

<figure class="image" style="float:right"><img alt="Saturn V" src="assets/image1.jpg" width="200" />
<figcaption>Roll out of Saturn V on launch pad</figcaption>
</figure>
  4. Switch to wysiwyg and try to click behind or before image.

Result: You won't see cursor blinking anywhere and you are not able to type in editor.

This is happening only with widgets.

#11939 Table resize plugin should also allow resizing row height confirmed New Feature Normal
Description

Table resize plugin should also allow resizing row height.

#11949 Undo does not work after cut or delete word using native context menu confirmed Bug Normal
Description

Browsers: All

  1. Open some sample with basic version of editor.
  2. Select some word (double click or using keyboard).
  3. Open native context menu on selected word (right click or by using keyboard).
  4. By selecting appropriate value from native context menu (delete or cut) remove word from editor.
  5. Use Ctrl + Z combination to undo chagnes.

Actual result: Undo action was not called. Probably because was not added to stack.

Please note: this bug occur in basic mode which does not support editor custom context menu.

#11954 Invalid selection after deleting a table. confirmed Bug Normal
Description

After removing a table selection is being placed in buggy place, in addition to that editor seems to lose the focus.

  1. open any sample with CKEditor (i.e. samples/replacebyclass.html)
  2. switch to source and use following source code: 
    <h1>foobar</h1>

<p><strong>Apollo 11</strong> was the spaceflight that landed the first humans,</p>

<table border="1" cellpadding="1" cellspacing="1" style="width:500px">
	<tbody>
		<tr>
			<td>&nbsp;</td>
			<td>&nbsp;</td>
		</tr>
	</tbody>
</table>

<table border="1" cellpadding="1" cellspacing="1" style="width:500px">
	<tbody>
		<tr>
			<td>&nbsp;</td>
			<td>&nbsp;</td>
		</tr>
	</tbody>
</table>

<p>Armstrong spent about...</p>
  3. switch back to wysiwyg mode
  4. remove second table, with one of two options:
    1. right click with your mouse, and use "Delete Table" option
    2. navigate selection using keyboard inside the table, press shift + f10 and use "Delete Table"

Expected result:

  1. Table should be removed.
  2. Selection should be collapsed, and moved at the beginning of next element: 
    ^Armstrong spent (...)
  3. Editor should keep focus
  4. No empty line should be visible.

Current result:

  1. editor leaves a temporary empty line, where selection is put
  2. editor loses focus

additional info:

  1. I've experienced it in FF and Chrome @Win7
  2. This seems to be not a (recent) regression, because I've been able to repro it in 4.2.3
#11958 HTML in head title is changed to cke_protected confirmed Bug Low
Description
  1. Open Full Page Editing sample.
  2. Put this html in the Source mode: 
    <html>
<head>
	<title><!-- foo --></title>
</head>
<body></body>
</html>
  3. Change mode to the WYSIWYG and back to source.

Expected: 

<title>&lt;!-- foo --&gt;</title>

Actual: 

<title>&lt;!--{cke_protected}{C}%3C!%2D%2D%20foo%20%2D%2D%3E--&gt;</title>
#11959 [IE10] Tableresize: Unable to get property 'toLowerCase' of undefined or null reference confirmed Bug Normal
Description
  1. Open Table resize sample.
  2. Set the following html (we need horizontal scollbar and table wider than WYSIWYG area): 
<p>&nbsp;</p>
<p>&nbsp;</p>
<p>&nbsp;</p>
<p>&nbsp;</p>
<p>&nbsp;</p>
<table border="1" style="width:1500px">
	<tbody>
		<tr>
			<td>foo</td>
			<td style="width: 1000px;">bar</td>
		</tr>
	</tbody>
</table>
  1. Hover the last pillar.

Result: Error in the console: 

SCRIPT5007: Unable to get property 'toLowerCase' of undefined or null reference 
element.js, line 711 character 3

I was not able to reproduce it in Chrome, FF nor IE11.

Screen cast attached.

#11963 Can't replace block widgets with anchor confirmed Bug Normal
Description
  1. open sample with image2 plugin (/samples/plugins/image2/image2.html)
  2. focus first image widget instance
  3. click "Anchor" button in toolbar menu

Expected result:
Since anchor should not be appliable to block widgets, end user would expect one of these two solutions:

  • focused widget to be replaced with anchor
  • anchor button disabled

Current result:
Nothing happens.

additional info:

  1. tested with Chrome, IE
#11964 max width for medias confirmed New Feature Normal
Description

If users can resize images, flash or iframes, there should be a maximum for the width value. The value could be the width of the parent element.

Example: 

<div style="width: 500px">
<img src="myimage.jpg" />
</div>

In image dialog, if is set a width value greater than 500, the value should be changed to 500, because the width of the parent div is 500 pixels.

If resize handlers are used, resizing should be stopped, when width is greater then 500 pixels.

Same for other medias like flash, iframes, tables or own fake elements.

Is it possible? Think this would be a nice feature, aspecially for inline editors.

#11966 [IE] Compatibility view: Caret goes outside of the last block element, when showblocks is enabled confirmed Bug Normal
Description

You're able to move caret outside of any block element (in classic editor that will be directly into a body).

I've checked briefly and it seems that it was introduced in 4.3.0, since i couldnt repro it in 4.2.3, please double check it.

  1. open any sample with CKEditor (i.e. samples/replacebyclass.html)
  2. enable showblocks plugin
  3. press ctrl + end

Expected result:
Caret should be stopped at the end of last node.

Current result:
Caret is placed after last block element, directly in body elmeent.

additional info:

  1. I've experienced it in IE9 compat mode, did not tested in other IEs.
  2. Seems that issue does not appear without showblocks plugin.
#11967 [Opera@Win] Inline editor created from textarea blinks when clicking confirmed Bug Normal
Description
  1. Open "Replace Textarea with Inline Editor" in the Opera browser.
  2. Click several times to set cursor in the different position in the editor.

Result: Text area is blinking every time I click.

I was not able to reproduce that issue on Chrome.

Everything is fine in standard inline sample.

Tested with Opera 21.0@Win7.

#11968 Error in dialog box when trying to replace found text with empty block. confirmed Bug Normal
Description

Browsers: All

  1. Open sample ​http://presets.ckeditor.dev/4.4.1/full-all/ckeditor/samples/replacebyclass.html.
  2. Select sentence: Apollo 11 was in first paragraph.
  3. Open find dialog.
  4. Click find button.
  5. In replace tab click 'Replace'.
  6. Click close.

Actual result: There is an error in console: Uncaught TypeError: Cannot read property 'type' of null ckeditor.js:139

Plase note: similar to #11423

#11978 [Gecko] Fix D&D for multiselection confirmed Bug Normal
Description

After a testing phase for D&D some issues related to Firefox multiselection were found:

​https://docs.google.com/document/d/1hG4H0r21MXNkRd3amDEOBPygJe3ehBAXFWAal2DptGQ/edit?usp=sharing

Bug 14, Bug 16.

This issues should be fixed after closing #11636.

Part of D&D umbrella: #11437.

#11979 No support for ctrl key while dropping confirmed Bug Normal
Description

In browsers like FF, IE you are able to press ctrl before (and hold it until) dropping a content. This way you should be able to copy dragged content instead of moving it.

  1. open sample with CKEditor with new DnD
  2. drag any element/text inside the editor
  3. while dragging hold the ctrl key

Notice how the cursor changes to indicate a copy

  1. drop it in any place
  2. release ctrl key

Expected result:
Origin instance should not be removed.

Current result:
Origin is being moved ignoring ctrl key.

additional info:

  1. Blink does not provide handling for this feature
  2. I'm not sure if this feature is present in OSX
#11988 Image width set in percentages changed to be in pixels after drag and drop in chrome confirmed Bug Normal
Description

Summary: When dragging an image with width style in percentage in chrome, the width is changed to be in pixels.

Reproduction:

  • Open standard demo in chrome browser
  • Add an image and set it's width property to be in percentages, for example 50%
  • Drag the image to a new position

Result: The width value is changed to be pixels

I think this is related to issue #9998, I beilive this is due to content editable behavior in chrome

#12004 Use BASE_PATH in tests confirmed Task Normal
Description

When ​https://github.com/benderjs/benderjs/issues/12 will be done we should modify tests to use BASE_PATH to get global assets instead of related paths.

#12007 Elements path - viewport is scrolled and selection placed incorrectly when clicking on specific elements inside widget confirmed Bug Normal
Description
  1. Open ​http://ckeditor.dev/plugins/widget/dev/nestedwidgets.html
  2. Place caret inside simple box.
  3. Click the 3rd element (2nd div) in elements path.
  4. Viewport is scrolled and selection is placed outside widget

Expected: contents of simplebox'es content should be selected.

See: comment:2.

#12014 Add className to a CKEditor.dialog instance confirmed New Feature Normal
Description

Hello,

When creating a new plugin, we almost always need to use a dialog instance. And maybe sometimes, we need to override default ckeditor style. In order to give all liberty of the theme of Ckeditor to the user, it would be a great idea to add a className attribute on the dialog configuration, so that plugin developers can add their own css without overriding everything else.

#12019 Styles' states and values when block widget is selected confirmed Piotrek Koszuliński Bug Normal
Description
  1. Open ​http://ckeditor.dev/plugins/widget/dev/nestedwidgets.html
  2. Focus one of block widgets.
  3. See format and styles drop-downs, basic styles, etc. All are enabled and many times values are incorrect - e.g. format is set to "normal div" for image2 inside nested editable (additionally, this is not consistent between framed and inline editors).

All these behaviours come from a decision we made while working on widget system. We decided that styles will be applicable to block widgets, because they may contain nested editables in which these styles may be applicable, even if they are not applicable to the widget itself (and besides widget styles they never are). This reasoning definitely makes sense - if I have 3 nested editables inside a widget and I want to make all text inside them bold I can focus widget and apply bold. Alternatively, I would have to apply bold three times. To make all this work we needed to modify style system so it traverse non-editable fragments of content, finds nested editables, retrieves their filter instances and asks them whether styles is allowed inside. This part works great.

Unfortunately, all the rest is a mess. Problems are caused by inability to correctly set style's command state and value. See the following scenarios.

  1. Focus captioned image2 and apply bold. Bold is applied in the caption. Now, try to remove the bold. You can't, because bold button is still off after applying bold. Why is it off? Because <figure> is not inside a <strong>. Are we able to set the state to on? I don't think so - this would mean traversing focused widget's DOM, finding nested editables and then what? Enabling style if it's applied to entire content of all nested editables? Or at the beginning of any editable? In any case this would be a performance killer, because we not only need to find editables, but also first editable place inside them (text node, image, space before <br>, widget, etc).
  2. Imagine the above scenario with captioned image2 nested in simple box. The algorithm will need to go inside that image2 when simple box is selected.
  3. What is the expected format drop-down's value when simplebox is focused and it its content there are headings and paragraphs? Simplebox has a header in which format should be disabled. Then it has content, which may start with a normal block (paragraph, heading) or for example a captioned image. Image does not allow changing format (so should drop-down be disabled?), but after image there may be paragraphs, so maybe format should be enabled (but then what with the value?).
  4. Imagine all these scenarios with widgets containing multiple nested editables of different types. User will never intuitively know what will happen and what's just happened.

And I'm sure that there are move lovely cases like these.

Therefore, I don't see other option than disabling styles when block widget is focused. Only widgets styles should be active.

Note that this won't change the behaviour of applying style to a selection containing some content outside a widget and a widget (e.g. bolding after doing CTRL+A on text with captioned images). Style will still be applied to all places in which it's allowed. There's a substantial difference between these cases - the start of selection. When selection does not contain only a widget, it starts in an editable place, so we can check style's state in that place.

#12023 Improve performance of element.find() and element.findOne() confirmed Task Normal
Description

These methods have to set a temporary id on the element because querySelectorAll has different behaviour that anyone could expect and it applies the selector to entire DOM tree, even below the root.

Additionally, querySelectorAll is not as fast as its simpler friends - getElementsByTagName and getElementsByClassName.

It may be worth to test such addition to find and findOne: 

		if ( selector.match( /^\.[a-z_-]+$/i ) ) {
			var found = this.$.getElementsByClassName( selector.slice( 1 ) )[ 0 ];
			return found ? new CKEDITOR.dom.element( found ) : null;
		} else if ( selector.match( /^[a-z_-]+$/i ) ) {
			var found = this.$.getElementsByTagName( selector )[ 0 ];
			return found ? new CKEDITOR.dom.element( found ) : null;
		}

This needs a research, but I'm setting milestone, because we're planning to work on #10903.

#12026 'Cut' option in tool bar is still enabled even when there is no text in the text area confirmed Bug Normal
Description

1.Select all the characters in the text area 2.Click Cut option in tool bar 3.Now there are no characters in the text area, but still the Cut option is enabled

Browser: IE8, OS: Windows7

#12028 Attribute contenteditable="false" in table cell does not fully disable content modification confirmed Bug Normal
Description

I found out that attribute contenteditable="false" in table cell does not fully disable cell content modification. I was able to modify cell content with following setup:

  1. Create editor

CKEDITOR.replace( 'editor1', {
allowedContent: true
});

  1. Copy-paste table

<table>
<tbody>
<tr>
<td contenteditable="false">Commander</td>
<td>Neil A. Armstrong</td>
</tr>
<tr>
<td contenteditable="false">Command Module Pilot</td>
<td>Michael Collins</td>
</tr>
<tr>
<td contenteditable="false">Lunar Module Pilot</td>
<td>Edwin &quot;Buzz&quot; E. Aldrin, Jr.</td>
</tr>
</tbody>
</table>

  1. Click a cell having contenteditable set to false
  2. Run commands from the toolbar such as create new list, align text center or block quote
  3. Content in the cell is edited and new elements inserted (such as p-tag) to the cell. When align text was clicked repeatedly, p-tags were inserted before the table which came visible after toggling between source and WYSIWYG

I would assume that when setting table cell attribute contenteditable as false that no operation / command could be run after user has clicked the cell / has focus on the cell.

A solution would be to disable all buttons on toolbar when user puts focus to element having contenteditable false. Then it would be more evident for the user that the content can not be edited.

← 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 →
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