-
Notifications
You must be signed in to change notification settings - Fork 182
Add support for Copy/Cut/Paste keys #2506
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
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. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some hints on the API errors:
- The
@since
tag does not use the SWT bundle version but an Eclipse release version. It should be something like3.132
together with a minor version increment of the SWT bundle and fragments to3.132
because 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.