After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 730739 - alt-esc (cycle-windows) no longer cycles through all windows in workspace
alt-esc (cycle-windows) no longer cycles through all windows in workspace
Status: RESOLVED FIXED
Product: gnome-shell
Classification: Core
Component: general
3.12.x
Other Linux
: Normal normal
: ---
Assigned To: gnome-shell-maint
gnome-shell-maint
: 668629 753166 762742 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2014-05-25 22:56 UTC by Emilio Pozuelo Monfort
Modified: 2017-01-05 16:50 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
switcherPopup: Add a return value to _keyPressHandler (3.34 KB, patch)
2014-05-26 13:30 UTC, Rui Matos
committed Details | Review
altTab: Take over cycle-windows/cycle-group keybindings (6.26 KB, patch)
2016-05-19 13:42 UTC, Florian Müllner
none Details | Review
altTab: Take over cycle-windows/cycle-group keybindings (6.13 KB, patch)
2016-05-19 14:09 UTC, Florian Müllner
committed Details | Review
windowManager: Avoid code duplication (6.73 KB, patch)
2016-05-19 14:40 UTC, Florian Müllner
committed Details | Review

Description Emilio Pozuelo Monfort 2014-05-25 22:56:10 UTC
I just upgraded from 3.8 to 3.12. In 3.8, alt-tab used to cycle through all the windows in the workspace, but that seems broken in 3.12. Right now alt-tab seems to only switch between the two last used windows in the workspace, ignoring all the others.
Comment 1 Rui Matos 2014-05-26 09:06:19 UTC
What's the output of:

$ gsettings get org.gnome.desktop.wm.keybindings switch-applications
$ gsettings get org.gnome.desktop.wm.keybindings switch-windows

?
Comment 2 Emilio Pozuelo Monfort 2014-05-26 09:22:23 UTC
emilio@titan:~$ gsettings get org.gnome.desktop.wm.keybindings switch-applications
['<Super>Tab']
emilio@titan:~$ gsettings get org.gnome.desktop.wm.keybindings switch-windows
@as []

However, it is cycle-windows that I was using:

emilio@titan:~$ gsettings get org.gnome.desktop.wm.keybindings cycle-windows
['<Alt>Escape']

That used to work fine, but no longer does. switch-windows could work as a workaround though.
Comment 3 Rui Matos 2014-05-26 09:38:14 UTC
Now I'm puzzled. You had cycle-windows set to <Alt>Tab and after updating it turned into <Alt>Escape ?
Comment 4 Emilio Pozuelo Monfort 2014-05-26 09:47:40 UTC
*Sigh*, I said alt-tab in my original description but I really meant alt-esc... sorry for the confusion.

