Add cut/copy/paste to the context menu

[ficristo]

Switch cut/copy/paste to use document.execCommand and add them to the context menu.
Before, on right click the underline word were selected but it was counterintuitive for the context menu usage so I removed it.
We need adobe/brackets-shell#567 to make the paste command work.

[marcelgerber]

First things first: We don't need the ignoreCommand handler any more :)

What's the behaviour on Linux, as execCommand("copy/cat/paste") is only supported in Chrome 42+?

[ficristo]

Oddly seemed to work for me on Ubuntu 16.04 without any other changes.
Would be nice to have some confirmations.
(I didn't try the installer and I don't know if would change anything)

[marcelgerber]

Yeah, confirmed on Linux now. Works without any shell changes.
Works fine on Windows for me, too.

I just noticed one issue:

1. Position your cursor on the end of a line that consists of only spaces for indentation.
2. Right-click anywhere in that line

Result: The whitespace is selected, which makes pasting a whole lot harder, as it will delete the whitespace.

[ficristo]

The whitespace is selected

I cannot reproduce...

[marcelgerber]

Strange. I can't reproduce any more either.

[zaggino]

Tested. "Cut" and "Copy" work fine, "Paste" doesn't do anything for me. Windows shell.

[ficristo]

"Paste" doesn't do

Have you tested with the shell PR? Otherwise is expected.

[zaggino]

Ah, my bad. Gonna test and merge if it works.

[ficristo]

Can you wait to merge? I need to cleanup a bit this first.

[zaggino]

👍 ping me then

[marcelgerber]

@ficristo Would love to merge this after your cleanup work is done!

[ficristo]

I've removed the unused ignoreCommand.
Now it's ready for another pass.

[marcelgerber]

All good.
Thank you!

[GuyRogers]

Thanks so much for the Cut/Copy/Paste. It was one of the few things that was driving me mad in Brackets 👍