Bug 497338 - Bind "Copy" and "Paste" keys on keyboards with dedicated keys for these to the copy and paste actions
Summary: Bind "Copy" and "Paste" keys on keyboards with dedicated keys for these to th...
Status: CONFIRMED
Alias: None
Product: frameworks-kconfig
Classification: Frameworks and Libraries
Component: general (show other bugs)
Version: 6.7.0
Platform: unspecified Linux
: NOR wishlist
Target Milestone: ---
Assignee: Matthew Dawson
URL:
Keywords: junior-jobs
Depends on:
Blocks:
 
Reported: 2024-12-11 18:56 UTC by ostap-tymur-tymchenko
Modified: 2025-01-11 12:54 UTC (History)
5 users (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments
before (19.02 KB, image/jpeg)
2024-12-11 18:56 UTC, ostap-tymur-tymchenko
Details
after (20.16 KB, image/jpeg)
2024-12-11 18:57 UTC, ostap-tymur-tymchenko
Details

Note You need to log in before you can comment on or make changes to this bug.
Description ostap-tymur-tymchenko 2024-12-11 18:56:21 UTC
Created attachment 176530 [details]
before

SUMMARY
Copy is not a shortcut for copy and Paste isnt a shortcut for paste

STEPS TO REPRODUCE
1.  Have a keyboard with a Copy or Paste dedicated key (KC_COPY, KC_PASTE in qmk)
2.  Try to use Copy or Paste

OBSERVED RESULT
No copy or paste

HOW TO RESOLVE:
Make Copy a shortcut for copy and Paste a shortcut for paste

SOFTWARE/OS VERSIONS
Operating System: Nobara Linux 40
KDE Plasma Version: 6.2.2
KDE Frameworks Version: 6.7.0
Qt Version: 6.7.2
Kernel Version: 6.11.5-200.fsync.fc40.x86_64 (64-bit)
Graphics Platform: Wayland
Processors: 12 × AMD Ryzen 5 7640U w/ Radeon 760M Graphics
Memory: 14.9 GiB of RAM
Graphics Processor: AMD Radeon 760M
Manufacturer: Framework
Product Name: Laptop 13 (AMD Ryzen 7040Series)
System Version: A5

ADDITIONAL INFORMATION
I've attached a screenshot of copy shortcut before/after
Comment 1 ostap-tymur-tymchenko 2024-12-11 18:57:27 UTC
Created attachment 176531 [details]
after
Comment 2 David Redondo 2024-12-16 14:01:12 UTC
Needs changing in kstandardshortcut this should make it work via our platformtheme. 

It will not  make it work in gtk apps.
Comment 3 David Redondo 2024-12-16 14:09:31 UTC
Should be relatively change if you want to go for it.

Place to change it would be here https://invent.kde.org/frameworks/kconfig/-/blob/master/src/gui/kstandardshortcut.cpp#L83 
However it already has two standard shortcuts, so it needs adding a third field to struct KStandardShortcutInfo
Comment 4 ostap-tymur-tymchenko 2024-12-17 17:10:38 UTC
(In reply to David Redondo from comment #3)
> Should be relatively change if you want to go for it.
> 
> Place to change it would be here
> https://invent.kde.org/frameworks/kconfig/-/blob/master/src/gui/
> kstandardshortcut.cpp#L83 
> However it already has two standard shortcuts, so it needs adding a third
> field to struct KStandardShortcutInfo

Maybe it shouldn't be a configurable shortcut at all? I can easily see how someone who doesn't know of kc_copy kc_paste would get very confused. I also think that a user unbinding kc_copy would in 99 out of 100 cases be by accident, and so maybe should be hard coded. 

Also to note that other keys exist to that we have not mentioned, like for example kc_cut which should be tested.