[gnome-shell/gbsneto/icon-grid-part2: 10/10] controlsManager: Don't fade icon grid while dragging



commit 3c6d31899a125c4317eb31aa7fc53772b476e2b2
Author: Georges Basile Stavracas Neto <georges stavracas gmail com>
Date:   Wed Jul 3 14:53:09 2019 -0300

    controlsManager: Don't fade icon grid while dragging
    
    As pointed out by designers, fading it signals that the
    icon grid is not a drop target, when now it actually is.
    
    Remove the fade effect applied to the icon grid when
    dragging. Since this is the only caller of fadeIn() and
    fadeHalf(), also remove these methods.
    
    https://gitlab.gnome.org/GNOME/gnome-shell/merge_requests/664

 js/ui/overviewControls.js | 11 -----------
 js/ui/viewSelector.js     | 18 ------------------
 2 files changed, 29 deletions(-)
---
diff --git a/js/ui/overviewControls.js b/js/ui/overviewControls.js
index c9f9872c8e..cf24a5e566 100644
--- a/js/ui/overviewControls.js
+++ b/js/ui/overviewControls.js
@@ -447,17 +447,6 @@ var ControlsManager = class {
         layout.connect('allocation-changed', this._updateWorkspacesGeometry.bind(this));
 
         Main.overview.connect('showing', this._updateSpacerVisibility.bind(this));
-        Main.overview.connect('item-drag-begin', () => {
-            let activePage = this.viewSelector.getActivePage();
-            if (activePage != ViewSelector.ViewPage.WINDOWS)
-                this.viewSelector.fadeHalf();
-        });
-        Main.overview.connect('item-drag-end', () => {
-            this.viewSelector.fadeIn();
-        });
-        Main.overview.connect('item-drag-cancelled', () => {
-            this.viewSelector.fadeIn();
-        });
     }
 
     _updateWorkspacesGeometry() {
diff --git a/js/ui/viewSelector.js b/js/ui/viewSelector.js
index 9aef3b9ddb..c4201239e8 100644
--- a/js/ui/viewSelector.js
+++ b/js/ui/viewSelector.js
@@ -599,23 +599,5 @@ var ViewSelector = class {
         else
             return ViewPage.SEARCH;
     }
-
-    fadeIn() {
-        let actor = this._activePage;
-        actor.ease({
-            opacity: 255,
-            duration: OverviewControls.SIDE_CONTROLS_ANIMATION_TIME / 2,
-            mode: Clutter.AnimationMode.EASE_IN_QUAD
-        });
-    }
-
-    fadeHalf() {
-        let actor = this._activePage;
-        actor.ease({
-            opacity: 128,
-            duration: OverviewControls.SIDE_CONTROLS_ANIMATION_TIME / 2,
-            mode: Clutter.AnimationMode.EASE_OUT_QUAD
-        });
-    }
 };
 Signals.addSignalMethods(ViewSelector.prototype);


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]