Custom Query

Filters
 
Or
 
  
 
Columns

Show under each result:

Results (2101 - 2200 of 2591)

← 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 →
Ticket Summary Status Owner Type Priority Milestone
#12794 SCRIPT5022: IndexSizeError on table caption change in IE9-11 confirmed Bug Normal
Description

Javascript error in IE9-11:

  • Create new table on CKEditor area;
  • Enter some table caption using "Table Properties" dialog;
  • Press "Ok" button on table properties dialog;
  • Set mouse pointer on entered caption's text value;
  • Click right mouse button (at the entered caption) and select "Table Properties" from context menu;
  • Edit table caption value and click "Ok";

As a result you will receive javascript error: SCRIPT5022: IndexSizeError ckeditor.js, line 407 character 528

#6344 Script Auto-Include Path Issues in IE8 Compatibility Mode confirmed Bug Normal
Description

When using CKEditor dynamically in IE8 Compatibility mode CKEditor will crash if a hash (#) is in the url. For example, when calling CKEditor.replace, post page load, and after a hash is set, CKEditor will try to include the following paths:

​http://20.20.0.100/Tests/CKEditor/Example/#/../ckeditor/config.js?t=A8LE4JO

​http://20.20.0.100/Tests/CKEditor/Example/#/../ckeditor/lang/en.js?t=A8LE4JO

In both cases the paths to the actual files should be: ​http://20.20.0.100/Tests/CKEditor/Example/../

and NOT

​http://20.20.0.100/Tests/CKEditor/Example/#/../

Clearly this is a major issue as CKEditor will not load at all in IE8 Compatibility mode. This seems to only be an issue in this mode.

#8951 Scriptloader may fail when loading sequences of already loaded files review Sa'ar Zac Elias Bug Normal
Description

The following code will fail (the file c.js will not be loaded and the second callback will not be triggered): 

CKEDITOR.scriptLoader.load( [ 'a.js', 'b.js' ], function()
{
	alert( 'a' );
	CKEDITOR.scriptLoader.load( [ 'a.js', 'b.js', 'c.js' ], function()
	{
		alert( 'b' );
	});
} );
#9637 Scripts can be executed from ckeditor using preview plugin confirmed Bug Normal
Description

Copy pasting a HTML snippet from a browser window containing scripts opens channel for the execution of the script from ckeditor using the preview plugin.

Replication steps:

  1. Create a test html page with the following content

<html> <head></head> <body>

<a href="javascript:alert('Executing

Javascript:alert()')">test</a>

<script>alert('Script Tag') </script>

</body> </html>

  1. Save the html and open it using a browser.
  2. Copy the content of the rendered html ( i.e the hyperlinked word “test”) and paste it in RTE.
  3. Click the preview plugin in RTE.
  4. In the preview window that opens, if the hyperlink ‘test’ is clicked it will execute the script.

This posses a security threat as it opens a channel for executing malicious script can be executed from ckeditor. Writing a server code to filter the scripts will not help as it might corrupt the data. Another problem is that server code will trigger only when data is posted but in this case scripts can be executed using preview pluign even before posting data.

#4920 Script tags are not indented in the output HTML confirmed Bug Normal
Description

This is not a bug, but it simply doesn't look nice.

Using the following code in full page mode: 

<html>
	<head>
		<title>CKEditor Sample</title>
		<meta content="text/html; charset=utf-8" http-equiv="content-type" />
		<script type="text/javascript" src="script1.js"></script>
		<script type="text/javascript" src="script2.js"></script>
		<link href="/search" rel="search" />
		<link href="my.css" rel="stylesheet" type="text/css" />
	</head>
	<body>
		<p>
			test</p>
	</body>
</html>

it is transformed into: 

<html>
	<head>
		<title>CKEditor Sample</title>
		<meta content="text/html; charset=utf-8" http-equiv="content-type" />
<script type="text/javascript" src="script1.js"></script><script type="text/javascript" src="script2.js"></script>		<link href="/search" rel="search" />
		<link href="my.css" rel="stylesheet" type="text/css" />
	</head>
	<body>
		<p>
			test</p>
	</body>
</html>

Note that <script> tags are in one line and in the same line there is also one <link> element.

This bug existed also when the full page mode was not available.

#10821 Scrollbars cannot be used with big pictures in certain cases. confirmed Bug Normal
Description

Description: If I have a large picture in the editor and it is selected while the picture is about center aligned the scrollbars don't work properly or don't work at all.

STR:

  • Open a clear DEMO page (I used with IE9)
  • Insert a large picture into the editor (the horizontal and vertical scrollbars appear)
  • Display the center of the picture (The scrollbars are about to the middle of its way)
  • Select the picture
  • Try to move the scrollbars
#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.

#14760 ScrollIntoView also expands selection to entire paragraph confirmed Bug Normal
Description

Steps to reproduce

  1. With the example page, select some text (anywhere seems to work).
  2. Hit the button -- which runs the following code : 
       function scroll() {
      var selection = editor.getSelection();
      selection.scrollIntoView();
    }

Expected result

The selection is scrolled into view, but remains the same selected text.

Actual result

The selection expands to the enclosing paragraph.

Other details (browser, OS, CKEditor version, installed plugins)

Tested on Chrome

#16996 Scroll jumps in Chrome when closing dialog and inline editor new Bug Normal
Description

Steps to reproduce

  1. Go to ​http://sdk.ckeditor.com/samples/inline.html
  2. Go to one of the inline editors, then scroll until the top of the editor is hidden
  3. Open an image dialog and click to "Cancel"
  4. The scroll jumps when it gets the focus back

Expected result

Using Firefox on the same computers doesn't make the scroll jump.

Other details (browser, OS, CKEditor version, installed plugins)

Chrome 58.0.3029.81 (64-bit) OS X Sierra 10.12.4 CKeditor 4.6.2

#14520 SDK sample for balloonpanel confirmed Task Normal
Description

Let's create a meaningful SDK sample that would show how useful the Balloon Panel plugin can be without Accessibility Checker.

Features we'd like to highlight:

Setting custom content in the balloon panel. Automatic balloon repositioning on editor scroll (classic editor). Alignment to the borders of the viewport in case of classic editor. As for the content of this sample we're looking for propositions, so feel free to propose something that would make sense to end user/developer.

Related issue on SDK tracker: ​https://github.com/ckeditor/ckeditor-sdk/issues/173

#2378 Search for better options than EncodeConfig assigned Wiktor Walc Bug Normal
Description

At this moment it seems not very intuitive hot to assign SkinPath in PHP. To achieve path pointing to: 

"/sub1/# CMS/sub2/"

ones have to assign: 

"/sub1/%2523 CMS/sub2/"

...there must be an easier way to do it.

#11041 Search plugin doesn't scroll properly when there is only "BR content" in editor confirmed Bug Normal
Description

Steps to reproduce:

  • mkdir ckeditor-bug
  • cd ckeditor-bug
  • wget ​http://download.cksource.com/CKEditor/CKEditor/CKEditor%204.2.2ckeditor_4.2.2_full.zip
  • unzip ckeditor_4.2.2_full.zip
  • Copy test.html from this ticket's attachment to here
  • Call up test.html in a browser
  • Click magnifying glass in toolbar to call up search.
  • Enter xxx into search box
  • Click "Search" button multiple times. xxx occurs twice in the text, at the beginning and in the middle. The cursor jumps around to the start and end of the text without ever showing the second occurance of xxx.

Edit:

  1. Put code from attached file in to editor.
  2. You can also write your own content but editor needs to be set to enter mode BR.
  3. Try to find 'xxx' or whatever word there is in your text and repeats throughout the content (there has to be few pages of content).

Result: Editor will match and select word but will not scroll to it correctly (This can be checked when you resize editor to match whole content).

Problem can be reproduced in all browsers from CKEditor 3.4

#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.

#1398 Select all then delete or backspace clears formatting confirmed Bug Normal
Description

tested: svn-trunk Oct-10-2007
Happens in Fx and IE7:

  1. Set text formatting and type some text.
  2. Select all from the toolbar or context menu.
  3. Press either backspace or delete, menu formatting is cleared.
  4. Note: if instead of delete, you started typing then formatting is not lost.

(splitting up multiple issues from ticket 565)

#2554 Select All using Ctrl-A does not work in modal dialog confirmed Bug Normal
Description

The 'Select All' toolbar button can be used to select all content in the editor window. This works when the editor is loaded in a normal window or in a modal dialog window. However, although the Ctrl-A keyboard shortcut works in a normal window, it does nothing when the editor is running in a modal dialog box.

Versions: 2.6.2 OS: Windows XP Browser: IE6

Steps to reproduce:

Implement an instance of FCKEditor in a page, and load the page in a modal dialog box (Window.showModalDialog() in IE). Enter some text in the editor. Use the 'Select All' toolbar button, and note that all content is correctly selected. Deselect the content. Now press Ctrl-A. Note that content is NOT selected.

Load the same page in a normal window. Repeat the above steps. Note that Ctrl-A does now select all of the content.

#13071 Select content on drop, move selection to the end on paste confirmed Bug Normal
Description

All application I tested have a different behavior for copy and paste and for drag and drop.

  • After paste, caret is at the end of the pasted content (what makes sense, user most probably wants to start typing there).
  • After drop, dropped content is selected (what also makes sense, because it may happen that user drops the content in not exactly correct position and wants to drag it again).

At the moment we always put the caret at the end, handle the drop in the "paste" way. The problem is that now insertHtml set the selection and this method does not know what operation causes insertion. Fortunately paste event has such information, so the selection should be done by paste event or information about the type of the selection should be passed to the insertHtml. The second way seems to be better because of backward compatibility.

#7033 Select dialog applies multiple="true" instead of multiple="multiple" confirmed Bug Normal
Description

Select dialog should apply multiple="multiple" and not multiple="true".

#8363 Selected Style and Font not visible in V2 and office skin confirmed Bug Normal
Description

IE6,7,8

TC

  • open ​Skins sample
  • go to v2 or office skin
  • without setting focus in edit area, select style from dropdown, select font from dropdown

Expected

Selected style and font, now are visible in toolbar boxes

Actual

Selection only blinks and dropbox stays clear

When You type something and then select font/style, then selection is visible.

#16780 selectElement and/or extractSelectedHtml work differently in Chrome and Safari pending Bug Normal
Description

Steps to reproduce

  1. I have the following html: 
     <p>Some text...</p>
 <question>
   <answer>
      More text ...
   </answer>
 </question>

I am trying to replace the <question> element with another html string.

  1. My code: 
editor.getSelection().selectElement(element);
// here element is an instance of CKEDITOR.dom.element.
// Its nodeName property is 'QUESTION' in all browsers

editor.extractSelectedHtml();
editor.insertHtml(myString);
  1. This works fine in Chrome and Firefox but doesn't work in Safari.

In Safari I get the following error when running extractSelectedHtml: TypeError: undefined is not an object (evaluating 'a.startContainer.getDtd()'). The inner html of the <question> element gets deleted, but the <question> tag itself stays.

I tried to check what editor.getSelectedHtml method returns in Chrome and Safari. In Chrome it returns CKEditor.dom.documentFragment with question as firstChild, and in Safari it returns CKEditor.dom.documentFragment with answer (inner node) as firstChild.

Expected result

I would expect my code snippet to produce the same result in all browsers.

Is this working as intended? Is there any workaround for this issue in Safari?

#7507 Select element gets 'duplicated' in FireFox confirmed Bug Normal
Description

Paste the folowing code: 

<form action="formAct" id="myid" method="get" name="form">
	<select name="myselect" size="1"><option selected="selected" value="val1">opt1</option><option value="val2">opt2</option><option value="val3">opt3</option></select></form>
<p>
	<select name="myselect" size="1"><option selected="selected" value="val1">opt1</option><option value="val2">opt2</option><option value="val3">opt3</option></select></p>
  1. Place the cursor after select element in form and press enter


Result: seceond empty select element is created.

Bugs #7502 #7503 describe issues with select and eneter key for other browsers.

#16848 Selecting all triggering on certain characters pending Bug Normal
Description

Steps to reproduce

  1. Be on Windows and a Webkit browser
  2. Position the cursor in such a way that pressing Ctrl+A will, using the code added in ​https://github.com/ckeditor/ckeditor-dev/commit/c08aa2ca61a7119f56a9805a8b959ffa83f38457, select all editor contents
  3. Press Ctrl+*Alt*+A

Expected result

Ctrl+Alt+A is used to enter the character ą on a US keyboard layout. The ą character should be inserted.

Actual result

The editor's contents are selected.

Other details (browser, OS, CKEditor version, installed plugins)

Suggested fix (which actually probably explains the issue better than the above description)

plugins/widgetselection/plugin.js

Change:

`

if ( evt.data.getKey() == 65 && ( CKEDITOR.env.mac && data.metaKey
!CKEDITOR.env.mac && data.ctrlKey ) ) {

`

To add a !data.altKey check on Windows.

`

if ( evt.data.getKey() == 65 && ( CKEDITOR.env.mac && data.metaKey
!CKEDITOR.env.mac && data.ctrlKey && !data.altKey ) ) {

`

#12248 Selecting and deleting last entries of a list will remove the entire list confirmed Bug Normal
Description

Markup 

<ul>
    <li>1</li>
    <li>2
        <ul>
            <li>2.1)</li>
        </ul>
    </li>
    <li>3</li>
