Skip to content
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

Text field keyboard shortcuts broken on Linux #210

Closed
nyanpasu64 opened this issue Jul 19, 2021 · 2 comments
Closed

Text field keyboard shortcuts broken on Linux #210

nyanpasu64 opened this issue Jul 19, 2021 · 2 comments
Labels
bug Something isn't working samples

Comments

@nyanpasu64
Copy link

Running RmlUi master 747d73c built from source, on Linux KWin X11. In the demo app's "Forms" tab, I cannot use keyboard shortcuts like Ctrl-C or Ctrl-V in text areas; it instead types "c" or "v". Copying and pasting works on Windows.

Operating System: Arch Linux
KDE Plasma Version: 5.22.3
KDE Frameworks Version: 5.84.0
Qt Version: 5.15.2
Kernel Version: 5.12.15-zen1-1-zen (64-bit)
Graphics Platform: X11
Processors: 12 × AMD Ryzen 5 5600X 6-Core Processor
Memory: 15.6 GiB of RAM
Graphics Processor: NVIDIA GeForce GT 730/PCIe/SSE2

mikke89 added a commit that referenced this issue Jul 19, 2021
… Fixes shortcut keys (eg. cut/copy/select all), see #210.
@mikke89
Copy link
Owner

mikke89 commented Jul 19, 2021

We haven't implemented system clipboard in the Linux and macOS shells, now they only copy and paste internally in the application. This feature should be a good exercise for someone familiar with these platforms.

With that said, filling of the text (c/v) should not occur when typing a shortcut key. Thanks for letting me know, that one should be fixed now.

@mikke89 mikke89 added enhancement New feature or request good first issue Good for newcomers samples help wanted Extra attention is needed bug Something isn't working and removed enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed labels Jul 19, 2021
@mikke89
Copy link
Owner

mikke89 commented Jul 19, 2021

I think we should open a new issue for the system clipboard feature if this is desired. The main issue here is considered the unintended c/v text pasting and should now be fixed.

@mikke89 mikke89 closed this as completed Jul 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working samples
Projects
None yet
Development

No branches or pull requests

2 participants