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

animation: Add parameter for hiding stopped Spinner actor

Browse Source 
Not hiding leaves the empty actor space visible, which may have an
undesirable effect on the parent element's size or spacing/padding.

https://gitlab.gnome.org/GNOME/gnome-shell/merge_requests/832
This commit is contained in:
Joonas Henriksson 2019-11-18 22:24:05 +02:00
parent 6af25b282c
commit 8f4e91a738
1 changed files with 12 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
js/ui/animation.js
View File

@ -141,12 +141,15 @@ class Spinner extends AnimatedIcon {
_init(size, params) {
params = Params.parse(params, {
animate: false,
hideOnStop: false,
});
let file = Gio.File.new_for_uri('resource:///org/gnome/shell/theme/process-working.svg');
super._init(file, size);
this.opacity = 0;
this._animate = params.animate;
this._hideOnStop = params.hideOnStop;
this.visible = !this._hideOnStop;
}
_onDestroy() {
@ -156,6 +159,7 @@ class Spinner extends AnimatedIcon {
play() {
this.remove_all_transitions();
this.show();
if (this._animate) {
super.play();
@ -179,11 +183,18 @@ class Spinner extends AnimatedIcon {
opacity: 0,
duration: SPINNER_ANIMATION_TIME,
mode: Clutter.AnimationMode.LINEAR,
onComplete: () => super.stop(),
onComplete: () => {
super.stop();
if (this._hideOnStop)
this.hide();
},
});
} else {
this.opacity = 0;
super.stop();
if (this._hideOnStop)
this.hide();
}
}
});