Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 338645

Summary: [KeyBindings] Unable to copy & paste from Text widget with CTRL+C and CTRL+V if workbench is used
Product: [RT] RAP Reporter: Bassspieler <m.kempe>
Component: RWTAssignee: Project Inbox <rap-inbox>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P1 CC: m.kempe, rsternberg
Version: 1.4   
Target Milestone: 1.4 M6   
Hardware: All   
OS: All   
Whiteboard:

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.