Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 338645 - [KeyBindings] Unable to copy & paste from Text widget with CTRL+C and CTRL+V if workbench is used
Summary: [KeyBindings] Unable to copy & paste from Text widget with CTRL+C and CTRL+V ...
Status: RESOLVED FIXED
Alias: None
Product: RAP
Classification: RT
Component: RWT (show other bugs)
Version: 1.4   Edit
Hardware: All All
: P1 normal (vote)
Target Milestone: 1.4 M6   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-03-02 05:37 EST by Bassspieler CLA
Modified: 2011-03-03 07:35 EST (History)
2 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Bassspieler CLA 2011-03-02 05:37:21 EST 
Build Identifier: RAP 1.4 M5

In M4 Ctrl+C, Ctrl+V works. In M5 not. You can still use the context menu to copy and paste, but the keybinding seems to be missing.

Reproducible: Always

Steps to Reproduce:
1. Try to copy text from a text widget by using Ctrl+C
Comment 1 Ivan Furnadjiev CLA 2011-03-02 06:27:04 EST 
In 1.4 M5 we implemented a keybinding support in the workbench and enabled the default workbench keybindings ( Bug 282449 ). CTRL+C and CTRL+V are default workbench keybindings (in org.eclipse.rap.ui\plugin.xml they are defined as M1+C and M1+V), which prevents the browser CTRL+C and CTRL+V keybindings to work.
Comment 2 Ralf Sternberg CLA 2011-03-02 07:22:40 EST 
Setting this to P1, because it's a regression. Ctrl-C and Ctrl-V should work. Can't we simply exclude these key sequences from the default list?
Comment 3 Ivan Furnadjiev CLA 2011-03-02 07:45:44 EST 
Excluding CTRL+C and CTRL+V keybindings from the default list is an easy fix, but this will not prevent the problem to reappear if someone define its own CTRL+C, Ctrl-V keybinding.
Comment 4 Ralf Sternberg CLA 2011-03-02 15:10:22 EST 
(In reply to comment #3)
> Excluding CTRL+C and CTRL+V keybindings from the default list is an easy fix,
> but this will not prevent the problem to reappear if someone define its own
> CTRL+C, Ctrl-V keybinding.

As we don't have clipboard support, the Ctrl-C and Ctrl-V key bindings are not effective in RAP anyway. Therefore I would not hesitate to disable them, even if this is only a partial fix. At least it will solve the problem for most users.

But I wonder, if bindings for these key sequences are added explicitly, does the behavior really differ between RAP and RCP?
Comment 5 Ivan Furnadjiev CLA 2011-03-03 07:35:02 EST 
Ctrl-C (copy), Ctrl-V (paste) and Ctrl-X (cut) are removed from the default workbench keybinding list. Crtl-A (select all) and DEL (delete) keybindings are removed from the list too as they are conflicting with the Text widget functionality.