</ul>

Selecting bullets '2.1' and '3' (in wysiwyg mode) followed by hitting 'backspace' or 'delete' will remove the entire list. It seems as if such a selection causes the editor to select the outer 'ul' as well. The same selection can be deleted safely by adding another list element at the end of the list.

Reproducible: always (Firefox, Chrome and IE)

#10517 Selecting cells by tab doesn't work when tabSpaces > 0 confirmed Bug Normal
Description
  • Set config.tabSpaces = 4
  • Create table and try to select next cell by tab

Expected: next cell is selected
Outcome: 4 spaces are inserted

#12479 selection bug with widgets assigned Szymon Kupś Bug Normal
Description

Hi,

I'm struggling with a strange bug with the image2 (​http://ckeditor.com/addon/image2, version 4.4.4) plugin. If I insert two widgets with a caption and align the first one left and leave the align of the last one none, the first widgets becomes unselectable.

The problem was reproducible on the demo side ​http://ckeditor.com/demo#widgets

This code should reproduce the problem: 

<figure class="image" style="float:left"><img alt="Apollo-CSM-LM" src="http://c.cksource.com/a/1/img/demo/apollo-csm-lm.png" width="200" />
<figcaption>Apollo CMS-LM spacecraft</figcaption>
</figure>

<figure class="image"><img alt="Apollo-CSM-LM" src="http://c.cksource.com/a/1/img/demo/apollo-csm-lm.png" width="200" />
<figcaption>Apollo CMS-LM spacecraft</figcaption>
</figure>

