Bug 310562 - Alt+Shift+Tab can't be used to "Walk Through Windows (Reverse)"
Summary: Alt+Shift+Tab can't be used to "Walk Through Windows (Reverse)"
Status: RESOLVED FIXED
Alias: None
Product: kwin
Classification: Plasma
Component: tabbox (show other bugs)
Version: 4.9.3
Platform: Arch Linux Linux
: NOR normal
Target Milestone: 4.10 Beta 2
Assignee: KWin default assignee
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-11-23 16:49 UTC by Martin
Modified: 2012-12-15 19:09 UTC (History)
1 user (show)

See Also:
Latest Commit: http://commits.kde.org/kde-workspace/7cd45390e8f11ec2be6b1733db2a0edf0b592367
Version Fixed In: 4.10
thomas.luebking: ReviewRequest+

Attachments
Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description Martin 2012-11-23 16:49:26 UTC 
When I press Alt+Shift+Tab it starts the windows switcher, but it does not step in reverse order.

I had the problem after clean install and I have it also when I was using "windows behavior / tasks switcher" to change to other combination and back to Alt+Shift+Tab.

My findings are:
* at least one more user experiences the same problem
* Alt+Shift+Tab combination works if I assign it to some other action for example "Run Command")
* other combination assigned to "Walk Through Windows (Reverse)" works for me, for example Ctrl+Alt+Tab
* Ctrl+Alt+Shift+Tab doesn't work either
* in opinion there is weird record in kglobalshortcutsrc (search for Alt+Shift+Backtab):
      Walk Through Windows (Reverse)=Alt+Shift+Tab,Alt+Shift+Backtab,Walk Through Windows (Reverse)


Reproducible: Always

Steps to Reproduce:
1. open multiple applications
2. press Alt+Shift+Tab
3. press Alt+Shift+Tab

Actual Results:  
windows switcher will be shown and it will not move in the list

Expected Results:  
it should move in the list of application in reverse order
Comment 1 Maxime Corteel 2012-11-23 16:53:19 UTC 
I experience the same behavior with 4.9.3.
Comment 2 Thomas Lübking 2012-11-23 17:01:13 UTC 
This is bug #153211,  274006, 174142 and several dupes.

Actually only the (default configured) Alt+Shift+Backtab works here, so get rid of Alt+Shift+Tab
What keyboard layout do you use?
Comment 3 Maxime Corteel 2012-11-23 17:12:15 UTC 
My keyboard layout is fr, but I get the same behavior with us...
Comment 4 Martin 2012-11-23 17:59:06 UTC 
I manually edited file ~/.kde4/share/config/kglobalshortcutsrc:

Walk Through Windows (Reverse)=Alt+Shift+Backtab,Alt+Shift+Backtab,Walk Through Windows (Reverse)

After reboot Alt+Shift+Tab works.

My layout is "us" and it is Logitech K520.
Comment 5 Thomas Lübking 2012-11-23 20:39:46 UTC 
It's because of the modal state of the tabbox.
KWin grabs input and handles shortcuts itself after the shortcut is initially invoked.
Since the original bug was not resolved by ensuring to read the right shortcut, but kglobalaccel to fire the shortcut for the wrong event you're still not triggering the "correct" (configured) shortcut (since you wanted to configure alt+shift+backtab in the first place) and kwin does not react on that one.

-> Therefore the tabbox implementation will have to copy the globalaccel (and by comment also Qt) hack on this (while i can not say i'm particularily happy with such solution)

@Martin: objections?
Comment 6 Thomas Lübking 2012-11-23 20:40:22 UTC 
mehhh - fixing component, since this is really a (compatibility) bug in the tabbox implementation.
Comment 7 Thomas Lübking 2012-11-23 21:09:16 UTC 
https://git.reviewboard.kde.org/r/107441/
Comment 8 Martin Flöser 2012-11-23 22:15:45 UTC 
> @Martin: objections?
it sucks, but no objections.

Looking forward to the day where we (at least partially) handle global 
shortcut ourselves thanks to Wayland.
Comment 9 Thomas Lübking 2012-12-14 22:17:29 UTC 
Git commit 7cd45390e8f11ec2be6b1733db2a0edf0b592367 by Thomas Lübking.
Committed on 23/11/2012 at 22:05.
Pushed by luebking into branch 'master'.

special tab/backtab handling to align kglobalaccel
FIXED-IN: 4.10
REVIEW: 107441

M  +46   -1    kwin/tabbox/tabbox.cpp

http://commits.kde.org/kde-workspace/7cd45390e8f11ec2be6b1733db2a0edf0b592367