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 770382 - Workspace switch indicator swallows mouse clicks on the entire screen
Workspace switch indicator swallows mouse clicks on the entire screen
Status: RESOLVED FIXED
Product: gnome-shell
Classification: Core
Component: general
3.20.x
Other Linux
: Normal normal
: ---
Assigned To: gnome-shell-maint
gnome-shell-maint
Depends on:
Blocks:
 
 
Reported: 2016-08-25 11:40 UTC by Timm Bäder
Modified: 2016-09-08 17:30 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
WorkspaceSwitcherPopup: allow input events to pass through (1.13 KB, patch)
2016-08-25 13:51 UTC, Rui Matos
committed Details | Review

Description Timm Bäder 2016-08-25 11:40:58 UTC
While the workspace switch indicator (that appears when pressing ctrl+alt+up/down) is visible, I'm unable to click anywhere on the screen. This is particularly annoying when trying to quickly focus another windows with the mouse directly after switching workspaces.
Comment 1 Rui Matos 2016-08-25 13:51:08 UTC
Created attachment 334145 [details] [review]
WorkspaceSwitcherPopup: allow input events to pass through

There's no particular reason for this actor to be reactive and thus
preventing input events to reach actors underneath, e.g. quickly
clicking on something while the popup isn't yet finished animating
out.
Comment 2 Timm Bäder 2016-08-30 13:58:08 UTC
That is the exact same fix I also tried locally but my second g-s instance didn't create any workspaces so I couldn't really try it. So if that fixes it for you, it's fine with me, thanks.
Comment 3 Timm Bäder 2016-09-08 14:05:18 UTC
Hey, can we get this in for 3.22?
Comment 4 Florian Müllner 2016-09-08 14:10:04 UTC
Review of attachment 334145 [details] [review]:

LGTM

::: js/ui/workspaceSwitcherPopup.js
@@ +21,2 @@
     _init : function() {
+        this.actor = new St.Widget({ reactive: false,

... or just remove the property altogether
Comment 5 Rui Matos 2016-09-08 17:30:48 UTC
Attachment 334145 [details] pushed as e4976b5 - WorkspaceSwitcherPopup: allow input events to pass through