[gnome-shell/gbsneto/icon-grid-part2: 10/10] controlsManager: Don't fade icon grid while dragging
- From: Georges Basile Stavracas Neto <gbsneto src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [gnome-shell/gbsneto/icon-grid-part2: 10/10] controlsManager: Don't fade icon grid while dragging
- Date: Thu, 8 Aug 2019 00:58:34 +0000 (UTC)
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]