Opened 7 years ago

Closed 4 years ago

#6033 closed Bug (fixed)

[IE] Pasting over a readonly block merge the siblings and the pasted content

Reported by: Sa'ar Zac Elias Owned by:
Priority: Normal Milestone:
Component: Core : Pasting Version: 3.4
Keywords: IE Cc:

Description

  • Load the following HTML:
    <p>
    	Para 1</p>
    <p contenteditable="false">
    	Para 2</p>
    <p>
    	Para 3</p>
    
  • Right click "Para 2".
  • Paste some content.

Notice that para 1, the pasted content and para 3 are merged into one paragraph.

Attachments (2)

krst_13.30.31.png (33.8 KB) - added by Krzysztof Studnik 7 years ago.
Confirmed on IE8
krst_13.36.15.png (30.0 KB) - added by Krzysztof Studnik 7 years ago.
Differences between browsers

Download all attachments as: .zip

Change History (6)

Changed 7 years ago by Krzysztof Studnik

Attachment: krst_13.30.31.png added

Confirmed on IE8

Changed 7 years ago by Krzysztof Studnik

Attachment: krst_13.36.15.png added

Differences between browsers

comment:1 Changed 7 years ago by Krzysztof Studnik

Component: GeneralCore : Pasting
Status: newconfirmed
Version: 3.4

In my opinion if contenteditable="false" is set, only Copy should be allowed, or everything should be blocked.

comment:2 in reply to:  1 Changed 7 years ago by Frederico Caldeira Knabben

Replying to krst:

In my opinion if contenteditable="false" is set, only Copy should be allowed, or everything should be blocked.

That's true if you're inside read-only content, but not if you select the entire read-only element as a whole, which is what I think is happening in this tc.

A read-only element works much like an image, so it can be entirely deleted/replaced if the selection fully includes it, outside its boundaries.

comment:3 Changed 7 years ago by Frederico Caldeira Knabben

Note that the following represents the selection in IE, when right clicking with the above tc:

<p>
	Para 1[</p>
<p contenteditable="false">
	Para 2</p>]
<p>
	Para 3</p>

So, theoretically, joining paragraphs is correct... but of course, this is not the way users see it.

comment:4 Changed 4 years ago by Piotrek Koszuliński

Resolution: fixed
Status: confirmedclosed

Fixed in 4.3 if fake selection is used (as in widgets).

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