From 6ce4605c3adb6933deb26d89ed0638298830b0df Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Florian=20M=C3=BCllner?= Date: Mon, 31 Aug 2020 21:11:18 +0200 Subject: [PATCH] workspacesView: Always clip to allocation We couldn't clip workspaces views during the overview transition when we used the "porthole" approach, but as view's allocation now always matches the expected visible area, we can just apply the clip unconditionally. https://gitlab.gnome.org/GNOME/gnome-shell/-/issues/3208 --- js/ui/workspacesView.js | 8 +------- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/js/ui/workspacesView.js b/js/ui/workspacesView.js index c3ad4b6cf..3b38bee02 100644 --- a/js/ui/workspacesView.js +++ b/js/ui/workspacesView.js @@ -74,6 +74,7 @@ class WorkspacesView extends WorkspacesViewBase { let workspaceManager = global.workspace_manager; super._init(monitorIndex); + this.clip_to_allocation = true; this._animating = false; // tweening this._gestureActive = false; // touch(pad) gestures @@ -96,10 +97,6 @@ class WorkspacesView extends WorkspacesViewBase { (ws, i) => this.set_child_at_index(ws, i)); }); - this._overviewShownId = Main.overview.connect('shown', () => { - this.clip_to_allocation = true; - }); - this._switchWorkspaceNotifyId = global.window_manager.connect('switch-workspace', this._activeWorkspaceChanged.bind(this)); @@ -147,8 +144,6 @@ class WorkspacesView extends WorkspacesViewBase { } animateFromOverview(animationType) { - this.clip_to_allocation = false; - for (let w = 0; w < this._workspaces.length; w++) { if (animationType == AnimationType.ZOOM) this._workspaces[w].zoomFromOverview(); @@ -232,7 +227,6 @@ class WorkspacesView extends WorkspacesViewBase { super._onDestroy(); this._scrollAdjustment.disconnect(this._onScrollId); - Main.overview.disconnect(this._overviewShownId); global.window_manager.disconnect(this._switchWorkspaceNotifyId); let workspaceManager = global.workspace_manager; workspaceManager.disconnect(this._updateWorkspacesId);