If you inspect the selection wrapper of the widget with none align, it covers the first widget (​http://imgur.com/veQbbwE). Could be a problem with the css float?

I tested it in Chrome (latest) and Firefox (latest) on a Mac (Version 10.8.5)

Thank you for your help!

#6443 SelectionChange not firing in all instances confirmed Bug Normal
Description

Run the attached code example. Click to put the cursor at beginning of first line - event fired. Now drag select within the same line - event NOT fired. Now drag select within the second line - event fired.

#13367 Selection Dialog: Field value field is missing read-only styles. review Olek Nowodziński Bug Normal
Description

The Selection field properties form has (Value) text field. When putting the cursor on this field for editing, it becomes not responsive. This happens in both bidi and English forms.

Steps: 1- Click on the Selection button. 2- Put the cursor on value text field. 3- Try to add text to this field.

#9996 Selection disappears from last line on getData() confirmed Bug Normal
Description
  1. open ​http://nightly.ckeditor.com/13-01-28-08-51/standard/samples/plugins/enterkey/enterkey.html
  2. select BR mode
  3. remove all text
  4. type in some text (line 1)
  5. press enter
  6. type in some more text (line 2)
  7. select all text on the second line
  8. open javascript console and run 
CKEDITOR.instances.editor1.getData()

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

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

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

#8517 Selection Field content is broken in firefox when clicked on left , right justification plugins. confirmed Bug Normal
Description

Steps to reproduce:

1.Click on Selection Field plugin and add some values. 2.After adding the content to the editor, select the 'selection field' click on the 'right align'.

  1. You will observe that the Selection Field is broken.

Note: This happens in firefox.

#9177 Selection Field dialog box fails to work after switching to Source mode. confirmed Bug Normal
Description

To reproduce in ckeditor 3.6.4 and IE 9.

  1. Go to ckeditor demo page
  2. Clear the page
  3. Click on the Selection Field icon
  4. Click in the "Text" type "abc" in that field and click on add then ok
  5. Click the Source icon to view the html
  6. Click the Source icon again to return to WYSIWYG mode
  7. Place the curson on the Select Field just added and right click to bring up a menu
  8. On this menu, click on Selection Field Properties to bring up a dialog box
  9. Click the OK button in the dialog box and see that nothing happens

Clicking on the Cancel button properly closes the dialog box. Once you go to source mode and return, you cannot add any more select list items since the OK button is unresponsive.

#7842 Selection handles in FF can't be set or removed confirmed Bug Normal
Description

If you click (select) an image, table, anchor, ... in FF then the selection handles appear. Now try to reset the selection using javascript code :

oEditor.getSelection().reset();

OR

var doc = oEditor.document; var range = new CKEDITOR.dom.range(doc); var body = doc.getBody(); range.setStart(body, 0); range.setEnd(body, 0); oEditor.getSelection().selectRanges( [range] );

The selection is removed (this can be checked with selection.getSelectedElement), but the selection handles in the wysiwyg area remain.

Also, when trying to select an element using javascript code (oEditor.getSelection.selectElement()), the selection handles do not appear.

I think this is a bug in CKEditor and not in FF because it did work in the previous version FCKEditor.

#5614 Selection incorrect after Undo review Garry Yao Bug Normal
Description

Environment

IE, enterMode = CKEDITOR.ENTER_BR;

Reproducing Procedures

  1. Open any of the sample page and click on 'New Page' to clear all contents;
  2. Click on 'Insert Horizontal Line' 3 times to insert 3 <hr>;
  3. Undo once to revert one <hr>
  4. Strart typing some text
    • Actual Result: The inserted text appears after the first <hr>;
    • Expected Result: The inserted text appears after the second <hr>;
#10554 SELECTION IS MODIFIED IN A SPECIAL CASES confirmed Bug Normal
Description

STR (you should use IE8):
1) Paste the following *source* code into CK-DEMO source view (keep the 3 lines as is):

<pre>
window
</pre>

3) switch back to wysiwyg view
2) double click on the word "window" and change the font family.

RESULT: The first two letter are omitted from the selection.

#9130 Selection is wrong when using collapsed styles confirmed Bug Normal
Description

Tested on Chrome and Safari, didn't check the others.

  • Open a sample and clear the editor.
  • Hit Bold, Underline and Italics.
  • Type a bit. Text should be seen as expected.
  • Hit Bold, Underline and Italics.
  • Type a bit more.
  • Hit CTRL+A.

Selection jumps to to the beginning of text (Chrome) / first character (Safari).

#8441 Selection not updating when moving from left -> right confirmed Bug Normal
Description

The editor.on( 'selectionChange' ) is not called when text is selected with the mouse moving from left to right when using Chrome v14.0.

This can be veryfied in the nightly build: ​http://nightly.ckeditor.com/7293/_samples/replacebyclass.html

In order to replicate the bug place the cursor in between the bold text "sample text" and the link "CKEditor". Now drag the selection towards the left. Notice how the enabling of the bold action-button in the toolbar shifts to TRISTATE_ON and then to TRISTATE_OFF when past the bold text.

