Add support for Copy/Cut/Paste keys#2506
Add support for Copy/Cut/Paste keys#2506yaslama wants to merge 3 commits intoeclipse-platform:masterfrom
Conversation
Test Results104 files - 14 104 suites - 14 6s ⏱️ - 10m 17s Results for commit 1b9667c. ± Comparison against base commit a479377. This pull request removes 4356 tests.♻️ This comment has been updated with latest results. |
HeikoKlare
left a comment
There was a problem hiding this comment.
Some hints on the API errors:
- The
@sincetag does not use the SWT bundle version but an Eclipse release version. It should be something like3.132together with a minor version increment of the SWT bundle and fragments to3.132because of the API additions. - The change is effectively a breaking API change, which is why API tooling complains that a major version increment is required (see "Add API Field" at https://github.com/eclipse-platform/eclipse.platform/blob/master/docs/Evolving-Java-based-APIs-2.md#evolving-api-classes). Since adding such a constant usually does not break any compatibility, it is often accepted even though theoretically breaking API compatibility. In such a case, you need to add an API filter for it. The IDE should show you a quick fix for that at the introduced constants if you have set up your workspace with an Oomph setup as described in the contribution information.
|
Thanks @HeikoKlare. I tried to follow your instructions. Can you please check again? The only thing that wasn't clear to me was "together with a minor version increment of the SWT bundle". Thanks! |
Currently the SWT bundle ( Also note that all my comments are only related to producing a compiling and tooling-compatible state. I have not evaluated the actual change so far, which I am also not fully able to as I am not on Linux. So someone else should have a look on that as well. Another questions is whether this could/should also be extended to the Windows and MacOS implementations. |
These keys are present in some old unix keyboards, but more importantly, their keycodes can be mapped to physical keys in modern programmable keyboards. Using them in Linux is a way to be able to have the same keys for copy/pasting in GUI apps and in terminal apps instead of switching between ctrl-c/ctrl-v and ctrl-shift-c/ctrl-shift-v.
Thanks for the explanation. I just force-pushed a commit with these changes. I hope that it's ok now |
From what I understand, these keys are not supported in Windows/MacOS, but I am no 100% sure as I use only Linux |
|
@HeikoKlare Do you think that the PR is in a compiling and tooling-compatible state? |
|
I think the Javadoc should state prominently that those events are currently only supported on GTK. |
|
@mickaelistria I just added it |
These keys are present in some old unix keyboards, but more importantly, their keycodes can be mapped to physical keys in modern programmable keyboards.
Using them in Linux is a way to be able to have the same keys for copy/pasting in GUI apps and in terminal apps instead of switching between ctrl-c/ctrl-v and ctrl-shift-c/ctrl-shift-v.