Read-only regions, copy context menu action should be available

[Lynne Kues]

3.4 Beta

Run the code example. Select the read-only region. Select the text in the region. Invoke context menu. Copy action is disabled.

[Tobiasz Cudnik]

This can't be reproduced in FF, but in Chrome and IE, the context menu's "Copy" is disabled (although CTRL+C works as expected on not-editable selection).

[Sa'ar Zac Elias]

Manually checking if the range is valid.

[Frederico Caldeira Knabben]

Note that, as commented in the code, we're using the 'Cut' command to check the 'Copy' state. It's supposed to be related to a browser bug.

I've tracked the original ticket for that thing. It's #984.

So far, that change was ok, because it was valid to consider that the Copy state will be the same as the Cut state in an editable environment. But in fact, things are different with read-only regions.

I've simply changed the check to the use the "Copy" command, and things started working well. For performance, that would be the right solution for it.

But #984 states that this may cause bugs in some browser: IE and Safari. So, we must check whether this is true.

Finally, we should isolate the range check fix for the browsers that really need it, if any, using "Copy" for all browsers that support it properly.

[Sa'ar Zac Elias]

I didn't see any problem using stateFromNamedCommand( 'Copy' ).

[Frederico Caldeira Knabben]

Please have this one committed into the 3.4.x branch.

[Sa'ar Zac Elias]

Fixed with [5734].