Bug 156155 - effects->grabKeyboard() disables global shortcuts
Summary: effects->grabKeyboard() disables global shortcuts
Status: RESOLVED FIXED
Alias: None
Product: kwin
Classification: Plasma
Component: general (show other bugs)
Version: unspecified
Platform: Ubuntu Linux
: HI normal
Target Milestone: ---
Assignee: KWin default assignee
URL:
Keywords:
: 160647 165116 176282 176492 176495 187161 194536 (view as bug list)
Depends on:
Blocks:
 
Reported: 2008-01-19 03:22 UTC by Jonathan Thomas
Modified: 2023-06-03 21:27 UTC (History)
8 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Jonathan Thomas 2008-01-19 03:22:42 UTC
Version:            (using KDE 4.0.0)
Installed from:    Ubuntu Packages
OS:                Linux

I'd like the shortcut Crtl + F8 to actually toggle grid mode, instead of just initializing it. Basically, I'd like for pressing Ctrl + F8 in grid mode to take you back to the desktop you were in when you initialized grid mode.
This is a little creature comfort I got used to from Compiz... :P
Comment 1 Lubos Lunak 2008-04-10 13:32:00 UTC
*** Bug 160647 has been marked as a duplicate of this bug. ***
Comment 2 lucas 2008-06-12 14:15:01 UTC
And present windows while we are at it.
Comment 3 lucas 2008-06-13 10:56:24 UTC
Both plugins already have toggle support, however the moment they grab the keyboard events global shortcuts are disabled until ungrabbed.
Comment 4 Lubos Lunak 2008-06-28 13:05:10 UTC
*** Bug 165116 has been marked as a duplicate of this bug. ***
Comment 5 Klaus S. Madsen 2008-11-04 09:46:44 UTC
I concur, I'd really like CTRL+F8 and CTRL+F9 to toggle, since this just feels natural to me. Everytime I use it, and want to exit, I press the shortcut again, and when nothing happens, I have to remember that ESC exits.
Comment 6 lucas 2008-11-28 04:08:23 UTC
*** Bug 176282 has been marked as a duplicate of this bug. ***
Comment 7 David Nadlinger 2008-11-29 22:27:22 UTC
*** Bug 176492 has been marked as a duplicate of this bug. ***
Comment 8 David Nadlinger 2008-11-29 22:33:47 UTC
*** Bug 176495 has been marked as a duplicate of this bug. ***
Comment 9 Martin Flöser 2009-03-14 17:38:06 UTC
*** Bug 187161 has been marked as a duplicate of this bug. ***
Comment 10 Martin Flöser 2009-03-30 13:27:58 UTC
SVN commit 946787 by graesslin:

Check for the global shortcut in the effects so that the shortcut actually toggles the effect. This is a workaround needed because the keyboard grab disables the global shortcut.
FEATURE: 156155


 M  +39 -0     cube/cube.cpp  
 M  +11 -0     cube/cube.h  
 M  +15 -0     desktopgrid/desktopgrid.cpp  
 M  +7 -0      desktopgrid/desktopgrid.h  
 M  +27 -0     presentwindows/presentwindows.cpp  
 M  +10 -0     presentwindows/presentwindows.h  


WebSVN link: http://websvn.kde.org/?view=rev&revision=946787
Comment 11 Martin Flöser 2009-05-29 14:50:44 UTC
*** Bug 194536 has been marked as a duplicate of this bug. ***
Comment 12 ratijas 2022-09-17 07:55:19 UTC
Git commit 984388dba4a29746e07137bbad46186ef06a80b5 by ivan tkachenko.
Committed on 16/09/2022 at 22:57.
Pushed by ratijas into branch 'master'.

effects/windowview: Fix "Toggle Present Windows (Window class on current desktop)" shortcut

The effect was not turning off when pressing a shortcut again, unlike
other flavors of this effects.
FIXED-IN: 5.26

M  +3    -3    src/effects/windowview/windowvieweffect.cpp

https://invent.kde.org/plasma/kwin/commit/984388dba4a29746e07137bbad46186ef06a80b5
Comment 13 ratijas 2022-09-17 07:56:00 UTC
Git commit 4d55ad3761c07587284d98b10b016e3a9d743d78 by ivan tkachenko.
Committed on 17/09/2022 at 07:55.
Pushed by ratijas into branch 'Plasma/5.26'.

effects/windowview: Fix "Toggle Present Windows (Window class on current desktop)" shortcut

The effect was not turning off when pressing a shortcut again, unlike
other flavors of this effects.
FIXED-IN: 5.26
(cherry picked from commit 984388dba4a29746e07137bbad46186ef06a80b5)

M  +3    -3    src/effects/windowview/windowvieweffect.cpp

https://invent.kde.org/plasma/kwin/commit/4d55ad3761c07587284d98b10b016e3a9d743d78
Comment 14 Bug Janitor Service 2022-09-20 09:28:08 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/kwin/-/merge_requests/2981
Comment 15 Nate Graham 2023-06-03 21:27:40 UTC
This has been working for a while.