Opened 4 years ago

Last modified 3 years ago

#13841 confirmed Bug

Elements path selection and deletion work unexpectedly

Reported by: Lyuba Owned by:
Priority: Normal Milestone:
Component: General Version: 3.0
Keywords: Blink firefox Cc:

Description (last modified by Jakub Ś)

Steps to reproduce

  1. Start with the following source:
<div style="background:red;">some red text</div>
<div style="background:yellow;">some yellow text</div>
<div style="background:blue;">some blue text</div>
  1. In WYSIWYG, place cursor on the yellow text
  2. On the elements path bar at the bottom, click "div" to highlight the yellow div
  3. Press delete

Expected result

I would expect the yellow div to be removed, and the result to be:

<div style="background:red;">some red text</div>
<div style="background:blue;">some blue text</div>

Actual result

Instead, the context on yellow is removed, and the content of the following blue div is replaced inside instead (along with some line-height added for some reason):

<div style="background:red;">some red text</div>
<div style="background:yellow;"><span style="background-color:blue; line-height:1.6">some blue text</span></div>

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

Using Chrome on Linux.

Reproducible on http://ckeditor.com/demo#full

I feel like this was working differently (and used to just remove the div) in previous releases.


Please see comment:1 for Firefox result

Change History (1)

comment:1 Changed 3 years ago by Jakub Ś

Description: modified (diff)
Keywords: Blink firefox added
Status: newconfirmed
Version: 4.5.43.0

From What I have checked problem can be reproduced from CKEditor 3.0 in Blink and Webkit browsers.


Additionally in Firefox the result is also not as expected. When checking it with Firebug it looks like:

<div style="background:red;">some red text</div>
<br>
<div style="background:blue;">some blue text</div>

After switching to source:

<div style="background:red;">some red text</div>
<p>&nbsp;
<div style="background:blue;">some blue text</div>
</p>
}}}}
Which after switching to wysiwyg looks like:
{{{
<div style="background:red;">some red text</div>
<p>&nbsp;</p>
<div style="background:blue;">some blue text</div>
<p>&nbsp;</p>
}}}

Note: See TracTickets for help on using tickets.
© 2003 – 2019 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy