isa/gnome-shell
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

appDisplay/baseAppView: Cleanup animate()

Browse Source 
Simplify the opacity dance to simply setting it to the initial value
before animating (0 when animating in, and 255 when animating out),
and to the final value after the spring animation is done (vice-versa).

Part-of: <https://gitlab.gnome.org/GNOME/gnome-shell/-/merge_requests/1518>
This commit is contained in:
Georges Basile Stavracas Neto 2020-12-02 18:02:46 -03:00 committed by Marge Bot
parent b730875012
commit cd82691859
1 changed files with 9 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
js/ui/appDisplay.js
View File

@ -670,7 +670,6 @@ var BaseAppView = GObject.registerClass({
}
_doSpringAnimation(animationDirection) {
this._grid.opacity = 255;
this._grid.animateSpring(
animationDirection,
Main.overview.dash.showAppsButton);
@ -688,15 +687,16 @@ var BaseAppView = GObject.registerClass({
}
animate(animationDirection, onComplete) {
if (onComplete) {
let animationDoneId = this._grid.connect('animation-done', () => {
this._grid.disconnect(animationDoneId);
const animationDoneId = this._grid.connect('animation-done', () => {
this._grid.disconnect(animationDoneId);
this._grid.opacity =
animationDirection === IconGrid.AnimationDirection.IN
? 255 : 0;
if (onComplete)
onComplete();
});
}
});
this._clearAnimateLater();
this._grid.opacity = 255;
if (animationDirection == IconGrid.AnimationDirection.IN) {
const doSpringAnimationLater = laterType => {
@ -708,18 +708,18 @@ var BaseAppView = GObject.registerClass({
});
};
this._grid.opacity = 0;
if (this._viewIsReady) {
this._grid.opacity = 0;
doSpringAnimationLater(Meta.LaterType.IDLE);
} else {
this._viewLoadedHandlerId = this.connect('view-loaded',
() => {
this._clearAnimateLater();
this._grid.opacity = 255;
doSpringAnimationLater(Meta.LaterType.BEFORE_REDRAW);
});
}
} else {
this._grid.opacity = 255;
this._doSpringAnimation(animationDirection);
}
}