So it's alt-esc, i.e. cycle-windows, that isn't working well for me. alt-tab/super-tab is working well, but I don't use that.
Comment 5 Jasper St. Pierre (not reading bugmail) 2014-05-26 12:17:59 UTC
Yeah, I removed the Alt-Escape code in mutter, forgetting we didn't have a switcher popup replacement for it in gnome-shell. I don't really want to add back the code there, as it was quite dusty. I'll see about adding back a replacement to gnome-shell.
Comment 6 Emilio Pozuelo Monfort 2014-05-26 12:52:40 UTC
(In reply to comment #5)
> Yeah, I removed the Alt-Escape code in mutter, forgetting we didn't have a
> switcher popup replacement for it in gnome-shell. I don't really want to add
> back the code there, as it was quite dusty. I'll see about adding back a
> replacement to gnome-shell.

switch-windows could do as a replacement if doing:

press alt
press escape
release escape
press escape

cycled through apps. Unfortunately hat doesn't work; you need the mouse pointer or the arrow keys to cycle through apps. But I just want to do alt-esc-esc-esc :-)
Comment 7 Rui Matos 2014-05-26 13:30:58 UTC
Created attachment 277197 [details] [review]
switcherPopup: Add a return value to _keyPressHandler

This allows us to handle keybindings that use Escape instead of just
dismissing the popup unconditionally when we see an Escape press.
--

That was a bug indeed.
Comment 8 Florian Müllner 2014-05-26 13:44:41 UTC
Review of attachment 277197 [details] [review]:

OK
Comment 9 Rui Matos 2014-05-26 13:56:19 UTC
Attachment 277197 [details] pushed as dd85670 - switcherPopup: Add a return value to _keyPressHandler
Comment 10 Florian Müllner 2014-05-26 14:00:41 UTC
Reopening, as cycle-windows is still broken - the patch just fixed Escape in switch-windows/applications.
Comment 11 Emilio Pozuelo Monfort 2014-05-26 14:15:01 UTC
(In reply to comment #9)
> Attachment 277197 [details] pushed as dd85670 - switcherPopup: Add a return value to
> _keyPressHandler

Nice. Can this go to gnome-3-12 as well please?
Comment 12 Rui Matos 2014-05-26 15:26:17 UTC
42619cd..0327bd0  gnome-3-12 -> gnome-3-12
Comment 13 Florian Müllner 2015-08-03 06:44:49 UTC
*** Bug 753166 has been marked as a duplicate of this bug. ***
Comment 14 Marek Kašík 2015-10-06 09:45:53 UTC
Hi, is there any progress on this?
Comment 15 Florian Müllner 2016-02-26 19:25:50 UTC
*** Bug 762742 has been marked as a duplicate of this bug. ***
Comment 16 Florian Müllner 2016-05-19 13:42:03 UTC
Created attachment 328188 [details] [review]
altTab: Take over cycle-windows/cycle-group keybindings

The code to handle cycling through windows without showing a popup
was removed from mutter a while ago, which left the corresponding
keybindings mostly broken (i.e. they now only switch between two
windows). With the various switch-foo keybindings handled by the
shell, it is now easier to take over the cycle-foo keybindings as
well.
Comment 17 Oscar Morante 2016-05-19 14:02:04 UTC
Hi Florian,
I just tested your patch, and it's still switching only between the last two windows.  It also has a slightly different behavior and now the window is only focused when you release the alt key insted of doing it instantly like before.
Comment 18 Florian Müllner 2016-05-19 14:09:31 UTC
Created attachment 328189 [details] [review]
altTab: Take over cycle-windows/cycle-group keybindings

(In reply to Oscar Morante from comment #17)
> I just tested your patch, and it's still switching only between the last two
> windows.

Using mutter/gnome-shell from master? Are there any warning? FWIW, it works as expected in my testing ...


> It also has a slightly different behavior and now the window is only focused
> when you release the alt key insted of doing it instantly like before.

Yeah, I did that on purpose. The other switcher keybindings allow canceling the operation with Escape, so I didn't want to break that in case the binding itself doesn't involve Escape. Apparently that doesn't work without an actual popup though, so I updated the patch to switch windows immediately - that also fixes the ambiguity when switching between two unfocused windows that don't overlap ...
Comment 19 Marek Kašík 2016-05-19 14:11:58 UTC
I've just tested it too on Fedora 24 and it cycles all the windows now. The window is focused after I release Tab key but it works for me.
Comment 20 Marek Kašík 2016-05-19 14:31:24 UTC
The windows are focused immediately with the newer patch.
Comment 21 Florian Müllner 2016-05-19 14:40:35 UTC
Created attachment 328205 [details] [review]
windowManager: Avoid code duplication

The various switcher keybindings are handled identically, except for
the popup that is shown; update the code to reflect that instead of
duplicating the code again and again.

Optional cleanup on top of the last patch.
Comment 22 Oscar Morante 2016-05-19 14:48:03 UTC
(In reply to Florian Müllner from comment #18)
> Using mutter/gnome-shell from master? Are there any warning? FWIW, it works
> as expected in my testing ...

Oh, I tested on top of the last release, so that must be it.  Thanks for fixing this :)
Comment 23 Rui Matos 2016-05-19 15:02:51 UTC
Review of attachment 328205 [details] [review]:

looks good
Comment 24 Rui Matos 2016-05-19 15:03:25 UTC
Review of attachment 328189 [details] [review]:

looks good, works fine, ship it
Comment 25 Florian Müllner 2016-05-19 16:05:02 UTC
Attachment 328189 [details] pushed as bd6e7f1 - altTab: Take over cycle-windows/cycle-group keybindings
Attachment 328205 [details] pushed as bf8d306 - windowManager: Avoid code duplication
Comment 26 Florian Müllner 2017-01-05 16:50:57 UTC
*** Bug 668629 has been marked as a duplicate of this bug. ***