Custom Query
Results (801 - 900 of 2591)
Ticket | Summary | Status | Owner | Type | Priority | Milestone |
---|---|---|---|---|---|---|
#10306 | Anchor allows spaces in id attribute. | confirmed | Bug | Normal | ||
Description |
According to http://www.whatwg.org/specs/web-apps/current-work/multipage/elements.html#the-id-attribute - there can't be any spaces in id attribute. If you put in editor code like: <p><a href="#first anchor">#first anchor</a></p> <p>Lots of P</p> <p><a id="first anchor" name="first anchor"></a></p> it will work in e.g. Chrome or IE9. If you press preview button then click on link at the top you will see it works and browsers properly do the "jump" to the bottom. If you put this code in HTML5 Validator you will get 2 messages:
I think that in this case first message is the one we should focus on and not allow entering spaces into anchor. Problem can be reproduced from CKEditor 3.6.5 rev. [7618] when id for anchor was introduced. I have written "if you paste such code" but same code is produced when you create anchor with dialog and then create link to anchor with another dialog. |
|||||
#10319 | Link Properties Dialog should search both Encoded and UTF-8 values | confirmed | Bug | Normal | ||
Description |
Background:
Real-life use case: CKEditor issue: Beacuse of the above, code loaded into editor may look like: <p><a href="#els%C5%91">line1</a></p> <p>many lines...</p> <p><a id="első" name="első">line101</a></p> If you doble-click on link, link properties dialog will show but value első will not be shown as selected in dropdown.
I think editor should look for both UTF-8 and encoded value (encodeURIComponent perhaps). Problem can be reproduced from CKEditor 3.0 in both CKE 3.x and CKE 4.x (v4) |
|||||
#10336 | Filter should understand complex styles | confirmed | New Feature | Normal | ||
Description |
Currently it won't create rules from following styles: font_style: { element: 'span', attributes: { 'class': '#(family)' }, overrides: [ { element: 'span', attributes: { 'class': /^Font(?:Comic|Courier|Times)$/ } } ] }, fontSize_sizes: 'Smaller/FontSmaller;Larger/FontLarger;8pt/FontSmall;14pt/FontBig;Double Size/FontDouble', fontSize_style: { element: 'span', attributes: { 'class': '#(size)' }, overrides: [ { element: 'span', attributes: { 'class': /^Font(?:Smaller|Larger|Small|Big|Double)$/ } } ] } Perhaps this can be a part of #9991. |
|||||
#10340 | DTD caching makes it impossible to modify DTD even before first editor creation | confirmed | Bug | Normal | ||
Description |
CKEDITOR.dtd.customtag = { em:1 }; CKEDITOR.dtd.$block.customtag = 1; CKEDITOR.dtd.body.customtag = 1; Load: <customtag>foo</customtag> It will be transformed to: <customtag></customtag> <p>foo</p> I think that we don't have to rewrite all code which caches DTD objects because that would ruin performance or increase complexity in some places. Therefore we can introduce event CKEDITOR#dtd fired when DTD is created and allowing its modifications. EDIT: The problem is DTD caching makes it impossible to extend it after loading CKEditor source. |
|||||
#10356 | "learn mode" for ACF | confirmed | New Feature | Normal | ||
Description |
Many people will disable ACF because it modifies whatever content exists upon initial load if the classes, attributes, styles... haven't been added to the allowedContent. A simple example is the stylesheet parser: you have been forced to make it disable the ACF in order to have things work. Instead, the ACF could have an optional "learn mode" that adds whatever exists in the initial source to the allowedContent for that instance (maybe even configurable to work only on elements/classes/attributes/styles) So now the stylesheet parser doesn't need to completely disable ACF: the first run of ACF sees some classes that aren't included in the ACF, so it adds those classes. Then the stylesheet is loaded and adds all the desired classes for the content. Nothing has been lost. The user then tries to paste something from an external page and the ACF is run and cleans up removing anything undesired. If the user copies and pastes something inside the editor, then all that content now is allowed and no filtering happens. everyone is happy. |
|||||
#10358 | [IE]:Editor removes if after ­ | confirmed | Bug | Normal | ||
Description |
Result: Problem occurs in all versions of IE from CKEditor 4.0 (it works in CKE 3.0 - 4.0 beta). |
|||||
#10359 | [Webkit] Extra and inline styles when copying and pasting aligned text | confirmed | Bug | Normal | ||
Description |
Follow the cases below. Set initial data on the editor, then C-a, C-c, C-v and get data.
replacebycode (extra , inline style on
|
|||||
#10362 | Introduce editor#feature event | confirmed | New Feature | Normal | ||
Description |
Based on: http://ckeditor.com/forums/CKEditor/How-to-remove-attributes-which-are-removed-from-dialog-windows It is not possible to modify feature's properties in convenient way or in some cases at all. Editor#feature should be fired when feature is registered by filter#addFeature, but before its rules/transformations/etc are registered. This should be possible: CKEDITOR.replace( 'editor1', { on: { feature: function( evt ) { if ( evt.data.name == 'table' ) evt.data.allowedContent = 'table[border,summary];caption tbody thead tfoot;th td tr[scope]'; } } } ); |
|||||
#10392 | Dropdowns don't scroll to top when opened. | confirmed | Bug | Normal | ||
Description |
To reproduce:
Result: It starts scrolled down rather than back at the top.
This is happening because we use 1 float panel for dropdowns and don't reset on every open. Sure when opening new dropdown user would expect it to be at the top. Since we are using one float panel we can either live with it or perhaps remember last dropdown opened and if current dropdown is different scroll it up. For those who want to have float panel always scrolled up please use below code: var editor = CKEDITOR.replace( 'editor1', {} ); editor.on( 'panelShow', function( ev ) { ev.data._.iframe.$.contentWindow.scrollTo(0,0); });
Another solution would be adding onOpen methods to below plugins (e.g. after onRender):
\ckeditor4-git\plugins\format\plugin.js onOpen : function(){ this._.panel._.iframe.$.contentWindow.scrollTo(0,0); }, |
|||||
#10400 | CKEDITOR(inline) crashes when using ol/ul list | confirmed | Bug | Normal | ||
Description |
I am using Inline CKEitor version 4.0.2. I have a ol which has many li tags. In each li, there are contenteditable divs initialized with Inline CKEditors. Now I create a ol/ul list inside the contenteditable div. Next I add some more content below the ol/ul list. Then I select the complete text below the list and complete text of any li together(see attachment for selection) and press enter, the editor crashes. Here is the stacktrace of the error: Uncaught TypeError: Cannot call method 'getParent' of null ckeditor.js:139 CKEDITOR.dom.range.setStartBefore ckeditor.js:139 CKEDITOR.dom.range.moveToBookmark ckeditor.js:123 CKEDITOR.dom.selection.selectBookmarks ckeditor.js:331 p.exec ckeditor.js:616 exec ckeditor.js:162 CKEDITOR.tools.extend.execCommand ckeditor.js:195 CKEDITOR.plugins.enterkey.enterBlock ckeditor.js:619 m ckeditor.js:618 a.addCommand.exec ckeditor.js:619 exec ckeditor.js:162 CKEDITOR.tools.extend.execCommand ckeditor.js:195 c ckeditor.js:171 h ckeditor.js:10 CKEDITOR.event.CKEDITOR.event.fire ckeditor.js:12 (anonymous function) Note: This does not happen if the text is not selected completely either of the li or the text below it. Also, it happens only upon clicking enter. There is no error on clicking backspace. And it does not happen if my CKEdtitor div is not present in an li tag of an ol. Here is the jsfiddle demonstrating the issue: http://jsfiddle.net/FLZhn/1/ I have fount this bug also in 4.0.1 and 4.1.1 Let me know if you have any other queries. Thanks. |
|||||
#10411 | Bookmarks are rooted to documentElement. | confirmed | Bug | Normal | ||
Description |
Issue reported on our support channel. In CKEditor 4.1, Image bookmarks are rooted to the documentElement (see CKEDITOR.dom.element::getByAddress). This is probably fine if the editor is in an iframe, but problematic if the 'divarea' plugin is enabled. If an element is inserted before the editor, the selection will be placed in the wrong position (or get an error) when invoking undo. To reproduce:
Result: Error is thrown - IndexSizeError: Index or size is negative or greater than the allowed amount. Problem can be reproduced from CKEditor 4.0 beta. |
|||||
#10412 | IE7-8: Undo moves cursor to beginning of div | confirmed | Bug | Normal | ||
Description |
Problem can be reproduced from CKEditor 3.0 in both CKE 3.x and 4.x (v4). To reproduce:
Result: cursor is moved to the beginning of inner div |
|||||
#10427 | Menu Styles get broken in compatibility view | confirmed | Bug | Normal | ||
Description |
This is continuation of #10292 It seems that menu styles can get broken in IE8-10 when XUA is set to IE8 and compatibility view is turned on. To reproduce:
Problem can be reproduced from CKEditor 4.0 with both skins (kama and moono). |
|||||
#10431 | BR tag replacement seems to be to agressive. | confirmed | Bug | Normal | ||
Description |
Results: <p> </p> <br /> <p> </p>
Enter_Div: Br and second P gets wrapped in div. I'm not sure if this is error or not. Perhaps this is how this mode should work? <p> </p> <div><br /> <p> </p> </div> Enter_P: You get 4 P tags. This is definitely an error as there should be maximum 3 p tags. It looks like code responsible for br replacement matches one P tag to many. <p> </p> <p> </p> <p> </p> <p> </p>
Problem in enter mode P can be reproduced from CKEditor 4 beta. |
|||||
#10432 | [IE9-10] Image properties don't get updated. | confirmed | Bug | Normal | ||
Description |
To reproduce:
Result: width and height fields are empty. Problem can be reproduced in IE9 and IE10 from CKEditor 4.0 Beta |
|||||
#10439 | Initial command states are naive, biased and buggy | confirmed | Bug | Normal | ||
Description |
The problem emerged while testing #10027. SymptomsWe expect command states to be properly set when editor is ready. A state of every context-sensitive command should reflect an initial element path of the editor, the same way command states are updated when selection start changes and element path is updated.
This is, however, working not quite as good as expected. Some commands (and UI buttons) have wrong states when editor starts. This suggests that some actions are unavailable when, apparently, they can be used immediately before clicking in editable and displaying the caret. Of course, the issue has alternatives: some features are
We missed this bug because in most cases "wrong" command states match the initial content of the editor and it's hard to see this as there are so many buttons to be checked. Also we introduced Examples
Open attached commandstates.html in Chrome (the last chapter explains why). Use The first two samples show naive unlink command behavior. Play with startDisabled property to see awkward situations. The third sample should be working just fine, at least in Chrome. The fourth sample explains the origin of this ticket. Note: when you click the very first editable element in editable (link, paragraph, list element), states are automatically corrected. Code
So far, the origin of this issue is expected to be within
The early debugging (Chrome only) revealed that This can be "fixed" by changing a single line of code to something like this: this.on( 'mode', function() { updateCommands.call( this ); this.forceNextSelectionCheck(); this.selectionChange( 1 ); } );
This piece of code is nothing like a gentle solution though. It's ugly and dangerous since it forces FF and IE "thing"The dirty fix doesn't work in FF and IE (but it could).
Please note that justify sample doesn't work in FF and IE at all (all justify commands are TRISTATE_OFF). This is because the initial CKEDITOR.instances.editor1.elementPath().lastElement
Unfortunately, if we want to fix initial command states, the initial Further testingI'd love to see all commands tested with different initial contents of the editor. To save time, we could choose 10 or 15 critical and very common commands (buttons) and create tests to assert initial states. This would protect us from further troubles both in inline and framed instances. We deal with this kind of bugs very often: in almost every single release some command states are fixed. Even if there are tests that check initial states of some commands, they're in different files, hard to develop and control. This approach is weak and we lose time because of this. Required fixes |
|||||
#10442 | Automatic features registration for menubuttons | confirmed | New Feature | Normal | ||
Description |
Currently only buttons added to toolbar are automatically registered as features. We should also register positions of menubuttons (dropdowns). Based on: #10440. |
|||||
#10444 | The order of several events differs among creators | confirmed | Bug | Low | ||
Description |
Extracted from ticket:10370#comment:3. Most likely this issue requires lots of coding with no significant improvement to the editor.
Problem is that This can be checked with: var editor = CKEDITOR.inline( 'editable' ); editor.on('contentDom', function(){ console.log('contentDom'); }); editor.on('dataReady', function(){ console.log('dataReady'); }); |
|||||
#10445 | IE8 - JS error is thrown when you right-click on Magic Line | confirmed | Bug | Normal | ||
Description |
To reproduce:
Message: 'getRanges().0' is empty or not an object Problem can be reproduced only in IE8 |
|||||
#10452 | Image size doesn't get refreshed in Image Preview | confirmed | Bug | Normal | ||
Description |
Result: Width field is showing 22 but image size in preview is the same as first loading the page. I was able to reproduce this in demo and nightly in all browsers. I wasn't able however to reproduce this on my local machine in default editor (not sure why). |
|||||
#10459 | Improve English texts | confirmed | Task | Normal | ||
Description |
With CKEditor 4 released it's high time to revise the wording of some of our commands and UI elements. |
|||||
#10463 | Clipboard toolbar becomes accessible in read-only mode | confirmed | Bug | Normal | ||
Description |
Problem can be reproduced from CKEditor 4.0 beta in Opera and Firefox.
Result: Cut icon gets enabled. You can't cut text but this doesn't look nice. This TC is also reproducible in Webkit but enables more buttons plus Webkit has another TC to enable buttons thus it is reported here: #9627 |
|||||
#10473 | Introduce argument for range#cloneContents which will preserve ids | confirmed | New Feature | Normal | ||
Description |
Extracted from #10469.
It would be cool if |
|||||
#10483 | Incorrectly focused editable after switching between modes | confirmed | Bug | Normal | ||
Description |
Follow up of #10438.
This ticket is required to close #10439. |
|||||
#10497 | Empty lines are removed in enter mode BR - continuation | confirmed | Bug | Normal | ||
Description |
This is continuation of http://dev.ckeditor.com/ticket/10146#comment:26. Seems ticket isn't fully fixed. Problem can be reproduced in every browser. To reproduce:
Result: As you can see two BR's were changed into single . NOTE: This is happening if you use one or two BR's. If you use three BR's then result is "br br nbsp". Assuming that last Br has to be changed into nbsp, should it be "br nbsp" when two BR's are used?
It seems ticket #10146 hasn’t been fully fixed thus this one should be closed and the former reopened. |
|||||
#10499 | [Webkit] Content gone while copying via context menu with keyboard. | confirmed | Bug | Normal | ||
Description |
In latest Chrome:
Since v3. |
|||||
#10502 | [FF]Right click on some elements causes Error: Permission denied to access property 'nodeType' | confirmed | Bug | Normal | ||
Description |
Error: Permission denied to access property 'nodeType' (line 25) |
|||||
#10511 | [Chrome] Shift+F10 opens editor and browser native context menu. | confirmed | Bug | Normal | ||
Description |
In latest Chrome 27 when you open editor menu with Shift+F10 browser menu also opens. If you try this outside of editor browser menu opens which means that Chrome is using Shift+F10 as shortcut for its native menu. To reproduce:
Result both menus are opened. When you have two menus opened press Esc and native one will close. This can be used as workaround but we should block this native menu somehow and make sure that we don’t block it for good as users can turn off editor menu (there is such config option) and use browser native one Problem can be reproduced from CKEditor 3.0 in both CKE 3.x and 4.x. |
|||||
#10527 | IE8 - Link not preserved on double-click in inline editor. | confirmed | Bug | Normal | ||
Description |
Results: Now there are two issues.
Message:
SCRIPT5007: Unable to get value of the property 'collapsed': object is null or undefined Problem can be reproduced in IE8 only in inline mode only from CKEditor 4.0 beta |
|||||
#10536 | API imporevements | confirmed | Bug | Normal | ||
Description |
I'd like to gather in this ticket some API improvements we could make. Of course most of them are not obvious, so they can be controversial. Also, in every case backward compatibility should be ensured.
|
|||||
#10555 | It is not possible to catch key event when deleting list | confirmed | Bug | Normal | ||
Description |
Result: key event isn't caught. Problem can be reproduced from CKEditor 4.0. This issue was reported on our support channel. |
|||||
#10557 | [IE8] Delete creates unordered list from ordered list | confirmed | Bug | Normal | ||
Description |
Result: Ordered list gets changed into unordered one. Problem can somewhat be reproduced from CKE 3.0 (orphan li elements were left) but this particular TC where one list is changed into another can be reproduced from CKEditor 3.4.2 in both CKE 3.x and 4.x. |
|||||
#10568 | Focus space configuration for magicline. | confirmed | New Feature | Normal | ||
Description |
A new configuration option should be introduced to restrict magicline focus spaces to some user-selected subset like: config.magicline_triggers = {hr:1,div:1}; Ticket extracted form the pull request. |
|||||
#10584 | IE8 throws error on delete | confirmed | Bug | Normal | ||
Description |
To reproduce:
Result JS error is thrown.
Message: Invalid argument Problem can be reproduced in both branches CKE 3.x and 4.x. I was able to reproduce it from CKEditor 3.3.
Bug History:
Possible reason: It has turned out in my tests that simple space between image and Apollo word is to blame. If I have removed this space there was no error. <h1><img alt="Saturn V carrying Apollo 11" class="right" src="assets/sample.jpg" /> Apollo 11</h1> In CKEditor 4.x this image is aligned to the right and it doesn't look like source of the problem but in older versions there is something extra happening which may prove what I have written. Copy sample code from CKE 4.x to 3.x - till version 3.3 When you removed image space was left and no error occurred. From CKE 3.3 removing image removes also this space. |
|||||
#10594 | Error when replacing element with <br> in HTML filter | confirmed | Bug | Normal | ||
Description |
|
|||||
#10595 | Wrong caret position after ENTER on list item with sub-list | confirmed | Bug | Normal | ||
Description |
<ul> <li>x <ul> <li>y</li> </ul> </li> </ul>
Current results: The new list item will be properly created, but the caret will be blinking right before "x". Expected results: The caret should be blinking in the new empty list item. |
|||||
#10597 | Event sample | confirmed | Task | Normal | ||
Description |
There should be a sample to show how events works and which event is fired in which case. I think that it should be a list of events and the editor and every time the event if fired a right event on the list should blink. |
|||||
#10606 | Inline form textarea using jQuery adapter | confirmed | New Feature | Normal | ||
Description |
It should be possible to replace textarea with inline editor using jQuery adapter. Now jQuery adapter check what is the type of element and if it is textarea it use replace otherwise it use inline so it is not possible do use inline on textartea. In my opinion the best solution is to create additional configuration option with will be parsed by the adapter i.e.: $( 'textarea' ).ckeditor( { creator: inline } ); should create editor using inline function. |
|||||
#10607 | Remove "indentlist" require from "list" | confirmed | Bug | Normal | ||
Description |
This is a followup for ticket:10599#comment:3. Ideally, the "list" plugin should not require "indentlist". The problem is that there is a execCommand( 'outdent' ) call in the list plugin code, which makes this require necessary. Additionally, we have removed some tests with (edbb565 @tests), because they were not compatible with this require. If we fix this, we should revert those tests back. |
|||||
#10616 | IE11 numbers lists from zero IN COMPATIBILITY MODE | confirmed | Bug | Normal | ||
Description |
Insert ordered list into editor or open page with list in IE11 compatibility mode. Numbers start from 0. |
|||||
#10617 | [IE]: BIDI button breaks toolbar when in compatibility mode | confirmed | Bug | Normal | ||
Description |
Result: some toolbar buttons get very long. This happens in IE7 by default and in other IEs when they are set in compatibility mode. |
|||||
#10627 | Removing form element removes whole paragraph | confirmed | Bug | Normal | ||
Description |
Results:
This problem can be reproduced in Safari 6 (Mac) and Chrome 28 from CKEditor 3.3. Before this version there was no tiny space before textfiled and removing worked ok. |
|||||
#10628 | Samples for basics makes no sense | confirmed | Task | Normal | ||
Description |
When you open samples in basics preset some (most) of them does not work properly (as described). In example:
This is only small list for basic preset and there could be similar problems for standard. I think that for a user who just want to check how basics editor works all of this things look like bugs. |
|||||
#10636 | Error thrown when in/outdenting inside of a list element (caret in a paragraph) | confirmed | Bug | Normal | ||
Description |
<ol> <li>x <p>y^</p> <ol> <li>z</li> </ol> </li> </ol>
|
|||||
#10641 | Find considers elements with display: none | confirmed | Bug | Normal | ||
Description |
Extracted from the question on SO. When searching for some text, editor also considers invisible elements. It looks strange when the dialog suggests something was found but nothing is highlighted. We could at least provide some configuration option to bypass this strange behavior or basically fix it. |
|||||
#10645 | Preview tab in DocProps plugin is missing utf-8 meta tag. | confirmed | Bug | Normal | ||
Description |
Preview tab in DocProps plugin is missing utf-8 meta tag. Because of this all non-latin characters are wrongly displayed. |
|||||
#10655 | TAB leaves the editable when cannot indent anything | confirmed | Bug | Normal | ||
Description |
If we handle tab in all other lists' elements, then it should be blocked in the first one and in all other places where tab does nothing. It cannot be so inconsistent, because this is confusing and irritating. Either tab leaves editable or it indents lists/blocks. Not both at the same time. This could be configurable by keystrokes - by default tab is added as a keystroke executing 'indent' command. If someone needs it for leaving editable, then he can reset this keystroke. Or the opposite - tab by default can be used for navigation, but it's behaviour can be changed by setting a keystroke. And yes, there's no good solution ;|. |
|||||
#10660 | Menu Button may be missing default mode | confirmed | Bug | Normal | ||
Description |
Problem: button gets disabled.
This is happening because plugin doesn't use I believe that either there should be default mode for menu button or this property should be made required. Problem can be reproduced in both CKEditor 4.x and 3.x |
|||||
#10679 | IE complains if "@" is used right after comment start | confirmed | Bug | Normal | ||
Description |
Problem was described on forum: http://ckeditor.com/forums/CKEditor/ckeditor-causes-comments-prefixed-by-in-subsequent-scripts-to-be-executed-as-code-in CKEditor uses conditional compilation to determine whether it is ran on IE. If CKEditor's source file is followed (even indirectly) by: //@ or: /*@ IE will parse this comment as a CC statement (although, I have no idea why :|) causing a very likely syntax error. Possible workarounds:
|
|||||
#10697 | CKBuilder: add a flag to mark parts of code needed in release version for tests | confirmed | New Feature | Normal | ||
Description |
We might need a special support for marking parts of code that are needed for testing and should be removed in release version. In short, they should be left in "test release" version. Sample code: https://github.com/cksource/ckeditor-dev/blob/t/9764/plugins/widget/plugin.js#L420-L428 // %REMOVE_START% // Expose for tests. , getNestedEditable: getNestedEditable, createEditableFilter: createEditableFilter // %REMOVE_END% }; |
|||||
#10712 | Editor.insertElement will insert passed element into every range in the selection | confirmed | Bug | Normal | ||
Description |
This is a FF's only issue. Currently (and this is a state from 3.0) Editor.insertElement will take passed element, clone it multiple times and insert into every range. This is a surprising behaviour, because we lose control over inserted element - we keep the reference only to one of them. Correct behaviour IMO would be to clean all ranges and insert element only to the first (or last?) of them. While working on this ticket it would be worth to review and correct insertHtml which does not clean other ranges when doing insertion. |
|||||
#10734 | icon strip generated in local builder includes all the icons | confirmed | Bug | Normal | ||
Description |
The icons.png generated by the local version of CKBuilder includes all the images instead of just the used icons like the online version does. Go to http://ckeditor.com/download and download the Basic package. Now use the included build-config.js in a local version of CKEditor and use the build.sh script Compare both icons.png files and you'll see that instead of 7Kb it's 17Kb and includes all the plugins. |
|||||
#10737 | UI languages sample should show some BiDi features | confirmed | Task | Normal | ||
Description |
Hey guys, The UI languages sample aims to show that the editor could be used in an RTL environment. While the basic goal of this sample is indeed reached (the UI does show up in Hebrew), it's worth showing some of the appropriate BiDi features CKEditor has to offer. When one changes the UI language in the sample, he might expect to try and write something using the chosen language. Currently, if the language is changed to (e.g.) Hebrew, the cursor stays on the left and the text is actually written as LTR. This could be solved either by changing the default writing direction on this sample accordingly, or at least by providing LTR/RTL and alignment buttons on the toolbar. P.S - "Try "arabic" or "hebrew" to check RTL support:" - Arabic and Hebrew should be capitalized (proper nouns). Just my two cents :) |
|||||
#10745 | [FF&Inline] Cells selection is lost when merging cells | confirmed | Bug | Normal | ||
Description |
To debug add
Note: |
|||||
#10748 | Prevent save event instead of submit | confirmed | Task | Normal | ||
Description |
Working on #10689 I realized that this part of code works magicly: // #8031 If textarea had required attribute and editor is empty fire 'required' event and if // it was cancelled, prevent submitting the form. if ( editor._.required && !element.getValue() && editor.fire( 'required' ) === false ) { // When user press save button event (evt) is undefined (see save plugin). // This method works because it throws error so originalSubmit won't be called. // Also because of hack there is try-cache in save plugin so error won't be shown. evt.data.preventDefault(); It should be fixed using save event instead of submit. |
|||||
#10749 | [Webkit/Blink] Loading URL with a # causes page to scroll | confirmed | Bug | Normal | ||
Description |
Open: http://ckeditor.com/demo# or samples/datafiltering.html# See that page is scrolled down. Reported on: http://ckeditor.com/forums/CKEditor/Scrolling-problem-when-using-empty-anchor-in-URL |
|||||
#10765 | Broken form submit with inline-textarea | confirmed | Bug | Normal | ||
Description |
Then what happens:
|
|||||
#10766 | [Webkit] Irresistible content when pasting into entirely selected editable | confirmed | Bug | Normal | ||
Description |
Note there's an unwanted paragraph aligned to the right at the end of editable: <p>Hello world!</p> <p style="text-align:right"> </p> Now remove the link from the original content and repeat all the steps. Use the following startup data: <p>Hello world!</p> <p style="text-align:right">Yay! I'm not so sticky anymore!</p> Result is just perfect: <p>Hello world!</p> |
|||||
#10767 | Webkit removes whole table | confirmed | Bug | Normal | ||
Description |
To reproduce:
Result: Whole table gets deleted. Problem can be reproduced from CKEditor 4.0.1 in Chrome and Safari only. |
|||||
#10785 | Editor content's stylesheet is removed from document after another editor's editable is reattached | confirmed | Bug | Normal | ||
Description |
Or:
Problem is somewhere here: https://github.com/ckeditor/ckeditor-dev/blob/master/core/editable.js#L681-L693 It seems that framed editor (wysiwygarea or sourcearea) does not increase stylesheet_ref, but it decreases it. Or that order of detach and new editable's setup is incorrect. |
|||||
#10788 | [Safari] Pressing ESC when in opened color platte/combo moves focus to editable. | confirmed | Bug | Normal | ||
Description |
Expected: focus goes back to color button in the toolbar so it's possible to select a different toolbar (TAB) or a different button (arrows). This is the behaviour of Chrome. Actual: focus goes to editable. Need to use the keystroke again to focus it. |
|||||
#10794 | [IE10 Quirks Mode]Toolbar for RTL languages looks bad | confirmed | Bug | Normal | ||
Description |
Result: toolbar has wrong styles (see attachment). 4.2.1 is first version with IE 10 QM support. |
|||||
#10795 | [IE10 Quirks Mode]Not code formating in source mode | confirmed | Bug | Normal | ||
Description |
Result: no code formatting (see attachment). 4.2.1 is first version with IE 10 QM support. |
|||||
#10796 | [IE] Text selection to end of caption causes JS error | confirmed | Bug | Normal | ||
Description |
since: 4.0 until master
Expected result:
Current result:
js error:
additional info: |
|||||
#10816 | Mention in the ACF rules documentation that div[*] does not allow styles and classes | confirmed | Task | Normal | ||
Description |
http://docs.ckeditor.com/#!/guide/dev_allowed_content_rules |
|||||
#10817 | Implement a time widget | confirmed | New Feature | Normal | ||
Description |
During works on #9764 we implemented simple time widget. It's not ready to be included in official release yet, so we're extracting it to this ticket. |
|||||
#10818 | Implement a captioned quote widget | confirmed | New Feature | Normal | ||
Description |
During works on #9764 we implemented simple captioned quote widget. It's not ready to be included in official release yet, so we're extracting it to this ticket. |
|||||
#10819 | [iOS] Hitting return will cause all text below cursor to delete | new | Bug | Normal | ||
Description |
Hitting return enough times in a CKEditor field will cause all text below cursor to delete, and will continue to delete text below cursor when pressing return. This is also reproducible using the demo you have online following these steps:
|
|||||
#10820 | [iOS] Calling insertText switches CKEditor into "Read Only" mode | new | Bug | Normal | ||
Description |
After using the CKEditor function
Text can be read and selected but no typing is allowed. This can happen after |
|||||
#10824 | Languages plugin - can't nest language markup | confirmed | Bug | Normal | ||
Description |
Issue is reproducable with Chrome/31.0.1627.0, Firefox 23.0, MSIE10 We're unable to nest multiple spans using languages plugin
Expected result:
Current result:
additional info:
|
|||||
#10834 | Image2: The caption should not allow <p> and <br> (disable ENTER) | confirmed | Bug | Normal | ||
Description |
Referring to: http://dev.ckeditor.com/ticket/10659#comment:8 |
|||||
#10872 | Unexpected bold when breaking a line after Removing Format on collapsed selection | confirmed | Bug | Normal | ||
Description |
I'm able to reproduce it since 3.6.6.1 in Chrome, Firefox and Safari (Mac). |
|||||
#10885 | Broken path to an anchor when CKEditor is located in a directory with a space | confirmed | Bug | Normal | ||
Description |
Kudos to Anna and Olek.
When there is an anchor in the content, CKEditor requests the following image: Note the weird path in the URL: /ckcke_contents_ltr0421/ Confirmed in 4.2.1 and 4.3 Beta. Did not check earlier versions. |
|||||
#10909 | Link to CHANGES.md in http://docs.ckeditor.com/#!/guide/dev_api_changes | confirmed | Task | Normal | ||
#10928 | Pulled quotes widget | confirmed | New Feature | Normal | ||
#10932 | Allow all (reasonable) elements as widgets' nested editables | confirmed | New Feature | Normal | ||
Description |
Currently nested editables can be only initialized on elements included in the CKEDITO.dtd.$editable hash which includes those which should be "safe". However, it would be nice to increase number of allowed elements, especially with inline ones. The workaround for now, if someone wants to try other elements, is to extend CKEDITOR.dtd.$editable object has early as possible. |
|||||
#10942 | Image2: Introduce a proper visual indicator of selected resizable image | confirmed | Bug | Normal | ||
Description |
Following the discussion https://dev.ckeditor.com/ticket/10835#comment:5 suggesting that we may may want to improve UX by putting some kind of an outline to the image, I open this ticket for research purposes and general brainstorming. |
|||||
#10946 | [FF] Cursor stuck in widget | confirmed | Bug | Normal | ||
Description |
You can not move cursor before widget. This bug could be reproduce with any inline widget. |
|||||
#10952 | Various issues when choosing inactive option in context menu | confirmed | Bug | Normal | ||
Description |
General problem - nothing should happen IMO when choosing disabled option. Browser specific:
|
|||||
#10970 | [IE] Not able to remove row in inline | confirmed | Bug | Normal | ||
Description |
SCRIPT16389: Incorrect function. inlinebycode.html, line 46 character 5 |
|||||
#10974 | Add support for inline nested editables | confirmed | New Feature | Normal | ||
Description |
Currently widget may have only block editables. Support for inline ones would be interesting. Note: there's a workaround if one wants to initialize inline editable - CKEDITOR.dtd.$editable has to be extended with name of that element. Includes: #10973 |
|||||
#10988 | [FF] Widgets: Caret goes outside of contenteditable on home/end keys | confirmed | Bug | Normal | ||
Description |
since: 4.3 (didn't check earlier) until major Firefox exclusive: pressing home/end key can put caret outside widget.
Expected result:
Current result:
additional info:
|
|||||
#10989 | [Webkit/Blink] Widgets and content after them disappear when pargraphs are merged using backspace | confirmed | Bug | Normal | ||
Description |
|
|||||
#10993 | Delete columns when rows are splitted | confirmed | Bug | Normal | ||
Description |
Column is being removed but console error occurs.
|
|||||
#11010 | Fix memory leaks caused by CKEDITOR.filter.instances storing nested editables' filters forever | confirmed | Bug | Normal | ||
Description |
Every nested editable which has allowed content rules defined has its filter instance. These instances are stored in nested editable instances, which most likely are correctly garbage collected. But filter instances are also stored in CKEDITOR.filter.instances and they are kept there forever. We could have filter.destroy() method removing instance from the instances hash and doing other cleanup if necessary. Then we could destroy filters when destroying widgets. |
|||||
#11022 | [Webkit&Blink] Nested editable does not get focus in certain situations | confirmed | Bug | Normal | ||
Description |
Expected:
Actual:
|
|||||
#11023 | Blocks and objects names should be taken from DTD in core/style.js | confirmed | Task | Normal | ||
Description |
There are list of blocks and objects names defined in core/style.js. CKEDITOR.dtd should be reused there. |
|||||
#11034 | IframeDialog has very small content in CKEditor 4 continued. | confirmed | Bug | Normal | ||
Description |
This is the continuation of ticket #10610. There are two methods of adding iframedialog to editor:
To summarize: adding iframe as contents and setting height to 100% will result in small iframe.
Why this is happening: Div surrounding iframe has height auto and all elements inside it (when you define % for iframe) are % so the wrapping div will never get dialog height in this case. Ticket #10610 didn't involve fixing dialog plugin but iframedialog plugin. As discussed with @fredck, fix from #10610 did its job in 50%. It should be checked why doesn't it work for second case and if something can be done about it. I’m attaching sample plugin for testing. |
|||||
#11035 | Deleting one of two subsequent spaces should leave normal space | confirmed | Bug | Normal | ||
Description |
Extracted from: #11024. Related to: #9929. There's WebKit&Blink bug:
If it won't be fixed in browsers we should take care of this when we'll be implementing custom delete/backspace support.
So |
|||||
#11038 | [IE] Click on MathJax/Image widget wrapped with link causes location change | confirmed | Bug | Normal | ||
Description |
Reproduced on IE9, but most likely concerns also other IEs. |
|||||
#11039 | [Webkit/Blink] Link created in nested editable is not being selected | confirmed | Bug | Normal | ||
Description |
|
|||||
#11055 | Image2 is not focused after pasting | confirmed | Bug | Normal | ||
Description |
Seleciton is placed inside nested editable. Instead, widget should be focused. |
|||||
#11059 | Widget reinitialization procedure should be redesigned | confirmed | Task | Normal | ||
Description |
Widget reinit/recreate procedure should be rethought. There's no way currently to reinitialize in a lightweight manner widget to which we lost references. It has to pass through entire process of setting attributes, classes, data, etc. What we need is only - rebinding listeners and initial widget#data. |
|||||
#11060 | [IE11+] Selection is not placed right above the cursor when clicking below body | confirmed | Bug | Normal | ||
Description |
Follow up for #10906. When clicking below body, selection should be placed right above the cursor. Now it's placed in the last position or at the beginning. |
|||||
#11062 | [IE9,10] Scrolling while dragging freezes browser window | confirmed | Bug | Normal | ||
Description |
Mouse pointer change and browser does not response. |
|||||
#11094 | [IE8] Editor loses focus when moving focus from nested editable to widget | confirmed | Bug | Normal | ||
Description |
Toolbar disappears. |
|||||
#11095 | [IE8] Copying single widget does not work when using toolbar button | confirmed | Bug | Normal | ||
Description |
Follow up for #11082.
There's a big chance that selection was lost and widget wasn't copied. |
|||||
#11111 | Range.enlarge test failing in Safari 7.0 and Chrome 37 | confirmed | Bug | Normal | ||
Description |
Test /dt/core/dom/range/enlarge.html#test_enlarge_element12 fails in latest Safari 7.0 that comes with OSX 10.9. It's also reproducible in Chrome 37 (see #12221). The root of the problem is the following condition: https://github.com/ckeditor/ckeditor-dev/blob/major/core/dom/range.js#L1039
In this case That particular test fails since CKeditor 4.0 in Safari 7 which makes it clear that we're facing Safari regression. Related bug in Chrome: #12221. |
|||||
#11112 | [Chrome] Copy + apply list + change list type -> break inline wdget | confirmed | Bug | Normal | ||
Description |
Result: <span tabindex="-1" contenteditable="false" data-cke-widget-wrapper="1" data-cke-filter="off" class="cke_widget_wrapper cke_widget_inline cke_widget_selected" data-cke-display-name="math" data-cke-widget-id="6"> <span class="math-tex cke_widget_element" data-cke-survive="1" data-cke-widget-data="{"math":"\\(k\\)"}" data-cke-widget-keep-attr="0" data-widget="mathjax" style="display: inline-block;"> <iframe style="border: 0px; width: 12px; height: 19px; display: inline; vertical-align: middle;" scrolling="no" frameborder="0" allowtransparency="true" src="javascript:void(0)"></iframe> <iframe style="border: 0px; width: 12px; height: 19px; display: inline; vertical-align: middle;" scrolling="no" frameborder="0" allowtransparency="true" src="javascript:void(0)"></iframe> </span> <img src="data:image/gif;base64,R0lGODlhAQABAPABAP///wAAACH5BAEKAAAALAAAAAABAAEAAAICRAEAOw%3D%3D" class="cke_reset cke_widget_mask"> <span class="cke_reset cke_widget_drag_handler_container" style="background-color: rgba(220, 220, 220, 0.498039); background-image: url(http://presets.ckeditor.dev/4.3/full-all/ckeditor/plugins/widget/images/handle.png); top: -15px; left: 0px; background-position: initial initial; background-repeat: initial initial;"> <img class="cke_reset cke_widget_drag_handler" data-cke-widget-drag-handler="1" src="data:image/gif;base64,R0lGODlhAQABAPABAP///wAAACH5BAEKAAAALAAAAAABAAEAAAICRAEAOw%3D%3D" width="15" title="Click and drag to move" height="15" draggable="true"> </span> <img src="data:image/gif;base64,R0lGODlhAQABAPABAP///wAAACH5BAEKAAAALAAAAAABAAEAAAICRAEAOw%3D%3D" class="cke_reset cke_widget_mask"> <span class="cke_reset cke_widget_drag_handler_container" style="background-color: rgba(220, 220, 220, 0.498039); background-image: url(http://presets.ckeditor.dev/4.3/full-all/ckeditor/plugins/widget/images/handle.png); top: -15px; left: 0px; background-position: initial initial; background-repeat: initial initial;"> <img class="cke_reset cke_widget_drag_handler" data-cke-widget-drag-handler="1" src="data:image/gif;base64,R0lGODlhAQABAPABAP///wAAACH5BAEKAAAALAAAAAABAAEAAAICRAEAOw%3D%3D" width="15" title="Click and drag to move" height="15" draggable="true"> </span> </span> The same happens with placehoder plugin. |
|||||
#11113 | [IE] Elementspath does not display valid path for first click after selectall | confirmed | Bug | Normal | ||
Description |
since: 4.0 until major This issue also appears in widgets content editables.
Expected result:
Current result:
Second case:
... and now elementspath shows
additional info:
|