javascript/NativeScript
1
0
Fork 0
mirror of https://github.com/NativeScript/NativeScript.git synced 2025-08-16 11:42:04 +08:00
Code Issues Packages Projects Releases Wiki Activity

fix(animations): avoid steady mem consumption rise (#6004)

Browse Source
This commit is contained in:
Martin Yankov
2018-07-04 13:44:01 +03:00
committed by GitHub
parent a37da87c08
commit bcadcbe7b5
1 changed files with 14 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

19
tns-core-modules/ui/animation/keyframe-animation.ts
View File

@ -106,7 +106,7 @@ export class KeyframeAnimation implements KeyframeAnimationDefinition {
}
}
animations.map(a => a["curve"] ? a : Object.assign(a, {curve: info.curve}));
animations.map(a => a["curve"] ? a : Object.assign(a, { curve: info.curve }));
const animation: KeyframeAnimation = new KeyframeAnimation();
animation.delay = info.delay;
@ -230,9 +230,19 @@ export class KeyframeAnimation implements KeyframeAnimationDefinition {
}
}
else {
let animationDef = this.animations[index];
(<any>animationDef).target = view;
let animation = new Animation([animationDef]);
let animation;
const cachedAnimation = this._nativeAnimations[index - 1];
if (cachedAnimation) {
animation = cachedAnimation;
}
else {
let animationDef = this.animations[index];
(<any>animationDef).target = view;
animation = new Animation([animationDef]);
this._nativeAnimations.push(animation);
}
// Catch the animation cancel to prevent unhandled promise rejection warnings
animation.play().then(() => {
this.animate(view, index + 1, iterations);
@ -241,7 +251,6 @@ export class KeyframeAnimation implements KeyframeAnimationDefinition {
}).catch((error: any) => {
traceWrite(typeof error === "string" ? error : error.message, traceCategories.Animation, traceType.warn);
}); // tslint:disable-line
this._nativeAnimations.push(animation);
}
}