Custom Query

Filters
 
Or
 
  
 
Columns

Show under each result:


Results (1701 - 1800 of 2591)

Ticket Summary Status Owner Type Priority Milestone
#11427 [IE8] Many "Invalid argument" errors when beginning of the document is removed confirmed Bug Normal
Description
  1. Open replacebyclass.html sample in IE 8.
  2. Select content: from strike through three and up to the begging of the document.
  3. Press Ctrl+C or Delete.

Result: Error: Invalid argument and Browser crash.

The same in inline.

IE9 and Chrome works fine.

Since 4.0.

#11428 Elementspath entries should not be dragable confirmed Bug Normal
Description

Buttons in elementspath should not be draggable. Now you can drag it into editable. What's worse in IE10, IE11 it will cause following exception:

SCRIPT5009: 'CKEDITOR' is undefined 
replacebyclass.html, line 355 character 1
  1. open any sample with CKEditor (i.e. samples/replacebyclass.html)
  2. click somwehere inside editable
  3. click move body elements path part into editable

additional info:

  1. it should be easily fixable with setting onmousedown="return false" on anchors
#11429 [IE11] Can't place space at the beginning of text input confirmed Bug Normal
Description
  1. open any sample with CKEditor (i.e. samples/replacebyclass.html)
  2. use "Find" button from toolbar
  3. (you'll have Find what input focused) start pressing space

Expected result:
Spaces should be placed at the beginning of input

Current result:
No spaces are placed (they may be placed at the end of the input though)

#11433 [IE11] Image - crashes upon editing image properties confirmed Bug Normal
Description
  1. open any sample with CKEditor (i.e. samples/replacebyclass.html)
  2. clear editor contents
  3. insert new image
  4. set following properties
  1. double click at image (in order to open properties dialo)

Expected result:
Image properties dialog should be shown.

Current result:
Browser crashes.

additional info:

  1. sometimes you're able to repro it without setting alt text
#11434 [IE] Exception thrown while pasting page break confirmed Bug Normal
Description

TC 1 (general, IE9)

  1. Open samples/replacebyclass.html
  2. Insert page break.
  3. Select inserted page break.
  4. C-c
  5. Change selection.
  6. C-v
SCRIPT5007: Unable to get value of the property 'isBlock': object is null or undefined 
editable.js, line 1708 character 5

Works in 3.x.


TC 2 (IE11)

  1. open any sample with CKEditor table plugin (i.e. samples/replacebyclass.html)
  2. switch to source mode, and use following source code:
    <div style="page-break-after: always;"><span style="display:none">&nbsp;</span></div>
    <table border="1" cellpadding="1" cellspacing="1" style="width:500px">
    	<tbody>
    		<tr>
    			<td>
    			<p>aa</p>
    
    			<p>bb</p>
    			</td>
    			<td>&nbsp;</td>
    		</tr>
    		<tr>
    			<td>&nbsp;</td>
    			<td>&nbsp;</td>
    		</tr>
    	</tbody>
    </table>
    
  3. switch back to wysiwyg mode
  4. copy page break into first (top left) table cell

Expected result:
Operation should not cause exception.

Current result:
Exception is being thrown:

SCRIPT5007: Unable to get property 'isBlock' of undefined or null reference
File: ckeditor.js, Line: 323, Column: 45
#11442 [Blink, Webkit, IE11] Comments inside iframe tags get messed up confirmed Piotrek Koszuliński Bug Low
Description

How to reproduce.

  1. Go to the demo Page http://ckeditor.com/demo#full
  2. Switch to source mode
  3. Delete all the content
  4. enter the following content
    <iframe src="http://google.de"><!-- Hello -->This is a test</iframe>
    
  5. switch to wysiwyg mode
  6. switch back to source mode enter the comment is messed up
<iframe src="http://google.de">&lt;!--{cke_protected}{C}%3C!%2D%2D%20Hello%20%2D%2D%3E--&gt;This is a test</iframe>
#11470 [Umbrella] a11yhelp dialog needs attention confirmed Task Normal
Description

There are several things concerning the dialog that should be investigated/fixed/re-factorized:

  1. Keyboard button labels are not consistent

    Editor Context Menu Press Shift+Ctrl+F10 or APPLICATION KEY to open context-menu. Then move to next menu option with TAB or DOWN ARROW. Move to previous option with SHIFT+TAB or UP ARROW. Press SPACE or ENTER to select the menu option. [...]

This is because some labels are hard-coded (upper case) and some are generated from langfiles (lower case). #11196 allowed translation of keys and we should take advantage of that feature. All hard-coded labels must be converted into dynamic ones.

  1. We must verify whether help strings still match editor features. Some of them may be outdated/irrelevant/misleading. If necessary, some new ones should be written.
  1. Some instructions are too complicated, some are unclear, e.g. lots of translators do not understand the MagicLine-related texts so we should revise the contents overall.
  1. We must investigate a11y aspect of the dialog, e.g. compatibility with screen readers (JAWS). Perhaps <kbd> tags would be suitable here and there.

Related tickets we could also take into consideration:

  • #8490: Plugins should extend a11yhelp to make the dialog dynamic and (build/editor)-dependent. At the moment a11yhelp creates a delusion that some features are enabled while, in fact, the plugins to which they belong are not even loaded.
  • #9638: "Press ALT 0 for help" iframe voice label makes no sense when a11yhelp is not loaded (i.e. basic build).
  • #10929: Spelling Mistakes on Accessibility Instructions dialog.
#11471 When toolbar position is fixed, dropdowns in toolbar aren't scrolling correctly confirmed Bug Normal
Description

Hi, when you open SharedSpace demo

http://ckeditor.com/ckeditor_4.3_beta/samples/plugins/sharedspace/sharedspace.html

and you add those styles

style="position: fixed; top: 0px; left: 0px; right: 0px; bottom: auto; z-index: 10000; width: auto;"

to div with id "top" it will work well for editable areas in div, but for textareas it's broken. Dropdown is moving along with page scroll.

http://i.imgur.com/2cOkbBT.png

#11473 Remove deprecated ieXCompat confirmed Task Normal
Description

env.ie6Compat ... env.ie9Compat are deprecated since 4.0. After clean-up (#11422) these variables are rarely used:

env.ie9Compat -> 1 time
env.ie8Compat -> 2 times
env.ie7Compat -> 0 times
env.ie6Compat -> 0 times

They should be finally removed.

#11475 [Chrome] No context menu on text input when editor doesn't have focus confirmed Bug Normal
Description

Steps to reproduce:

  1. Add a text input to the editor, it’s most notable when the text input is placed beyond the size of the editor window
  2. Click somewhere in the page outside of the editor so it doesn’t have focus
  3. Right or ctrl + click on the text input element

When the input element is clicked, the focus jumps to the top of the editor window, and when adding a contextMenu listener this actually passes the top most element as the one being clicked.

Browser & OS: Google Chrome 32.0.1700.77 Mac OS X 10.9.1

#11481 Multiple Link insert cause missing ID confirmed Bug Normal
Description

Steps to reproduce:

  • Insert 3 links on the page, which URL "www.google.com" and Id: 1,3,3.
  • Double click on the inserted links to check the Ids, only the first link Id is retained, all other links after that have empty Id.

Browser and OS: Window Firefox 26.0 & Chrome 32.0.1

#11488 Dialog content empty when it is told it will have tabs confirmed Bug Normal
Description

In Simple Plugin (Part 1) guide, in Dialog Window Tabs part, after first code segment, there is a phrase, that tells reader, that result of changes in code segment can be seen immediately and that it will have two tabs in place.

Expected: As told Instead: There are no tabs, assuming because the tabs have no elements inside at that moment.

http://docs.ckeditor.com/#!/guide/plugin_sdk_sample_1

#11491 Pressing delete key in highlighted text in blockquote removes paragraph tags confirmed Bug Normal
Description

If you create a blockquote, enter some text, highlight the text and then press the Delete key. The paragraph(s) inside the blockquote disappear and you can type directly into it.

For FF you need more than one line of text highlighted.

#11495 CKEDITOR.ui.dialog.select size option doesn't work due to css confirmed Bug Normal
Description

CKEDITOR.ui.dialog.select size option doesn't work due to css.

select.cke_dialog_ui_input_select {
	height: 24px;
	line-height: 24px;
	background-color: #FFF;
	border: 1px solid #C9CCCF;
	border-top-color: #AEB3B9;
	padding: 2px 6px;
	outline: none;
	-moz-border-radius: 3px;
	-webkit-border-radius: 3px;
	border-radius: 3px;
	-moz-box-shadow: 0 1px 2px rgba(0,0,0,.15) inset;
	-webkit-box-shadow: 0 1px 2px rgba(0, 0, 0, 0.15) inset;
	box-shadow: 0 1px 2px rgba(0, 0, 0, 0.15) inset;
}

height parameter is limiting select area.

#11501 Visible Text Selection disappers in inline editor confirmed Bug Normal
Description

Dear Team,

In CKEditor Inline when you make text selection and then right-click on it, context manu appears and selection is lost.

#11502 Synchronous calls of asynchronous methods causes errors confirmed New Feature Normal
Description

See e.g. #11295. Calling editor.destroy() right after (or before it's completed) CKEDITOR.replace() or editor.setData() causes errors.

That's because developer has to care about calling methods when previous finished. There are callbacks and events which notify developer when action is completed, but it's his duty to find which method can be called when.

However, not every developer understand problems with asynchronous methods and not in every case it's easy to handle this. For example when editor is loaded in one tab of some UI component and user switches between tabs too quickly, destroy() may be called before editor fully initialises.

On the other hand, if editor's methods would take care of that (e.g. destroy() would wait until initialisation finished), then API would start to work unpredictably. Developer would never know if destroy() will be done immediately or if it's going to wait until something (setData, initialisation, etc.) ends. This may be even worse situation than the current one.

We could make even longer step and make all editor's methods asynchronous and e.g. based on promises. Then everything would be fine... if you understand all of this :D.

Therefore, instead of forcing developers to understand some not trivial concepts to do basic stuff we can simply clarify this in docs. But there still will be cases in which it will be hard to handle editor state (like the tabs case).

Opinions needed.


More details about the current situation in http://dev.ckeditor.com/ticket/11502#comment:8

#11503 [Umbrella] Further widgets integration with ACF confirmed New Feature Normal
Description

Cases we need to solve

  1. Integration with styles applicable to widgets. See #11297.
  2. Disabling ACF in body of some widgets - e.g. mediaembed. See #11737 and #10925.
  3. Filtering pasted widgets. See #11115.

Cause of problems

  1. ACF filters data before widgets are upcasted. Therefore for example mediaembed cannot notify ACF that children of some elements should not be filtered and it's not possible to create allowed content rule based on a widget name and style properties allowed on that widget element.
  2. In pasted or dropped content we have the upcasted and initialized HTML of a widget. Therefore we cannot filter it, because ACF is applied to the data format.

Solution

ACF after upcasting

The advantage of this solution is that ACF would know everything about widgets, so it could make precise decisions. Additionally, while filtering pasted content there would be no problem at all, because it would be the same case.

However, there are two problems which makes this idea incorrect:

  • Assuming that we've got some complex widget like image2, which makes a lot of transformations while upcasting, how would ACF filters the upcasted image? How could it strip, without breaking widget, figcaption and figure if they are not allowed? It's simply impossible, without some special actions from the upcast method. It could theoretically ask ACF if specific components of widget are allowed, but it's tricky and still doesn't solve this issue completely.
  • Moreover, when ACF would filter upcasted version of widgets it would be impossible or a lot harder to define ACF rules. One would have to know internal widget structure to define those rules. In my opinion this makes the idea totally wrong. ACF filters data format and that's a foundation we have to keep.

ACF filters data

Previous section proved that ACF needs to be applied to data, not to inner HTML. This means that the current way of processing is the only correct one, but on the other hand we still have those three cases, which are listed at the beginning, that have to be solved.

  1. Integration with styles applicable to widgets - a solution has been proposed in http://dev.ckeditor.com/ticket/11297#comment:8. It may not be 100% precise in some cases, but there's another ACF's characteristic which we have to keep in mind. It's not meant to be a security filter. It's meant to ensure good UX in as many cases as possible. Therefore when uncommon constellation of element and its attributes slips through its hands it's not a blocker.
  1. Disabling ACF without a possibility to communicate through element's attributes. In this case the only possible solution is a callback executed by ACF which will notify it about how it should proceed. For example mediaembed will add a callback which will tell ACF that all descendants of element should be skipped if element has mediaembed's class. Callback will return binary flags, because we may need more options in the future. Example of flags that we may need to implement:
  • DISABLE_ON_ELEMENT - skips only this element (continues on descendants),
  • DISABLE_ON_DESCENDANTS - skips descendants,
  • CONTINUE_ITERATION - continue iteration - used with DISABLE_ON_DESCENDANTS could inform ACF that it should look for a descendant that reenables filtering,
  • ENABLE - reenable filtering.

But of course for now we'll need just one of these flags, so only one should be implemented.

Ticket: #11737.

  1. Filtering pasted widgets. It's not possible to filter widget's upcasted version - only its source version can be filtered. If we had full control over clipboard we could downcast selected data on cut/copy and then filtering on paste would work like filtering on data load. Unfortunately, we don't have full control over clipboard and won't have even after #11437, because of IEs.

Therefore, the only solution might be to look for upcasted widgets in pasted data, extract their HTML, find out from which editors they came from, use those editors to downcast them, and replace their upcasted version with downcasted one in pasted data. This could be based on unique editors' ids and data-cke-widget-editor-id attributes marking widget wrappers. This won't work when copying between window tabs or frames or when editor instance has been destroyed in the meantime and I'm not sure what should happen in such situation.

Similar solution, maybe more straightforward and more reliable would be if editor, on cut/copy/dragstart if it cannot override the content, could retrieve widgets' HTML from selection, downcast them and store in the CKEDITOR object, so all other editors will be able to replace pasted widgets with their downcasted HTML stored in that object. Of course widgets have to be marked with unique ids to make this possible. This option is better than the previous one because widget's HTML will be kept even after editor is destroyed. We can also reuse the same mechanism in drag and drop for IEs in which we will need to store the data for a while. As the mechanism will be public 3rd party code will be able to integrate with it.

And the third solution, nearly perfect, is to control copy and cut and override data with HTML extracted by us and processed by htmlDP.toDataFormat. We will need to do the same for drag&drop, so it won't be a big problem to eventually make the same with copy/cut. Additionally, we would automatically solve the problem with Blink putting awful HTML into clipboard.

#11506 Image inside floating DIV looses selection with Context Menu on IE confirmed Bug Normal
Description

Descriptive summary: When you move the focus out of the image area and then do a right clic on the image, Image Properties option is not displayed in the context menu

  • I just erased a very detailed steps to reproduce and other information because akismet rejected my submit lots of times -
#11510 List plugin versions in the about-dialogue confirmed New Feature Normal
Description

This would help in debugging,

  • when a user says "Feature X does not work as expected".
  • you updated a plugin, but are not sure if it is in the browser yet or still cached somewhere server or locally.

I guess this is also the reason for any software's version string in about-dialogues.

PS: In my initial request in the forum i also asked is there a way to get plugin versions on the JS console of the browser inspector? which is currently unanswered...

#11522 Select all, replace text loses formatting on firefox confirmed Bug Normal
Description

Using current standard CKEditor demo:

  1. Select all the content in the demo editor and delete it.
  2. Enter some text (e.g. "original").
  3. Press Ctrl-A (or Cmd-A on Mac) to select all the text.
  4. Hit the Bold button to bold the text.
  5. Press Ctrl-A again to re-select all (even if it is already selected).
  6. Type in some new text (e.g. "new"), which will replace the original text.

On Chrome and Safari, the new text is bold (i.e. it retains the styling of the original text). On Firefox and IE11, the new text does not retain the original styling; it just gets the default styling.

I think the Chrome and Safari behaviour is correct, so Firefox and IE11 should work the same way.


Related issues: #11162, #8439, #8440.

#11524 BBCode plugin "consumes" line breaks when switching to source confirmed Bug Normal
Description

Tested on the demo using the BBcode plugin to confirm the bug. ( ckeditor.com/demo#bbcode )

When adding some line breaks between inline text and a block element, each time the user switch to source code, then switch back to preview. One line break disappear.

To reproduce the bug, use the demo (link above). Switch to source code and paste this code :

This is some [b]sample BBCode text[/b]. You are using CKEditor.





[list=1]
[*]test
[*]test
[*]test
[/list]



test test test



[quote]Test content[/quote]




[quote]Test content[/quote]

Switch back to preview, then switch back and forth to see that the line breaks between the first line of text and the list are disappearing and also the line breaks between the line "test test test" and the first quote, and finally also the line breaks between the two quotes.

It doesn't only happen when switching back to source. It has the same behavior when saving the content. If you save the content in a database, each time you save you lose some of the line breaks exactly as if you were switching to source then back to preview before saving.

#11530 Dynamic Templates with Dialog Helpers confirmed New Feature Normal
Description

I would vote to extend templates and make them dynamic through the usage of popup dialog helper. The dialog would:

  • contain user-configurable form
  • go through validation & error display
  • implement ui dependencies
  • return html from a function, given formState hash.

This would allow a web site builder to provide client with smart templates that would react to web editor input.

Enclosed please find a very rudimentary example of what it would do. Validation and error handling aspects are not polished at this moment - subject to further development upon feature approval.

#11538 Flying option menu when anchor present in URL confirmed Bug Normal
Description

Steps to reproduce:

  1. Visit a url where it ends with a # and you have a ckeditor instance (e.g. www.example.com#)
  2. Click on a menu option that has a flyout menu (e.g. fonts)

Result: You'll see that the flyout gets anchored to the top of the page Note: This only happens the first time you click on the option for flyout. All subsequent clicks anchor the flyout correctly.

This happens with barebones install of ckeditor 4.0+ & usage of CKEDITOR.replace

My environment:

  • OSX 10.8.5
  • Chrome 32.0.1700.102
#11562 allowedContent = true results in JS error when inserting elements confirmed Bug Normal
Description

When config.allowedContent = true; is set, the editor throws a JS error when inserting content (such as an image) into any unknown element. A simple test case is this:

<nobr>a</nobr>

Steps to reproduce: With the above content entered into the editor, set the cursor focus to be inside the unknown tag, then insert an image.

Inside core/editable.js, the function extractNodesData throws an error trying to look-up the allowed names for this element.

This is invalid HTML, so it is a user error, but the config.allowedContent = true; prevents it from being filtered out (as it should). We've seen it pop-up when users are copy and pasting content from other sources. So at a minimum, I think the editor should handle this case in some way.

My suggestion is when allowedContent is set to true, the editor should treat all unknown tags as a semantically meaning-less tag, perhaps inheriting the allowedNames from DIV elements.

I would be willing to submit a fix with the above suggestion, or implement another suggested fix if needed.

Thanks

Chrome error message:

Uncaught TypeError: Cannot read property 'img' of undefined ckeditor.js:319
c ckeditor.js:319
(anonymous function) ckeditor.js:326
CKEDITOR.editable.CKEDITOR.tools.createClass.proto.insertHtml ckeditor.js:300
(anonymous function) ckeditor.js:306
h ckeditor.js:10
CKEDITOR.event.CKEDITOR.event.fire ckeditor.js:11
CKEDITOR.editor.CKEDITOR.editor.fire ckeditor.js:13
CKEDITOR.tools.extend.insertHtml ckeditor.js:225
$.extend.editor.paste

Edit: This issue is about CKEditor throwing error if DTD was not extended.

#11565 Footnote plugin confirmed New Feature Normal
Description

I have noticed that there is no plugin for inserting footnotes into text using CKEditor although I found a Drupal module at https://drupal.org/project/footnotes bundled with CKEditor which has a footnote plugin.

I also found a DokuWiki module called ckgedit at https://www.dokuwiki.org/plugin:ckgedit used for editing bundled with CKEditor including a footnote plugin.

So one of these footnote plugins could be merged into the main release of CKEditor.

#11579 Merging of table cells containing an image widget fails in OSX using different browsers confirmed Bug Normal
Description

Setup:

  • OS-X 10.9.1
  • Chrome 32.0.1700.107 or Safari 7.0.1 (9537.73.11)

Steps te reproduce:

  1. Include the image2 plugin in your CKEditor build.
  2. Open /ckeditor/samples/replacebyclass.html
  3. Create a table with 2 rows and 2 columns (2x2).
  4. Put an image widget in the first cell, and some dummy text in the other three cells.
  5. Open a context menu on the first cell (containing the image) and select 'Merge Right'.

The following javascript error occurs:

Uncaught TypeError: Object #<error> has no method 'getDocument' plugin.js:402 placeCursorInCell plugin.js:402 createDef.exec plugin.js:767 exec command.js:52 CKEDITOR.tools.extend.execCommand editor.js:822 CKEDITOR.menu.CKEDITOR.tools.createClass._.onClick plugin.js:193 (anonymous function) tools.js:573 (anonymous function) tools.js:573 (anonymous function) plugin.js:361 (anonymous function) tools.js:659 CKEDITOR.tools.callFunction tools.js:687 onclick replacebyclass.html:1

#11581 Enter Backspace Shift+Enter fails on FF confirmed Bug Normal
Description

Tested with FF25 and 27.

Reproducalbe in demo

Type "AAA"
Enter
Backspace
Shift+Enter

--> expected: new line, but it make p

Backspace

--> cursor is on line under "AAA"

Type "B" -> In Line 1 is "AAAB"

With "down" u can reproduce it.

#11589 Invalid focus in link to anchor dialog confirmed Bug Normal
Description

Due to invalid focus in anchor dialog we can experience few issues. You're not able to:

  • focus other fields by pressing tab
  • close anchor dialog with esc
  1. open any sample with CKEditor (i.e. samples/replacebyclass.html)
  2. using Source button set html to:
    <p><a href="#foo">bar</a></p>
    
  3. return to wysiwyg mode
  4. double click link bar
  5. press esc key

Expected result:
Dialog should be closed (as it is for typical links)

Current result:
It's not closed unless you'll focus any input control inside of it.

additional info:

  1. Tested in Opera (blink), IE10, FF 27.0.1 @ Win8
  2. Issue was also present in 4.0 release (didn't check earlier releases)
#11593 [Image2] If only one dimension is set the missing one should not be set when resizing image confirmed Bug Normal
Description

TC1:

  1. Open image2 sample.
  2. Doubleclick image.
  3. Change the width to 300.
  4. See that height was set too although it was empty at the beginning. I'd expect that it was left empty.

Notice that in step 2 "keep ratio" is on. I think that it's ok, because it tells user that ratio will be kept if he changes size, which is true because the missing dimension is calculated automatically.

TC2:

  1. Open image2 sample.
  2. Doubleclick image to verify that only width is set. Close the dialog.
  3. Resize image using handler.
  4. Doubleclick image.
  5. See that both dimensions are set now.
  • Edit: TC2 is not about reading dimensions by dialog but about editor calculating both dimensions. In HTML when only one dimension is set the other one is automatically calculated by browser so we should not interfere it.
#11594 [FF] Dropdown arrows in the toolbar look nasty confirmed Bug Normal
Description

See attached image.

It's a Firefox bug: https://bugzilla.mozilla.org/show_bug.cgi?id=965966

#11595 Add support for MathJax block equations confirmed New Feature Normal
Description

Extracted from #11298

Currently MathJax supports only inline equations. It would be nice if block equations were supported as well.

Additionally plugins should detect what sort of equation it is dealing (probably based on brackets used).

#11596 [Umbrella] MathJax plugin improvements confirmed Task Normal
Description

This is umbrella ticked for MathJax improvements.

The list of the tickets with bug/feature requests for MathJax:

#11601 FF - Pasting Chinese characters adds spaces to the pasted text confirmed Bug Normal
Description

Firefox

Customer is complaining about spaces being inserted when pasting text in the editor.

To recreate:

Copy the following text. Paste it into the demo editor. Select and copy the text within the editor. Enter a few lines and paste the text again. You will see a space being added to the pasted text. See attachment.

该交易由登记或新办理发起。客户初次办理出口保理额度预申请或正式申请需要分别对进口商、出口商的相关信息进行补充录入。经办确认后,进、出口商信息按各分 行存储在本地,下次交易时输入进、出口商编号后可以直接调用。如果第二次录入时对原进、出口商信息有所更改,则新信息覆盖原信息。不同分行间信息独立存 储,不相互影响。出口商信息保存:栏位33-50.进口商信息保存:栏位61-73。

#11604 [IE] Unable to indent list selected by Element Path confirmed Bug Normal
Description

STR:

  1. Open the DEMO page with IE(9|10|11)
  2. Create a numbered or bulleted list with 3 elements.
  3. Select the list using Element Path (from the status bar).
  4. Press "Increase Indent" button.

RESULT: Nothing happens.

  1. Select the list by mouse. (Click on the first item and drag the selection to the last item.)
  2. Press "Increase Indent" button.

RESULT: The list can be indented.

#11605 [IE] Selection cached after making selection by mouse confirmed Bug Normal
Description

See ie-selection-cached.webm. Tested on IE8 and IE9.

  1. Write few paragraphs.
  2. Select all using mouse.
  3. Press list button.
  4. Click once in the second item.
  5. Press tab to indent it.

Result: entire list was indented. Most likely editor.getSelection() returned cached selection made in step 2.

#11606 [UX] UI Color Plugin cancel by "X" confirmed Bug Normal
Description
  1. Open "UI Color Plugin" sample.
  2. Press "UI Color Picker" icon.
  3. Change color.
  4. Close dialog using "X" in the top right corner.

Result: Alert "You have changed some options. Are you sure you want to close the dialog window?" is shown but changes are applied anyway.

Expectation: Alert will not be shown or changes will be reverted when I close dialog using "X".

I am for first option so alert should not be shown.

#11607 Custom direction "rtl" set for body in fullPage mode is reverted to "ltr" in data confirmed Bug Normal
Description
  1. Create editor with config.fullPage = true
  2. Set HTML:
    <html>
    <head>
    	<title></title>
    </head>
    <body dir="rtl"></body>
    </html>
    
  3. Show source/getData.

Expected: The same HTML as set in 2.

Actual:

<html>
<head>
	<title></title>
</head>
<body dir="ltr"></body>
</html>

The origin I discovered the bug with docprops plugin, which brings Language Direction field in its dialog.

What's wrong? First bad commit is git:751e298cca8. It's fine in 3.6.x.

#11609 [IE] List items annihilated after certain actions with Elements Paths confirmed Bug Normal
Description
  1. Use IE11.
  2. Set data
    <ul>
    	<li>x</li>
    	<li>y</li>
    	<li>z</li>
    </ul>
    
  3. Focus any list item.
  4. Click "ul" in Elements Path.
  5. Click indent: nothing happens (#11604).
  6. Put selection
    <ul>
    	<li>x^</li>
    	<li>y</li>
    	<li>z</li>
    </ul>
    
  7. Click indent.

Expected (considering that 5. is right):

<ul style="margin-left: 40px;">
	<li>x</li>
	<li>y</li>
	<li>z</li>
</ul>

Actual:

<ul>
	<li>x</li>
</ul>

Two list items are gone. Just like that. This ticket may be related to #11604.

#11610 [Blink] It is not possible to select text in with Shift+Click in link. confirmed Bug Normal
Description
  1. Open "Replace By Class" sample.
  2. Click after "humans" to move cursor.
  3. Press shift and click after "Neil".

What is the expected behavior? ", Americans Neil" is selected.

What went wrong? Cursor moved instead of select text.

This is actually a Blink/contenteditable bug and I reported it to the chromium project: http://code.google.com/p/chromium/issues/detail?id=345745&thanks=345745&ts=1393005706

#11614 Warnings about deprecated API usage in strict-mode confirmed Bug Normal
Description

Chrome reports this in developer console, when I click inside wysiwygarea:

body.scrollLeft is deprecated in strict mode. Please use 'documentElement.scrollLeft' if in strict mode and 'body.scrollLeft' only if in quirks mode.

This code do not produce any issues, however I'm not sure if users aren't scared a bit of it, for example.

#11622 No indentation of cell content possible if table cell is inside a list item confirmed Bug Normal
Description

Steps to reproduce:

  • Create an unordered list with two items
  • Inside the second list item, insert a table, insert some text in the table cells
  • Inside the table cell, click "Indent"

Actual behavior:

  • The second list item is indented

Expected behavior:

  • The text inside the table cell is indented
#11623 Text reflow is broken when deleting text at the end of a list followed by a table confirmed Bug Normal
Description

Steps to reproduce:

  • Create an unordered list. The text of the last list item must not be empty.
  • Insert a table immediately after the list. The first cell in the first row must have some text.
  • At the end of the last list item, press "Delete"

Actual behavior:

  • The text in the first table cell flows inside the list item (see screenshot). The first table becomes empty.

Expected behavior (one of the following):

  1. The entire table flows into the list item
  2. Nothing happens
#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.

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