If the same is done from left to right (starting with the cursor to the left of the bold text and then dragging the selection towards the right then the bold action-button is not affected?!?

#13633 Selection should stay when switching between modes confirmed New Feature Normal
Description

I am not sure what category this comes under so would like advice:

Currently using CKeditor 4.5.1 in a drupal 7 install to edit some quite large body fields - usually just plain html.

When I find part of the text I want to edit and look at the source code I click on 'Source' button but I am then taken to the top of the page showing code. Many of my pages have a lot of code so re-finding the relevant bit can be very time consuming.

Can we not have this work so that when I click on 'source' it returns me to the point at which I was viewing when I pressed the button?

#14533 Selectors passed to CKEDITOR.dom.element.prototype.find are not properly escaped confirmed Bug Normal
Description

Steps to reproduce

  1. Open ​https://jsfiddle.net/5487w3gc/3/
  2. Click the button located under the editor.

Expected result

The alert with Test is shown.

Actual result

The alert with Error is thrown is shown.

Other details (browser, OS, CKEditor version, installed plugins)

The problem is located inside the getContextualizedSelector function inside core/dom/element.js file. It escapes only the id of the element. The proposed fix could split the selector on all special chars likely to appear in a CSS selector and escapes them separately: ​https://gist.github.com/Comandeer/de6c832993a8b9c21b01

Blocked on #14451 (as it introduces CKEDITOR.tools.escapeCss method).

#2232 Send the current element values as parameters to the FileBrowser (link dialog) confirmed New Feature Normal
Description

In the link dialog, upon a click on "Browse Server" the current FCKeditor simply calls a uri defined by the configuration value 

FCKConfig.LinkBrowserURL = "..." ;

Assume we defined a custom link browser. Of course we could use in our custom link browser the javascript command 

opener.GetE("txtUrl").value

to access the current value given in the url textfield of the link dialog. But this is a client side action. Wouldn't it be great to have also a server-side possibility to know the current value?

I propose the following: Add some keys (placeholders) to the LinkBrowserURL which then gets replaced by the current values. 

FCKConfig.LinkBrowserURL = "custom.php?url=URL&proto=PROTOCOL" ;

Implementation: It is easy to implement this feature. In the file "editor/dialog/fck_link/fck_link.js" replace 

function BrowseServer()
{
	OpenFileBrowser( FCKConfig.LinkBrowserURL, FCKConfig.LinkBrowserWindowWidth, FCKConfig.LinkBrowserWindowHeight ) ;
}

by 

function BrowseServer()
{
	var uri = FCKConfig.LinkBrowserURL ;
	uri = uri.replace( /URL/g, encodeURIComponent(GetE("txtUrl").value) ) ;
	uri = uri.replace( /PROTOCOL/g, encodeURIComponent(GetE("cmbLinkProtocol").value) ) ;

	OpenFileBrowser( uri, FCKConfig.LinkBrowserWindowWidth, FCKConfig.LinkBrowserWindowHeight ) ;
}
#6251 serverpreview plugin confirmed New Feature Normal
Description

Hello!

Someone suggested I add a ticket for an updated plugin made in 2006 for FCK.

The plugin adds a Preview button that POST's the currently edited html to an user-specified URL, where one could add a layout, replace template vars etc, in order to preview the page as it would look.

You can find the code here: ​http://cksource.com/forums/viewtopic.php?t=18603 or attached.

#831 Set background image to tables and cells confirmed New Feature Normal
Description

Make it possible to choose a background image for a table/cell.

Moved from SF:
​http://sourceforge.net/tracker/index.php?func=detail&aid=1235507&group_id=75348&atid=543656

#8010 setData from dialog makes empty undo step confirmed Bug Normal
Description

On FF4 dialogs who sets contents with setData methode makes empty undo step.
I've written a little plugin for testing. Set extraPlugins:'diatest' and 'DialogTest' as toolbar button.

#13902 SetData gets slower and slower new Bug Normal
Description

Steps to reproduce

I'm sorry as i was not able to reproduce this in a simple case, but i found useful to report this bug anyway since I found at least a part of the solution. We use multiple CKEditors in a react application and we only have this bug in a particular and complex situation. Each time we try to simplify the context, the bug disappears.

Actual result

Each time we call SetData, it gets 2 times slower. This may be related to this bug : ​http://stackoverflow.com/questions/21388256/ckeditor-memory-leak-on-setdata

The problem comes from the clearListeners function of the editable.js file. For an unknown reason, some listeners become undefined, and we get out of the function before all listeners are removed, causing "afterSetData" listeners number to be doubled each call. Simply moving the try/catch inside the while solves the problem, but doesn't explain why some listeners are undefined. Hope this helps.

Acy

Other details (browser, OS, CKEditor version, installed plugins)

Windows 7 64bit, Chrome, CKEditor 4.5.3

#12717 setReadOnly() does not work well with the Shared Space plugin confirmed Bug Normal
Description

My mistake, here is the correct way to reproduce:

1- Use Chrome 39.0.2171.71 m (although the bug also occurs in Firefox, the present procedure only works in Chrome, I don't know why).

2- Go to the official Shared Space demo: ​http://ckeditor.com/ckeditor_4.3_beta/samples/plugins/sharedspace/sharedspace.html

3- In Chrome's console, type: for (var i in CKEDITOR.instances) CKEDITOR.instances[i].setReadOnly(true); ==> All buttons are now disabled.

4- Click anywhere on the shared toolbar. ==> Three buttons are now enabled: "Paste", "Paste as plain text" and "Paste from Word".

#8570 setReadOnly in combination with getSnapshot throws an error in IE confirmed Bug Normal
Description

If you call setReadOnly and getSnapshot synchronously, an Error is thrown in IE (7-9, not tested in 10PP)

Reproduce:

  1. Go to ​http://ckeditor.com/demo
  2. Open the IE development tools
  3. Into the console, type in one line CKEDITOR.instances.editor1.setReadOnly(true); CKEDITOR.instances.editor1.getSnapshot();
  4. Hit enter
  5. An error occurs.

Working variation:

  1. Reload page
  2. Type into the console CKEDITOR.instances.editor1.setReadOnly(true);
  3. Hit enter
  4. Type CKEDITOR.instances.editor1.getSnapshot();
  5. Hit enter
  6. No error occurs

A workaround is to set readOnly first and do the rest of the work in a timeout.

#859 Set Spell Check Language confirmed New Feature Normal
Description

Hi Fred,

It would be great if we could set the spell check language independantly of the toolbar languauge.

For example, I may be an English user who has the task to writting / edit content in French. I'd still like the English toolbar (as that is my mother tongue), but would like to be able to spell check the document in the language it is composed in.

This would be very helpful.

Thanks,

Michael

Moved from SF:
​http://sourceforge.net/tracker/index.php?func=detail&aid=1410153&group_id=75348&atid=543656

#14733 setting Italic fails for Japanese character, in Japanese locale pending Bug Normal
Description

Steps to reproduce

  1. change all locale to Japanese
  2. open the demo site (​http://ckeditor.com/demo) and type some Japanese character
  3. select those Japanese character and set Italic

Expected result

the selected text will set to Italic

Actual result

the selected text will still keep non-Italic

Other details (browser, OS, CKEditor version, installed plugins)

Win10+IE11/Edge

OS locale set to Japanese Browser locale set to Japanese (try to make sure it's 100% Japanese... it's the key to reproduce. i have a Win10 VM that cannot reproduce this and i suspect it's because it's not set Japanese completely)

#9178 Setting the selected option in Selection Field doesn't work properly. confirmed Bug Normal
Description

To reproduce

  1. Go to the CkEditor demo page
  2. Clear all content from the page
  3. Click on the Selection Field icon to add a select list
  4. Place the cursor in the "Text" field, type "aaa" and click on the "Add" button
  5. Place the cursor in the "Text" field again, type "bbb" and click on the "Add" button
  6. Click on the "bbb" field just added which is below the "aaa" field
  7. Click Set As Selected Value to make "bbb" the selected value.
  8. Go to source mode and notice the that selected="selected" is set on both the "aaa" and "bbb" options (see html below) 
<p>
	<select><option selected="selected" value="">aaa</option><option selected="selected" value="">bbb</option></select></p>

I would expect that setting the "bbb" field would remove the selected="selected" tag on the "aaa" entry and place it on the "bbb" entry since the "Allow Multiple Selections" check box has not been checked.

#16664 Setting two styles with the same name for different widgets is not properly handled by stylescombo plugin. confirmed Bug Normal
Description

Style sets for widgets can be set like: 

CKEDITOR.replace( 'editor1', {
    stylesSet: [
        { name: 'Clean', type: 'widget', widget: 'image', attributes: { 'class': 'clean' } }
    ]
} );

or in styles.js: 

CKEDITOR.stylesSet.add( 'default', [
    { name: '240p', type: 'widget', widget: 'embedSemantic', attributes: { 'class': 'embed-240p' } },
    { name: '360p', type: 'widget', widget: 'embedSemantic', attributes: { 'class': 'embed-360p' } },
] };

When trying to set two styles with the same name but for different widget type, like: 

{ name: '240p', type: 'widget', widget: 'embedSemantic', attributes: { 'class': 'embed-240p' } },
{ name: '240p', type: 'widget', widget: 'embed', attributes: { 'class': 'embed-240p' } },

The stylescombo uses only style defined for latter widget type. It happens because stylescombo uses style names as ids internally and there is a collision so previous style gets overwritten.

In some cases there may be a need to provide same styles for different widget types.

Expected result

It is possible to set styles with the same name for different widget types. It is properly handled by stylescombo.

Actual result

Stylescombo cannot handle styles with the same name.

Generally, there may be two cases:

  1. Setting different styles with the same name for different widget types.
  2. Setting same styles for different widget types.

Enabling adding styles with the same name for different widget types solves both of the above issues, however, it is not optimal for the second one (optimal could be, e.g. passing array of widget types/names). So we might consider different approaches for this issue.

#1687 Setup initial HTML code for the new blank page confirmed New Feature Normal
Description

Hello. I've tried to find this kind of feature request without any success. So I think it will be very useful if the webmaster will be able to setup initial HTML code for the "blank" page created by the "new page" button. In this case all important parts, such as css file + special headers and footers, which correspond to the site design, will be inserted into the new file automatically.

Thank you.

#13967 sharedspace create div's with duplicated ids confirmed Bug Normal
Description

Steps to reproduce

I suppose it's good practice not create duplicate ids in an html document, but ckeditor 4.5.5 + sharedspace is doing just that.

the following html demonstrate the error: 

<!DOCTYPE html>
<html lang="en">
<head>
    <meta charset="UTF-8">
    <title>Title</title>
</head>
<body>


<div id="top-ck-toolbar">
    <!-- ckeditor top toolbar is rendered here -->
</div>
<div id="bottom-ck-toolbar" style="position: fixed;bottom: 0;width: 100%;">
    <!-- ckeditor bottom toolbar is rendered here -->
</div>

<div id="content-ck">
    <textarea id="mytextarea"></textarea>
    <!-- ckeditor editable area is rendered below -->
</div>

<script src="http://cdn.ckeditor.com/4.5.5/full-all/ckeditor.js"></script>
<script src="https://code.jquery.com/jquery-2.1.4.min.js"></script>
<script>
    var ckeditor_config = {
        extraPlugins: [
            "sharedspace",

        ].join(),
        sharedSpaces: {
            top: "top-ck-toolbar",
            bottom: "bottom-ck-toolbar"
        },
        on: { // execute after ckeditor full load
            instanceReady: function (evt) {
                //by default, ckeditor create a div with the id is named "cke_" + ID_OF_TEXT_AREA

                var elements_with_same_id = $('[id="cke_mytextarea"]');

                $(elements_with_same_id[0]).css({border: '3px solid rgb(183, 172, 19)' });
                $(elements_with_same_id[1]).css({border: '3px solid rgb(43, 183, 19)' });
                $(elements_with_same_id[2]).css({border: '3px solid rgb(183, 19, 19)' });
                alert("number of duplicated elements with 'cke_mytextarea' id: "+ elements_with_same_id.length);

            }
        }
    };


    //start ckeditor
    CKEDITOR.replace("mytextarea", ckeditor_config);


</script>
</body>
</html>

Expected result

No generate duplicate ids.

I would suggest that ckeditor "top toolbar" and "button toolbar" should be id prefixed, respectively, "cke_toolbar_top" and "cke_toolbar_button".

Then, in case I have a textarea with id "mytextarea", the generated id to "top toolbar" and "button toolbar" is respectively, cke_toolbar_top_mytextarea" and "cke_toolbar_button_mytextarea".

#14642 Shift+Enter after pressing backspace inserts two br's instead of one confirmed Bug Normal
Description

Steps to reproduce

  1. On the demo at ​http://ckeditor.com/demo clear the contents of the editor
  2. Type anything on the first line
  3. Press Enter
  4. Press Backspace
  5. Press Shift+Enter

Expected result

One BR is inserted

Actual result

Two BR's are inserted

Other details (browser, OS, CKEditor version, installed plugins)

Firefox, Mac OS (El Capitan), CKEditor Version 4.5.9

#13004 SHift Enter Mode = P is not working for .NET site confirmed Bug Normal
Description

We have used CKEditor as control as following: 

 <CKeditor:CKEditorControl ID="txt_bodyHTML" runat="server" BasePath="/jscripts/CKeditor/"
                    Visible="False" DisableNativeSpellChecker="false"   ShiftEnterMode="P"  EnterMode="BR" >
</CKeditor:CKEditorControl>

Shift Enter Mode is producing the BR

#9520 Shorter way for node's type check confirmed Bug Normal
Description

The current way is long and isn't minifiable at all: 

node.type == CKEDITOR.NODE_TEXT || node.type == CKEDITOR.NODE_ELEMENT

We may introduce shorter version. E.g. method 'is' may be added to node: 

node.is( '#doc' );
// or
node.is( [ '#text', '#element' ] );

Tokens have to be prepended with hash, because we already have is method to check element name. Also, it was used to duck typing, what would has to be fixed.

This solution's flaw is that additional check has to be done when method is called on element. Thus, we may think of other method's name specialized in type checking. E.g. node#node ( node.node( 'text' ) ).

#12605 Should `hasAscendant` match `getAscendant`s pattern? confirmed New Feature Normal
Description

Following on from the update to ​`getAscendant` should hasAscendant have the same parameter pattern?

#851 Show an icon in the place of protected source code confirmed New Feature Normal
Description

The current implementation of Protected Source puts a comment like <!--{PS..0}--> so the user has no indication that there is something special there or any way to change that content except switching to source code and finding it there.

A great improvement would be to show an icon in those places, like it is done with <a> anchors. And secondly be able to edit the protected code in a popup window.

Moved from SF:
​http://sourceforge.net/tracker/index.php?func=detail&aid=1387517&group_id=75348&atid=543656

#607 Show "Basic" toolbar when colapsed. confirmed New Feature Normal
Description

Hi,

The editor is great, and has a lot of features. But first-time users might be intimidated by all the features.

Maybe it is possible to use 2 Toolbarsets, Basic and Advanced. If you click the Collapse/Expand button, the toolbar could switch from Basic to Advanced.

This way the new users could start with a simple toolbar, and the experienced user can expand the toolbar and have all the available options.

Greetings, Arjen

Moved from SF:
​http://sourceforge.net/tracker/index.php?func=detail&aid=1089610&group_id=75348&atid=543656

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

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

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

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

#2398 Show Blocks should also show lists confirmed Bug Normal
Description

Lists should also be displayed by the "Show Blocks" command.

#7886 Show border plugin + custom table dialog review_failed Garry Yao Bug Normal
Description

The "showborders" makes assumption on existence of certain table dialog field thus get broken when table dialog definition is reduced.

#873 show config errors in filemanager confirmed New Feature Normal
Description

I've noticed that an ongoing issue for many is that setting up file and image upload is difficult. Part of the reason is there is no documentation about how paths to the File etc folders are created, and also there is no feedback if you get it wrong - nothing to help with fixing it.

In using the plugin ImageManager from Zhuo, I noticed that it was much easier to fix config problems with upload directories because the code reports, when it can't access the directory, giving the url that it is trying to use. With just that little bit of feedback, you can usually fix the problem quickly.

If FileManager had some inscreen display, similar to this, I suspect that a lot of those "help me with file upload" problems would go away - or at least some good suggestions could be made for peeple who get the error.

I suggest this as a high priority, since I see many people spending a long time fixing this (and a number giving up after hours or days and not being able to fix it) - and my own experience of 2 days worth of frustrating debugging, trying different upload plugins and code hacking, before I could figure out enough to fix the problems I was having.

The waves of relief and gratitude from implementing this feature would be massive :-)

Moved from SF:
​http://sourceforge.net/tracker/index.php?func=detail&aid=1466105&group_id=75348&atid=543656

#808 Show table size while changing size confirmed New Feature Normal
Description

It would be nice to see the table width height and differences to the original table while dragging the sizing buttons.

Moved from SF:
​http://sourceforge.net/tracker/index.php?func=detail&aid=1188944&group_id=75348&atid=543656

#16687 &shy; character breaks paragraph wrapping in Chrome confirmed Bug Normal
Description

Steps to reproduce

  1. Set <p>&shy;</p> as content of the contenteditable element
  2. Click into the editor
  3. Start typing

Expected result

The typed text is wrapped by a <p></p> tag.

Actual result

The typed text is not wrapped but inserted before the <p>&shy;</p>. When hitting the Enter-key a new <p></p> is inserted and from then on everything gets wrapped.

Environment

Google Chrome Version 54.0.2840.100 (64-bit) Windows & Linux.
I could not reproduce this behavior with Firefox ESR 45.5.0; in Firefox everything is wrapped nicely.

#9786 Silent try-catch blocks that covers more than a native code should log caught errors confirmed Bug Normal
Description

​http://dev.ckeditor.com/ticket/9706#comment:4

Again we were close to miss some important error because it was thrown in silent try-catch (this time in selectionChange listener).

We should review all try-catches in code and:

  1. if that's a try-catch that covers only one critical line of native code that can throw an error which we cannot handle differntly, then it's ok.
  2. all other try-catches should be:
    1. removed (may be unsafe, because we'd have to test what'd be the impact).
    2. logged - we need logs (at least in DEV mode - we can introduce CKEDITOR.DEV flag with %REMOVE_LINE% annotation) of caught errors. Now it'd be unsafe to unwrap e.g. selectionChange listeners, so at least we should know about what was caught.
#9640 Simplified skin icon dev registration review Garry Yao Bug Normal
Description

The skin icons registration code for dev looks like complicated than it should, it should be handy for skin developer.

#5105 Simplify getContentElement so it uses only elementId review New Feature Normal
Description

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

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

Are there any drawbacks?

#9703 Simplify plugin dialog open with language entries confirmed Bug Normal
Description

It's now over-killed to open dialog in plugin, which has language files defined, e.g. the a11yhelp plugin: 

	var langCode = editor.langCode;
	langCode =
		plugin.availableLangs[ langCode ] ? langCode :
		plugin.availableLangs[ langCode.replace( /-.*/, '' ) ] ? langCode.replace( /-.*/, '' ) :
		'en';

	CKEDITOR.scriptLoader.load( CKEDITOR.getUrl( plugin.path + 'dialogs/lang/' + langCode + '.js' ), function() {
		editor.lang.a11yhelp = plugin.langEntries[ langCode ];
		editor.openDialog( commandName );
	});

It would be as easy as just adding the available language list to the plugin meta, the language file loading should be handled be the dialog plugin instead.

#12929 Single entry point for command state confirmed Bug Normal
Description

The problem:

Now the command state (if it is disable or not) is based on the "setState" method which is called by many plugins. It can be changed when selection change, content in selection change (#12618), mode change, editor is in readOnly mode and maybe some more cases. Also many plugins set state of the commands manually. We get a state spaghetti and it is very often that one part of code try to disable command and another enable it back.

The solution:

There should be one entry point for all parts of code which want to change the state. Good candidate seems to be refresh method and event. The example scenario can be readonly feature. It will:

  • add refresh event listener and disable every command if editor is in the read only mode,
  • call refresh() function when editor switch to the readonly mode.

This way every time the state of the command is suppose to be changed all listeners are called and we can be sure that we do not enable feature which should be disabled.

In fact direct usage of setState() is always wrong because we can not be sure if any other plugin want to change the state too.

The idea of changes is:

  • mark setState() as deprecated,
  • refresh event get evt.data.state parameter which can be changed by any listener.
  • commandDefinition methods (refresh and exec) are only (other then command.refresh()) methods which can change state directly (by adding data.state parameter or returning new state),
  • refresh() method contents may be moved to the refresh listeners, so it will be possible to add listener before them,
  • everything what refresh() will do is firing refresh event and setting the state based on what listeners will set.

This way if someone want to disable command and be sure no one else will enable it back, can add refresh listener with 0 priority, set evt.data.state = CKEDITOR.TRISTATE_DISABLED and cancel event.

It seems to be possible to apply these changes without breaking backward compatibility. setState method will still work (as bad as it works now), but it will be deprecated. Also it is pretty huge change, because setState is used in many plugins.

#845 Single file for File Browser config (language independent) confirmed New Feature Normal
Description

I know FCKeditor is not phpMyadmin, but really irks me is not having one file only to make configuration changes. Diving deep to find the right config file to activate uploading, for example, is a pain.

I know you try to cover php, JS, asp etc., but one file would be so much easier to work with. 'myconfig.js' could be used - if exists - and if exists, it would have everything in it.

Moved from SF:
​http://sourceforge.net/tracker/index.php?func=detail&aid=1361798&group_id=75348&atid=543656

#7352 Single quotes in attributes converted to double quotes confirmed Bug Normal
Description

When using single quotes in html attributes they somehow get converted to double quotes and then htmlencoded to &quot;

In my case this happens when I try to do this in the code view: 

<a href="URL" style="font-family: 'Trebuchet MS';">Link</a>

Switch back to normal view and to code again, and CKEditor transformed it into this: 

<a href="URL" style="font-family: &quot;Trebuchet MS&quot;;">Link</a>

Not quite the desired result.

Is there a hotfix for this or do I have to wait for it to get fixed? (couldn't find it on Google)

Edit:

I did some checking. Here are the results:

  • IE9 and IE10 - change ' character to &quot;
  • IE6-IE8 - delete the ' character
  • Webkit, Opera and Firefox leave the ' character untouched.

Since editor 4.x is creating HTML from scratch in some places perhaps it should introduce "template behaviour" in which " goes outside and ' goes inside (or something like that).

#10639 Single Space Does Not Cause Lines to Re-Wrap Correctly confirmed Bug Normal
Description

Repro steps:

Open the demo page. Place the cursor at the beginning of any line of a paragraph other than the first. Press backspace (deleting the previous space, causing the last word on the previous line to wrap to this line) Press space.

Expected: The line to re-wrap and the first word to return to the previous line, like it was originally. Saw: The re-separated words both stay on the second line.

Entering a second space causes the lines to rewrap appropriately.

​http://i.imgur.com/1mBJXtp.gif

#10556 Size (Font Size) menu dropdown jumps to page top when user clicks on it on chrome browser. confirmed Bug Normal
Description

Go to Ck Editor online Demo page on Chrome browser> Click on Full Featured> Click on Size (Font Size) menu.

Font Size dropdown with its items will go on page top.

#14604 [Skin building] Some versions of ImageMagic may corrupt icons while processing. confirmed Task Low
Description

While generating new icons using iconmaker.js script (./dev/iconmaker/iconmaker.js) we discovered that some icons generated on Linux and Mac differs.

On Linux current version is:

Version: ImageMagick 6.7.7-10 2014-03-06 Q16 ​http://www.imagemagick.org

and on Mac:

Version: ImageMagick 6.9.3-7 Q16 x86_64 2016-03-27 ​http://www.imagemagick.org

The problem is with newer version on Mac (which will probably land one day on Linux too) for specific icons it adds some blurring/background (not sure how to call it). It can be seen on attached screenshot comparing generated icons (left - mac, right - Linux).

It's not a big deal for now but we should keep in mind that such problem exists and may return in future.

ImageMagick convert method which is used by the script have ​plenty of options and maybe some of them may help to solve the issue.

#12616 Smart drop position plugin confirmed New Feature Normal
Description

If user drop a big image it should not break paragraph.

#9591 Smiley dialog - is the dark background for smileys really needed? confirmed Bug Normal
Description

Maybe it could be improved a bit?

Current design:

Less distracting background:

#2752 Smiley face short hand confirmed New Feature Normal
Description

Hi :)

When adding Smiley faces in FCKEditor I have to search for the smile, choose it, and then click ok.

Would it be possible to simply type a smiley shorthand for core smiley faces (the common ones) -

:) :( :x

And have the smiley face replaced with the corresponding image? Shorthand could be designated in the config file.

When the user presses save this replacement is saved out also. I'm able to do this by manipulating the oFCKeditor.Value string before it is saved. I search for :) :( etc and swap them for images, but I can't show this in the Editor at the moment.

Thanks,

Rob

#5482 Smiley icons preload confirmed Bug Normal
Description

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

#12366 Smileys plugin should use CKEDITOR.getUrl() to resolve images' paths confirmed Bug Normal
Description

However, it cannot add timestamp so we need a parameter in getUrl().

Extracted from #10804.

#4617 some of the Align field's options' captions are being amputated in IE confirmed Bug Normal
Description

to reproduce:
open the "multi-language interface" demo and choose a language that has long caption in the align field (Hebrew or Hungarain for example).
open the image dialog.
open the "align" text box to see all the options. notice that some of them are being amputated.

#16620 Source button disabled after block widget copy-paste confirmed Bug Normal
Description

Steps to reproduce

  1. Open ​http://tests.ckeditor.dev:1030/tests/plugins/widget/manual/block
  2. Insert a doublecolumn widget(empty icon).
  3. Cut the code snippet widget that's placed lower and paste it into the first editable in the doublecolumn widget.
  4. Click the Source button.

Expected result

Mode switched to source.

Actual result

Source button gets disabled.

Other details (browser, OS, CKEditor version, installed plugins)

Browser: Edge

#853 sourcecode view improvement confirmed New Feature Normal
Description

Hi,

I think personally it would be great if the source code view had wrap="off" to prevent line breaks where the source code actually continues and also the ceaseing of line breaking charactar striping so that you can indent source code and make new lines in the code without that disappearing on "OK".

Example. 

<script language="javascript" type="text/javascript">
alert('yay!');
</script>
<?php
echo "Hello again!";
?>

--> Upon click becomes 

<script language="javascript" type="text/javascript">
alert('yay!');
</script> <?php
echo "Hello again!";
?>

That line breaker should be left intact?!

Thanks! Keep it up, love 2.2!

Moved from SF:
​http://sourceforge.net/tracker/index.php?func=detail&aid=1390185&group_id=75348&atid=543656

#10159 Sourcedialog and Sourcearea plugins - better interoperability needed? confirmed Bug Normal
Description

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

1. Do not use different button names

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

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

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

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

2. sourcedialog could be shipped in std/full package

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

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

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

3. sourcedialog for inline editing sourcearea for framed mode?

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

#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
#10176 Sourcedialog Resizing confirmed Bug Normal
Description

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

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

#16995 Source mode messes iframe[src] attribute confirmed Bug Normal
Description

Steps to reproduce

  1. Open ​sample.
  2. Copy content of the textarea under the editor.
  3. Switch editor into the source mode.
  4. Paste copied code into the editor.
  5. Switch back to WYSIWYG mode.

Expected result

Empty iframe is visible inside the editor.

Actual result

Iframe with "If it's shown, the test is failed" text inside.

#1632 Source view: Activate undo/redo tool buttons confirmed New Feature Normal
Description

Browsers: Firefox 2.0.0.11 (win+lin), IE6+7

Shortcuts for undo and redo works fine when in source view, but the toolbar buttons is deactivated. The "Select all" toolbar button is active and it would be nice to have the undo/redo buttons active as well.

#9861 Spacebar keypress ignored in last cell of row when first cell expands full length confirmed Bug Normal
Description

To reproduce on demo page -Click the insert Table button

  • In the first cell of the first row, keep entering text until the line wraps.
  • Press the tab key (it will move you to the last cell of that row
  • Start entering text, you will notice any press of the space bar is ignored when at the end of your text line. If you navigate backward in the string, you are able to enter spaces.
#8552 Spaces lost on copying during editing confirmed Bug Normal
Description

Take

(a) the Authority;

(b) the Commissioners;

(c) the OFT; and

(d) in relation to credit unions in Northern

and copy it into ​http://ckeditor.com/demo - ensure there are six spaces after each closing bracket. Then copy the same text within the demo and paste it back into the demo page - five of the six spaces disappear''''' This is very frustrating. Using IE9 and Windows7 64-bit and CKEditor 3.6.2 (revision 7275)

#12569 Span gets removed or changed when pasting text. confirmed Bug Normal
Description

Use below code in contenteditable element or paste it into editor: 

<table border="0" cellpadding="0" cellspacing="0" style="width:100%">
  <tbody>
    <tr>
      <td align="center">
        Plain text pasted here
        <br>
        <span style="font-size:8pt;">
          PDF, 378 KB
        </span>
      </td>
    </tr>
  </tbody>
</table>

Now copy this Chinese text 字讀音,臺灣教育部異體字字典標同 (can also be done with european text), select PDF, 378 KB and paste the chinese text.

In Firefox span gets removed in editor and native contenteditable.

In Blink and IE:
In editor: 

<td>Plain text pasted here<br><span style="font-size:8pt"></span>字讀音,臺灣教育部異體字字典標同<span style="font-size:8pt"> </span></td>

In native contenteditable: 

<span style="text-align: start;">字讀音,臺灣教育部異體字字典標同</span>

I don't think we can do anything about Firefox but perhaps there is room for improvement for Blink and IE.

#14302 span tag replaced by font confirmed Bug Normal
Description

Steps to reproduce

  1. Go here ​http://sdk.ckeditor.com/samples/acf.html in section 'Adjusting Automatic ACF Mode'
  2. Select the text in green 'text colors' and click on backspace
  3. Type a new text

Expected result

Only the new text should be added

Actual result

A font tag with the previous color is added

Other details (browser, OS, CKEditor version, installed plugins)

Linux Fedora, chrome Version 40.0.2214.115 (64-bit). I also saw the bug on windows chrome and firefox

#10649 SPECIAL CHARACTER INSERTED IN WRONG LOCATION WITH SPECIFIC STEPS confirmed Bug Normal
Description

Description: An inserted special character is being inserted at the end of the previous paragraph even though the insertion point was in a new paragraph when I opened the special characters dialog. I can only recreate the problem with these specific steps.

Win7 32-bit, IE9 Steps To Recreate:

  1. Get a new page on DEMO site.
  2. Insert a numbered list and enter text for a few items in the list.
  3. Press [Enter] after entering text for one of the numbered items to create another numbered item, but don't add any text.
  4. Move the mouse pointer over the last numbered item to display the red 'Insert Paragraph Here' tool.
  5. Click the Insert Paragraph Here button. A new paragraph is inserted after the last numbered item.
  6. Click back into the last numbered item (which has no text) and press [Backspace] to remove this item.
  7. Click below the numbered list so that the insertion point is in the new paragraph you inserted in step 5.
  8. Click the Insert Special Character button on the toolbar and select some character to insert.

The special character is appended to the text of the last numbered list item instead of in the new paragraph which is where the insertion point was.

Note: You should select really a special char e.g. select the black diamond from the last line. See attached picture.

#899 Special Characters for Esperanto missing in the table confirmed New Feature Normal
Description

Please note, the ten special characters for esperanto Ĉĉ Ĝĝ Ĵĵ Ŝŝ Ĥĥ Ŭŭ should be inserted in the characters table hope soon!

Atentigo: la dek specifaj karakteroj por Esperanto, t.e. Ĉĉ Ĝĝ Ĵĵ Ŝŝ Ĥĥ Ŭŭ mankas en la tabelo. Bonvolu enigi ilin plej eble baldaŭ!

Moved from SF:
​http://sourceforge.net/tracker/index.php?func=detail&aid=1497416&group_id=75348&atid=543656

#16689 Special char dialog entries unnecessarily encoded confirmed Bug Normal
Description

Steps to reproduce

  1. Open CKEditor with Special Characters plugin.
  2. Use "Insert Special Character" button to open the dialog.
  3. Mouse over characters like &.

Expected result

Tooltip shows &.

Actual result

Tooltip shows &amp;.

Other details (browser, OS, CKEditor version, installed plugins)

Same thing happens e.g. for <, > characters.

#3811 Special chars dialog is slow confirmed Bug Normal
Description

A very slow feedback is given when moving the mouse over the characters in the special chars dialog. This is specially noticeable with IE.

#10695 Speech to text functionality confirmed New Feature Normal
Description

Submitted in Request ID: 1791

"We'd like to see the ability of your program to be functional within CKEditor so people who are poor typers can dictate what they want into a well known website editor. Any possibility of that, maybe as a plugin? Our experience with this suggests that using a headset is far more effective at getting correct input than a microphone built into a webcam."

Some links to show how speech recognition works:

​DEMO 1 (one of 3 pgms/apps Google lists with a similar function)

​DEMO 2

​DEMO 3

​DEMO 4

​DEMO 5

#8123 Spellchecker: provide an option to pass sensitive data using POST request confirmed WebSpellChecker.net New Feature Normal
Description

Right now the built-in spell checker sends the content entered by user using GET requests (at least after clicking "Finish checking"). It is causing problems for some companies, where security filters block certain URLs with specific keywords.

It would be nice to at least make it configurable (to allow user to choose POST instead of default GET for such requests), if not simply switch to POST by default.

Note that sending POST requets instead of GET would also allow spellchecker to split content into larger chunks. This is causing serious performance issues, as explained in #6119.

Sample URL:

/ckeditor/plugins/wsc/dialogs/ciframe.html?id=58745&cmd=part&data=g!2520teeth!2520you!2520have!2521!2526quot!253B!253Cbr!2520/!253E!250D!250A!2509To!2520which!2520the!2520wolf!2520replies!252C!2520!2526quot!253BThe!2520better!2520to!2520eat!2520you!2520with!252C!2526quot!253B!2520and!2520swallows!2520her!2520whole!252C!2520too.!253C/p!253E!250D!250A!253Cp!253E!250D!250A!2509A!2520!253Ca!2520href!253D!2522http!253Aen.wikipedia.org/wiki/Hunter!2522!2520title!253D!2522Hunter!2522!253Ehunter!253C/a!253E!252C!2520however!252C!2520comes!2520to!2520the!2520rescue!2520and!2520cuts!2520the!2520wolf!2520open.!2520Little!2520Red!2520Riding!2520Hood!2520and!2520her!2520grandmother!2520emerge!2520unharmed.!2520They!2520fill!2520the!2520wolf!2526!252339!253Bs!2520body!2520with!2520heavy!2520stones!252C!2520which!2520drown!2520him!2520when!2520he!2520falls!2520into!2520a!2520well.!2520Other!2520versions!2520of!2520the!2520story!2520have!2520had!2520the!2520grandmother!2520shut!2520in!2520the!2520closet!2520instead!2520of!2520eaten!252C!2520and!2520some!2520have!2520Little!2520Red!2520Riding!2520Hood!2520saved!2520by!2520the!2520hunter!2520as!2520the!2520wolf!2520advances!2520on!2520her!2520rather!2520than!2520after!2520she!2520is!2520eaten.!253C/p!253E!250D!250A!253Cp!253E!250D!250A!2509The!2520tale!2520makes!2520the!2520clearest!2520contrast!2520between!2520the!2520safe!2520world!2520of!2520the!2520village!2520and!2520the!2520dangers!2520of!2520the!2520!253Ca!2520href!253D!2522http

#6555 Spell checker: unable to correct mistakes in large documents confirmed Bug Normal
Description

When using the "Check Spelling" feature, spell checker checks all the mistakes pretty quickly, however it has serious problems with applying changes that users is trying to make.

Steps to reproduce

  • Load the attached document in source mode
  • Switch to wysiwyg mode
  • Click on a "Check Spelling" button
  • After the dialog is loaded, scroll down to the bottom of the article (inside of the dialog)
  • Find "14th" (which is highlighted), click on it, select "Utah" from Suggestions
  • Click "Change to"
  • Result: after waiting for over a minute, the word is still not changed
#981 Spellcheck - Extra Button to stop spellcheck and apply the changes made upto that point. confirmed New Feature Normal
Description

Spellcheck - Extra Button to stop spellcheck and apply the changes made upto that point.

With the spellcheck... you can keep correcting errors... and it is only until the last correction is made... that it then applies it.

There is the cancel button... that cancels everything.

Well... there are some times... that I am using the spellcheck... and I want to stop after a few corrections... and actually apply the corrections I have made up to that point. (ie) Not a total cancellation. It would be good if there was a new button that allowed for this to happen.

#11255 split merged cell vertically command transposes the cells confirmed Bug Normal
Description

Description (I attached a detailed image that shows every aspect of this issue well - please see it):
1) Create a cell and fill them with numbers - just for knowing where the cells moves at the end.
2) Merge down a cell at the right edge of the cell.
3) Split the merged cell vertically.

RESULT: The new cell comes to the first place of the affected row instead of the place where the split command was applied. Every other cells are shifted right. Note: If the merge/split action is applied to an intermediate cell (not on the last one in the row), there is no issue.

I can recreate it with the demo page with IE9, IE10.

#8177 Splitting cell vertically, causing incorrect rowspan. confirmed Bug Normal
Description

Environment

Related to #6111

TC

  • Open editor and insert table: 
    <table border="1" cellpadding="1" cellspacing="1" style="width: 500px;">
	<tbody>
		<tr>
			<td colspan="2" dir="rtl">
				<strong>1</strong></td>
			<td colspan="2">
				<strong>2</strong></td>
		</tr>
		<tr>
			<td>
				<strong>3</strong></td>
			<td>
				podział poziomy ltr</td>
			<td dir="rtl">
				<strong><u>4</u></strong></td>
			<td dir="rtl">
				podział poziomy rtl</td>
		</tr>
		<tr>
			<td colspan="2" dir="rtl" rowspan="3">
				<strong>5</strong></td>
			<td colspan="2">
				<strong>6</strong></td>
		</tr>
		<tr>
			<td colspan="2">
				podział pionowy ltr</td>
		</tr>
	</tbody>
</table>
<p>
	&nbsp;</p>
  • Switch to wysiwyg, and set cursor in cell with no. 1
  • from context menu select Cell - split vertically.

Actual result

<table border="1" cellpadding="1" cellspacing="1" style="width: 500px;">
	<tbody>
		<tr>
			<td colspan="2" dir="rtl">
				<strong>1</strong></td>
			<td colspan="2" rowspan="3">
				<strong>2</strong></td>
		</tr>
		<tr>
			<td colspan="2" dir="rtl">
				podział pionowy rtl</td>
		</tr>
		<tr>
			<td>
				<strong>3</strong></td>
			<td>
				podział poziomy ltr</td>
			<td dir="rtl">
				<strong><u>4</u></strong></td>
			<td dir="rtl">
				podział poziomy rtl</td>
		</tr>
		<tr>
			<td colspan="2" dir="rtl" rowspan="3">
				<strong>5</strong></td>
			<td colspan="2">
				<strong>6</strong></td>
		</tr>
		<tr>
			<td colspan="2">
				podział pionowy ltr</td>
		</tr>
	</tbody>
</table>
<p>
	&nbsp;</p>
#14430 Splitting/merging table cells messes up table layout confirmed Bug Normal
Description

Steps to reproduce

  1. Create a table with 7 row, 5 cols. See attachment starting-table.png.
  2. Merge right cell B3.
  3. Merge right cell C4.
  4. Split vertical cell B4.
  5. Merge down cell E5. Then split vertical.
  6. Split horizontal cell C4.

Expected result

Step 6 should split the cell horizontally and table layout should be maintained properly.

Actual result

The table layout gets messed up. See resulting-table.png.

#14230 Split uploadImage and image paste listeners confirmed New Feature Normal
Description

The idea of the uploadImage was to make it super simple so one can replace it with different upload widget if one want to have a different behavior. And in fact it is pretty simple, the only complex part is the paste listen. But this listener is in fact independent part of code and could be exported to another plugin, so custom upload image widgets does not need to load uploadImage plugin.

← 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