Pasting after 'cut'-ing text in i-text pastes previous data from system clipboard

[Koosmann]

On the kitchen sink demo, cmd+x and a subsequent cmd+v always pastes text cut/copied previously to the system clipboard.

This happens on osx 10.11.3 on chrome, safari, and ff.

[asturur]

i cannot replicate that i,m on windows.
Do you see some strange message in console?
Tomorrow i can try on a mac also.

[onassar]

I can confirm.
Cmd+x does does cut the text from the text field, but does not copy it to the machine's clipboard.

[asturur]

on windows it copies fine, but does not carry style if i use "cut" command.
Pasting is fine.

Please can you check console for strange messages?

[onassar]

Nothing in the console for me

[ozooner]

Tested on Windows(8), Linux & Mac. Can confirm that cut does not put the text to clipboard on any of the platforms. Tested at http://fabricjs.com/test/misc/itext.html

[asturur]

guys can you check if this demo page i made with latest changes fixes this issue?
To me it looks yes, but i would like your opinion too.

http://www.deltalink.it/andreab/fabric/iphone.html

[ozooner]

Works perfect on Linux!

[onassar]

@asturur I'm pretty far behind from HEAD; is there anyway you can point me to the exact parts of 56c477f that are relevant to fixing the cut + paste bug?

[asturur]

i ll try!

[onassar]

Thanks @asturur. Just comment here if you're able to and I'll test :)

[asturur]

ok i checked the commit, you have to insert it all.
The point is to handle CUT we now use the cut event and not the CTR+X keypress.
At least the commit you underlined is all necessary.

[asturur]

You should make an effort to switch to latest anyway, figure out what blocks you and see if is a stopper or not. I do not think we broke any api from 1.5 to now.

[onassar]

Okay thanks I merged that entire commit and it's working as expected. Thanks @asturur

[onassar]

@asturur I just realized this introduced a bug
Specifically, Command+a for selecting text is no longer working. Can you confirm this is happening on your side? When I remove the code, it works again.

[asturur]

please confirm that you can repro in kitchensink

[onassar]

I can't Does kitchen sink have your changes?

On Wed, Feb 1, 2017 at 2:14 PM Andrea Bogazzi ***@***.***> wrote: please confirm that you can repro in kitchensink — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#2773 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAlaStMtzv8wQleAscMSYiA_hzVUhlE6ks5rYNmngaJpZM4HUkdA> .

-- — Oliver Nassar

[asturur]

its on latest while you are on older and patch right?

[onassar]

You are correct. I found the fix and applied it. Thanks @asturur!