Opened 7 years ago

Closed 7 years ago

#9577 closed Bug (fixed)

Chrome+Inline: Context menu paste goes to the start of the text

Reported by: Frederico Caldeira Knabben Owned by: Piotrek Koszuliński
Priority: Normal Milestone: CKEditor 4.0
Component: General Version: 4.0
Keywords: WebKit Cc:

Description (last modified by Frederico Caldeira Knabben)

With Chrome and inline editing:

  1. Be sure you have some plain text in the clipboard.
  2. Enable an inline editor.
  3. Right click on the middle of the text, showing the context menu.
  4. Select "Paste".
  5. Fill the paste dialog with clipboard data and hit OK.

The pasted text will go at the start of the contents.

It doesn't happen with framed editing.

Change History (8)

comment:1 Changed 7 years ago by Piotrek Koszuliński

Status: newpending

Does clicking "paste" in context menu cause native paste for you? For me paste dialog is opened and when I paste clipboard content there and click "OK" content is inserted in the right place.

comment:2 Changed 7 years ago by Garry Yao

WFM as well, would you provide a concrete TC?

comment:3 Changed 7 years ago by Frederico Caldeira Knabben

Description: modified (diff)
Keywords: WebKit added
Status: pendingconfirmed

Well... that IS a concrete TC.

It happens constantly for me on both Chrome and Safari. I'm on a Mac, so maybe this is WebKit Mac only?

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

OK, I can confirm it on Chrome and Safari, but only on Mac. It doesn't happen constantly and the paste must be triggered from context menu. I also see that before opening context selection changes - entire right-clicked element is selected. Also - every type of content pasted/written in paste dialog will do the job.

Last edited 7 years ago by Piotrek Koszuliński (previous) (diff)

comment:5 Changed 7 years ago by Piotrek Koszuliński

Owner: set to Piotrek Koszuliński
Status: confirmedreview

Pushed t/9577.

sel variable was overwritten by pastebin selection. I'm not sure why, but selection that was unlocked after closing paste dialog wasn't aware of changes in text nodes made by the creation of bookmarks. Anyway - patch seems to fix this issue.

Last edited 7 years ago by Piotrek Koszuliński (previous) (diff)

comment:6 Changed 7 years ago by Garry Yao

Status: reviewreview_passed

comment:8 Changed 7 years ago by Piotrek Koszuliński

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