Alternative impl for copy-to-clipboard shortcut - Githubissues

keepassxreboot / keepassxc

KeePassXC is a cross-platform community-driven port of the Windows application “Keepass Password Safe”.

https://keepassxc.org/

Other

20.01k stars 1.42k forks source link

Alternative impl for copy-to-clipboard shortcut #10966

Open c4rlo opened 1 week ago

c4rlo commented 1 week ago

It turns out that the previous implementation, based on installing an event filter in every QAction instance, does not work on macOS, likely due to a Qt bug: #10929.

Attempt to work around this by using a different implementation of the same idea, by reacting to ShortcutOverride events in the MainWindow object.

Fixes #10929 – hopefully.

Testing strategy

Tested manually on Linux/Wayland. Also ran all tests, including GUI tests.

Type of change

✅ Bug fix (non-breaking change that fixes an issue)

droidmonkey commented 1 week ago

I'll give this a test on macOS