killTweensOf by (stringId || tween)

[jonlepage]

Hey guys did you think it possible include a destroy with string id?
gsap.killTweensOf('id');
or maybe make this possible ?
gsap.killTweensOf( gsap.getById('id') );

I know we can do this
gsap.getById('id').kill();
but in special context , if id already destroy, it will crash.

[jackdoyle]

I assume you're talking about an ID that was assigned to an animation instance (not an ID from a DOM element), right?

It should be as easy as:

var animation = gsap.getById("id");
if (animation) {
  animation.kill();
}

[jonlepage]

var animation = gsap.getById("id");
if (animation) {
  animation.kill();
}

yes, and this is precisely what I want to avoid. Create a scope variable and put conditional everywhere !
Also this work, but am not fan gsap.getById("id") && gsap.getById("id").kill()
These already cleaner but it not very performance friendly in game engine !

[jackdoyle]

Oh, there are easy solutions for that too:

// make your own function:
function killById(id) {
    var animation = gsap.getById(id);
    if (animation) {
        animation.kill();
    }
}

// then it's super-simple each time you want to kill something: 
killById("myID")

• OR -

// slightly less performant, but doesn't create any variables at all:
gsap.getById("id") && gsap.getById("id").kill();

[jonlepage]

In fact I hoped to see it natively in gsap to avoid having to manage files, but its ok, you probably have your reasons :).
ok so i create a special file polyfill_gsap.js.

/** kill a single tween by id.
 * @param {String} id - the string id of the tween animation.
 * @returns {Boolean} - return true if succeed.
 */
gsap.killTweenById = function killTweenById (id=''){
    const tween = this.getById(id);
    if(tween){
         tween.kill();
         return true;
    }
    return false;
}

I don't know if this is the best way to do it, however, if you have a better approach for your engine, your suggest welcome :)