[gnome-shell/wip/exalm/gestures: 6/11] workspaceAnimation: Extract WorkspaceAnimation
- From: Alexander Mikhaylenko <alexm src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [gnome-shell/wip/exalm/gestures: 6/11] workspaceAnimation: Extract WorkspaceAnimation
- Date: Mon, 5 Aug 2019 12:17:40 +0000 (UTC)
commit 6996ede0661dec73e5d8d68f37495be493c11f14
Author: Alexander Mikhaylenko <exalm7659 gmail com>
Date: Thu Jul 4 20:58:05 2019 +0500
workspaceAnimation: Extract WorkspaceAnimation
Simplify the logic a bit. Introduce WorkspaceAnimation class that reparents
the windows from current, surrounding and destination workspaces and manages
them. Expose 'progress' property and have WorkspaceAnimationController animate
it instead of animating everything separately.
https://gitlab.gnome.org/GNOME/gnome-shell/merge_requests/605
js/ui/workspaceAnimation.js | 442 ++++++++++++++++++++++----------------------
1 file changed, 218 insertions(+), 224 deletions(-)
---
diff --git a/js/ui/workspaceAnimation.js b/js/ui/workspaceAnimation.js
index 0d1c93688..43de8c188 100644
--- a/js/ui/workspaceAnimation.js
+++ b/js/ui/workspaceAnimation.js
@@ -1,7 +1,7 @@
// -*- mode: js; js-indent-level: 4; indent-tabs-mode: nil -*-
/* exported WorkspaceAnimationController */
-const { Clutter, Meta, Shell } = imports.gi;
+const { Clutter, GObject, Meta, Shell } = imports.gi;
const Main = imports.ui.main;
const Tweener = imports.ui.tweener;
@@ -9,125 +9,23 @@ const SwipeTracker = imports.ui.swipeTracker;
var WINDOW_ANIMATION_TIME = 0.25;
-var WorkspaceAnimationController = class {
- constructor() {
- this._shellwm = global.window_manager;
- this._movingWindow = null;
-
- this._switchData = null;
- this._shellwm.connect('kill-switch-workspace', (shellwm) => {
- if (this._switchData) {
- if (this._switchData.inProgress)
- this._switchWorkspaceDone(shellwm);
- else if (!this._switchData.gestureActivated)
- this._finishWorkspaceSwitch(this._switchData);
- }
- });
-
- global.display.connect('restacked', this._syncStacking.bind(this));
-
- Main.overview.connect('showing', () => {
- if (this._switchData) {
- if (this._switchData.gestureActivated)
- this._switchWorkspaceStop();
- this._swipeTracker.enabled = false;
- }
- });
- Main.overview.connect('hiding', () => {
- this._swipeTracker.enabled = true;
- });
-
- let allowedModes = Shell.ActionMode.NORMAL;
- let swipeTracker = new SwipeTracker.SwipeTracker(global.stage, allowedModes, false, false);
- swipeTracker.connect('begin', this._switchWorkspaceBegin.bind(this));
- swipeTracker.connect('update', this._switchWorkspaceUpdate.bind(this));
- swipeTracker.connect('end', this._switchWorkspaceEnd.bind(this));
- this._swipeTracker = swipeTracker;
- }
-
- _syncStacking() {
- if (this._switchData == null)
- return;
-
- let windows = global.get_window_actors();
- let lastCurSibling = null;
- let lastDirSibling = [];
- for (let i = 0; i < windows.length; i++) {
- if (windows[i].get_parent() == this._switchData.curGroup) {
- this._switchData.curGroup.set_child_above_sibling(windows[i], lastCurSibling);
- lastCurSibling = windows[i];
- } else {
- for (let dir of Object.values(Meta.MotionDirection)) {
- let info = this._switchData.surroundings[dir];
- if (!info || windows[i].get_parent() != info.actor)
- continue;
-
- let sibling = lastDirSibling[dir];
- if (sibling == undefined)
- sibling = null;
-
- info.actor.set_child_above_sibling(windows[i], sibling);
- lastDirSibling[dir] = windows[i];
- break;
- }
- }
- }
- }
-
- _getPositionForDirection(direction, fromWs, toWs) {
- let xDest = 0, yDest = 0;
-
- let oldWsIsFullscreen = fromWs.list_windows().some(w => w.is_fullscreen());
- let newWsIsFullscreen = toWs.list_windows().some(w => w.is_fullscreen());
-
- // We have to shift windows up or down by the height of the panel to prevent having a
- // visible gap between the windows while switching workspaces. Since fullscreen windows
- // hide the panel, they don't need to be shifted up or down.
- let shiftHeight = Main.panel.height;
-
- if (direction == Meta.MotionDirection.UP ||
- direction == Meta.MotionDirection.UP_LEFT ||
- direction == Meta.MotionDirection.UP_RIGHT)
- yDest = -global.screen_height + (oldWsIsFullscreen ? 0 : shiftHeight);
- else if (direction == Meta.MotionDirection.DOWN ||
- direction == Meta.MotionDirection.DOWN_LEFT ||
- direction == Meta.MotionDirection.DOWN_RIGHT)
- yDest = global.screen_height - (newWsIsFullscreen ? 0 : shiftHeight);
-
- if (direction == Meta.MotionDirection.LEFT ||
- direction == Meta.MotionDirection.UP_LEFT ||
- direction == Meta.MotionDirection.DOWN_LEFT)
- xDest = -global.screen_width;
- else if (direction == Meta.MotionDirection.RIGHT ||
- direction == Meta.MotionDirection.UP_RIGHT ||
- direction == Meta.MotionDirection.DOWN_RIGHT)
- xDest = global.screen_width;
-
- return [xDest, yDest];
- }
-
- _prepareWorkspaceSwitch(from, to, direction) {
- if (this._switchData)
- return;
+var WorkspaceAnimation = class {
+ constructor(controller, from, to, direction) {
+ this._controller = controller;
+ this._curGroup = new Clutter.Actor();
+ this._movingWindowBin = new Clutter.Actor();
+ this._windows = [];
+ this._surroundings = {};
+ this._progress = 0;
let wgroup = global.window_group;
let windows = global.get_window_actors();
- let switchData = {};
-
- this._switchData = switchData;
- switchData.curGroup = new Clutter.Actor();
- switchData.movingWindowBin = new Clutter.Actor();
- switchData.windows = [];
- switchData.surroundings = {};
- switchData.gestureActivated = false;
- switchData.inProgress = false;
- switchData.progress = 0;
- switchData.container = new Clutter.Actor();
- switchData.container.add_actor(switchData.curGroup);
+ this._container = new Clutter.Actor();
+ this._container.add_actor(this._curGroup);
- wgroup.add_actor(switchData.movingWindowBin);
- wgroup.add_actor(switchData.container);
+ wgroup.add_actor(this._movingWindowBin);
+ wgroup.add_actor(this._container);
let workspaceManager = global.workspace_manager;
let curWs = workspaceManager.get_workspace_by_index(from);
@@ -141,7 +39,7 @@ var WorkspaceAnimationController = class {
ws = workspaceManager.get_workspace_by_index(to);
if (ws == null || ws == curWs) {
- switchData.surroundings[dir] = null;
+ this._surroundings[dir] = null;
continue;
}
@@ -150,14 +48,14 @@ var WorkspaceAnimationController = class {
actor: new Clutter.Actor(),
xDest: x,
yDest: y };
- switchData.surroundings[dir] = info;
- switchData.container.add_actor(info.actor);
+ this._surroundings[dir] = info;
+ this._container.add_actor(info.actor);
info.actor.raise_top();
info.actor.set_position(x, y);
}
- switchData.movingWindowBin.raise_top();
+ this._movingWindowBin.raise_top();
for (let i = 0; i < windows.length; i++) {
let actor = windows[i];
@@ -172,22 +70,22 @@ var WorkspaceAnimationController = class {
let record = { window: actor,
parent: actor.get_parent() };
- if (this.movingWindow && window == this.movingWindow) {
- switchData.movingWindow = record;
- switchData.windows.push(switchData.movingWindow);
- actor.reparent(switchData.movingWindowBin);
+ if (this._controller.movingWindow && window == this._controller.movingWindow) {
+ this._movingWindow = record;
+ this._windows.push(this._movingWindow);
+ actor.reparent(this._movingWindowBin);
} else if (window.get_workspace().index() == from) {
- switchData.windows.push(record);
- actor.reparent(switchData.curGroup);
+ this._windows.push(record);
+ actor.reparent(this._curGroup);
} else {
let visible = false;
for (let dir of Object.values(Meta.MotionDirection)) {
- let info = switchData.surroundings[dir];
+ let info = this._surroundings[dir];
if (!info || info.index != window.get_workspace().index())
continue;
- switchData.windows.push(record);
+ this._windows.push(record);
actor.reparent(info.actor);
visible = true;
break;
@@ -197,20 +95,20 @@ var WorkspaceAnimationController = class {
}
}
- for (let i = 0; i < switchData.windows.length; i++) {
- let w = switchData.windows[i];
+ for (let i = 0; i < this._windows.length; i++) {
+ let w = this._windows[i];
w.windowDestroyId = w.window.connect('destroy', () => {
- switchData.windows.splice(switchData.windows.indexOf(w), 1);
+ this._windows.splice(this._windows.indexOf(w), 1);
});
}
- }
- _finishWorkspaceSwitch(switchData) {
- this._switchData = null;
+ global.display.connect('restacked', this._syncStacking.bind(this));
+ }
- for (let i = 0; i < switchData.windows.length; i++) {
- let w = switchData.windows[i];
+ destroy() {
+ for (let i = 0; i < this._windows.length; i++) {
+ let w = this._windows[i];
w.window.disconnect(w.windowDestroyId);
w.window.reparent(w.parent);
@@ -219,52 +117,70 @@ var WorkspaceAnimationController = class {
global.workspace_manager.get_active_workspace())
w.window.hide();
}
- Tweener.removeTweens(switchData);
- Tweener.removeTweens(switchData.container);
- switchData.container.destroy();
- switchData.movingWindowBin.destroy();
- this.movingWindow = null;
+ this._container.destroy();
+ this._movingWindowBin.destroy();
}
- animateSwitchWorkspace(shellwm, from, to, direction) {
- this._prepareWorkspaceSwitch(from, to, direction);
- this._switchData.inProgress = true;
+ _getPositionForDirection(direction, fromWs, toWs) {
+ let xDest = 0, yDest = 0;
- let workspaceManager = global.workspace_manager;
- let fromWs = workspaceManager.get_workspace_by_index(from);
- let toWs = workspaceManager.get_workspace_by_index(to);
+ let oldWsIsFullscreen = fromWs.list_windows().some(w => w.is_fullscreen());
+ let newWsIsFullscreen = toWs.list_windows().some(w => w.is_fullscreen());
- let [xDest, yDest] = this._getPositionForDirection(direction, fromWs, toWs);
+ // We have to shift windows up or down by the height of the panel to prevent having a
+ // visible gap between the windows while switching workspaces. Since fullscreen windows
+ // hide the panel, they don't need to be shifted up or down.
+ let shiftHeight = Main.panel.height;
- /* @direction is the direction that the "camera" moves, so the
- * screen contents have to move one screen's worth in the
- * opposite direction.
- */
- xDest = -xDest;
- yDest = -yDest;
+ if (direction == Meta.MotionDirection.UP ||
+ direction == Meta.MotionDirection.UP_LEFT ||
+ direction == Meta.MotionDirection.UP_RIGHT)
+ yDest = -global.screen_height + (oldWsIsFullscreen ? 0 : shiftHeight);
+ else if (direction == Meta.MotionDirection.DOWN ||
+ direction == Meta.MotionDirection.DOWN_LEFT ||
+ direction == Meta.MotionDirection.DOWN_RIGHT)
+ yDest = global.screen_height - (newWsIsFullscreen ? 0 : shiftHeight);
- Tweener.addTween(this._switchData,
- { progress: direction == Meta.MotionDirection.DOWN ? 1 : -1,
- time: WINDOW_ANIMATION_TIME,
- transition: 'easeOutCubic'
- });
- Tweener.addTween(this._switchData.container,
- { x: xDest,
- y: yDest,
- time: WINDOW_ANIMATION_TIME,
- transition: 'easeOutCubic',
- onComplete: this._switchWorkspaceDone,
- onCompleteScope: this,
- onCompleteParams: [shellwm] });
+ if (direction == Meta.MotionDirection.LEFT ||
+ direction == Meta.MotionDirection.UP_LEFT ||
+ direction == Meta.MotionDirection.DOWN_LEFT)
+ xDest = -global.screen_width;
+ else if (direction == Meta.MotionDirection.RIGHT ||
+ direction == Meta.MotionDirection.UP_RIGHT ||
+ direction == Meta.MotionDirection.DOWN_RIGHT)
+ xDest = global.screen_width;
+
+ return [xDest, yDest];
}
- _switchWorkspaceDone(shellwm) {
- this._finishWorkspaceSwitch(this._switchData);
- shellwm.completed_switch_workspace();
+ _syncStacking() {
+ let windows = global.get_window_actors();
+ let lastCurSibling = null;
+ let lastDirSibling = [];
+ for (let i = 0; i < windows.length; i++) {
+ if (windows[i].get_parent() == this._curGroup) {
+ this._curGroup.set_child_above_sibling(windows[i], lastCurSibling);
+ lastCurSibling = windows[i];
+ } else {
+ for (let dir of Object.values(Meta.MotionDirection)) {
+ let info = this._surroundings[dir];
+ if (!info || windows[i].get_parent() != info.actor)
+ continue;
+
+ let sibling = lastDirSibling[dir];
+ if (sibling == undefined)
+ sibling = null;
+
+ info.actor.set_child_above_sibling(windows[i], sibling);
+ lastDirSibling[dir] = windows[i];
+ break;
+ }
+ }
+ }
}
- _directionForProgress(progress) {
+ directionForProgress(progress) {
if (global.workspace_manager.layout_rows == -1)
return (progress > 0) ? Meta.MotionDirection.DOWN : Meta.MotionDirection.UP;
else if (Clutter.get_default_text_direction () == Clutter.TextDirection.RTL)
@@ -273,6 +189,117 @@ var WorkspaceAnimationController = class {
return (progress > 0) ? Meta.MotionDirection.RIGHT : Meta.MotionDirection.LEFT;
}
+ get progress() {
+ return this._progress;
+ }
+
+ set progress(progress) {
+ this._progress = progress;
+
+ let direction = this.directionForProgress(progress);
+ let info = this._surroundings[direction];
+ let xPos = 0;
+ let yPos = 0;
+ if (info) {
+ if (global.workspace_manager.layout_rows == -1)
+ yPos = Math.round(Math.abs(progress) * -info.yDest);
+ else
+ xPos = Math.round(Math.abs(progress) * -info.xDest);
+ }
+
+ this._container.set_position(xPos, yPos);
+ }
+
+ getDistance(direction) {
+ let info = this._surroundings[direction];
+ if (!info)
+ return 0;
+
+ switch (direction) {
+ case Meta.MotionDirection.UP:
+ return -info.yDest;
+ case Meta.MotionDirection.DOWN:
+ return info.yDest;
+ case Meta.MotionDirection.LEFT:
+ return -info.xDest;
+ case Meta.MotionDirection.RIGHT:
+ return info.xDest;
+ }
+
+ return 0;
+ }
+};
+
+var WorkspaceAnimationController = class {
+ constructor() {
+ this._shellwm = global.window_manager;
+ this._blockAnimations = false;
+ this._movingWindow = null;
+ this._inProgress = false;
+ this._gestureActivated = false;
+ this._animation = null;
+
+ this._shellwm.connect('kill-switch-workspace', (shellwm) => {
+ if (this._animation) {
+ if (this._inProgress)
+ this._switchWorkspaceDone(shellwm);
+ else if (!this._gestureActivated)
+ this._finishWorkspaceSwitch();
+ }
+ });
+
+ Main.overview.connect('showing', () => {
+ if (this._gestureActivated)
+ this._switchWorkspaceStop();
+
+ this._swipeTracker.enabled = false;
+ });
+ Main.overview.connect('hiding', () => {
+ this._swipeTracker.enabled = true;
+ });
+
+ let allowedModes = Shell.ActionMode.NORMAL;
+ let swipeTracker = new SwipeTracker.SwipeTracker(global.stage, allowedModes, false, false);
+ swipeTracker.connect('begin', this._switchWorkspaceBegin.bind(this));
+ swipeTracker.connect('update', this._switchWorkspaceUpdate.bind(this));
+ swipeTracker.connect('end', this._switchWorkspaceEnd.bind(this));
+ this._swipeTracker = swipeTracker;
+ }
+
+ _prepareWorkspaceSwitch(from, to, direction) {
+ if (this._animation)
+ return;
+
+ this._animation = new WorkspaceAnimation(this, from, to, direction);
+ }
+
+ _finishWorkspaceSwitch() {
+ Tweener.removeTweens(this._animation);
+ this._animation.destroy();
+ this._animation = null;
+ this._inProgress = false;
+ this._gestureActivated = false;
+ this.movingWindow = null;
+ }
+
+ animateSwitchWorkspace(shellwm, from, to, direction) {
+ this._prepareWorkspaceSwitch(from, to, direction);
+ this._inProgress = true;
+
+ Tweener.addTween(this._animation,
+ { progress: direction == Meta.MotionDirection.DOWN ? 1 : -1,
+ time: WINDOW_ANIMATION_TIME,
+ transition: 'easeOutCubic',
+ onComplete: this._switchWorkspaceDone,
+ onCompleteScope: this,
+ onCompleteParams: [shellwm] });
+ }
+
+ _switchWorkspaceDone(shellwm) {
+ this._finishWorkspaceSwitch();
+ shellwm.completed_switch_workspace();
+ }
+
_switchWorkspaceBegin(tracker, monitor) {
if (Meta.prefs_get_workspaces_only_on_primary() && monitor != Main.layoutManager.primaryIndex)
return;
@@ -284,10 +311,9 @@ var WorkspaceAnimationController = class {
let activeWorkspace = workspaceManager.get_active_workspace();
let progress = 0;
- if (this._switchData && this._switchData.gestureActivated) {
- Tweener.removeTweens(this._switchData);
- Tweener.removeTweens(this._switchData.container);
- progress = this._switchData.progress;
+ if (this._gestureActivated) {
+ Tweener.removeTweens(this._animation);
+ progress = this._animation.progress;
} else {
this._prepareWorkspaceSwitch(activeWorkspace.index(), -1);
progress = 0;
@@ -300,60 +326,36 @@ var WorkspaceAnimationController = class {
else
baseDistance = global.screen_height;
- let direction = this._directionForProgress(-1);
- let info = this._switchData.surroundings[direction];
- if (info != null) {
- let distance = horiz ? info.xDest : info.yDest;
- points.push(-Math.abs(distance) / baseDistance);
- }
+ let direction = this._animation.directionForProgress(-1);
+ let distance = this._animation.getDistance(direction);
+ if (distance != 0)
+ points.push(-distance / baseDistance);
points.push(0);
- direction = this._directionForProgress(1);
- info = this._switchData.surroundings[direction];
- if (info != null) {
- let distance = horiz ? info.xDest : info.yDest;
- points.push(Math.abs(distance) / baseDistance);
- }
+ direction = this._animation.directionForProgress(1);
+ distance = this._animation.getDistance(direction);
+ if (distance != 0)
+ points.push(distance / baseDistance);
tracker.confirmSwipe(baseDistance, points, progress, 0);
}
_switchWorkspaceUpdate(_tracker, progress) {
- if (!this._switchData)
- return;
-
- this._switchData.progress = progress;
-
- let direction = this._directionForProgress(progress);
- let info = this._switchData.surroundings[direction];
- let distance = 0;
- let xPos = 0;
- let yPos = 0;
- if (info) {
- if (global.workspace_manager.layout_rows == -1)
- yPos = Math.round(Math.abs(progress) * -info.yDest);
- else
- xPos = Math.round(Math.abs(progress) * -info.xDest);
- }
-
- this._switchData.container.set_position(xPos, yPos);
+ if (this._animation)
+ this._animation.progress = progress;
}
_switchWorkspaceEnd(_tracker, duration, endProgress) {
- if (!this._switchData)
+ if (!this._animation)
return;
let workspaceManager = global.workspace_manager;
let activeWorkspace = workspaceManager.get_active_workspace();
let newWs = activeWorkspace;
- let xDest = 0;
- let yDest = 0;
if (endProgress != 0) {
- let direction = this._directionForProgress(endProgress);
+ let direction = this._animation.directionForProgress(endProgress);
newWs = activeWorkspace.get_neighbor(direction);
- xDest = -this._switchData.surroundings[direction].xDest;
- yDest = -this._switchData.surroundings[direction].yDest;
}
if (duration == 0) {
@@ -363,34 +365,26 @@ var WorkspaceAnimationController = class {
return;
}
- let switchData = this._switchData;
- switchData.gestureActivated = true;
+ this._gestureActivated = true;
- Tweener.addTween(switchData,
+ Tweener.addTween(this._animation,
{ progress: endProgress,
- time: duration,
- transition: 'easeOutCubic' });
- Tweener.addTween(switchData.container,
- { x: xDest,
- y: yDest,
time: duration,
transition: 'easeOutCubic',
onComplete: () => {
if (newWs != activeWorkspace)
newWs.activate(global.get_current_time());
- this._finishWorkspaceSwitch(switchData);
+ this._finishWorkspaceSwitch();
} });
}
_switchWorkspaceStop() {
- this._switchData.progress = 0;
- this._switchData.container.x = 0;
- this._switchData.container.y = 0;
- this._finishWorkspaceSwitch(this._switchData);
+ this._animation.progress = 0;
+ this._finishWorkspaceSwitch();
}
isAnimating() {
- return this._switchData != null;
+ return this._animation != null;
}
set movingWindow(movingWindow) {
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]
