Opened 7 years ago

Last modified 5 years ago

#8667 confirmed Bug

[Webkit] Enter works incorrectly into PRE blocks

Reported by: Karen Ananiev Owned by:
Priority: Normal Milestone:
Component: UI : Enter Key Version: 3.5.1
Keywords: Webkit Cc:

Description

  • Open editor and clear all content
  • Apply Formatted paragraph format
  • Type some chars and hit Shift+Enter or Enter


Outcome: cursor jumps to the start of the current line and new line is added before this one.

Change History (5)

comment:1 Changed 7 years ago by Karen Ananiev

It seems that issue related with filling char (selection plugin) #1272

comment:2 Changed 7 years ago by Jakub Ś

Keywords: Webkit added
Status: newconfirmed
Version: 3.6.23.5.1

This got first broken in CKEditor 3.5.1 rev [6321]. When applying above TC you got the below:

<p>
	test test</p>
<p>
	test</p>
<pre>
</pre>
<p>
	&nbsp;</p>

After rev [6919] (CKE 3.6.1) the result changed to:

<pre>


</pre>
<pre>
test</pre>

The current result described in ticket has been reproducible from CKEditor 3.6.1 rev [6992] and looks like this:

<pre>

test</pre>
Last edited 7 years ago by Jakub Ś (previous) (diff)

comment:3 Changed 7 years ago by Karen Ananiev

The one more use case:

  1. Type any text and select it with ctrl-shift-left arrow
  2. Apply any font name
  3. Unselect text with right arrow key
  4. Hit Enter and type text, select it with ctrl-shift-left arrow and apply another font name


Note that range is collapsed, cursor jumped to start of the paragraph and font name is not applied.

I noticed that setting new text to filling char node (fillingChar.setText()) collapses the range.

comment:4 Changed 7 years ago by Jakub Ś

The one more use case:

  1. Type any text and select it with ctrl-shift-left arrow
  2. Apply any font name
  3. Unselect text with right arrow key
  4. Hit Enter and type text, select it with ctrl-shift-left arrow and apply another font name

@karena your last comment refers to bug #8617 and not this one.

Last edited 7 years ago by Jakub Ś (previous) (diff)

comment:5 Changed 6 years ago by Jakub Ś

#9176, #8166, #10477 are two more ticket where this repository [6919] is mentioned.

Last edited 5 years ago by Jakub Ś (previous) (diff)
Note: See TracTickets for help on using tickets.
© 2003 – 2017 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy