Re: [Dynapi-Dev] [Bug #128263] PathAnimation.stopAnimation() bug (& patch)

[Raymond S. <dst...@or...>]

I'm for including  the 'PathAnimation.prototype.stopAnimationNow ' method
Richard discussed.  If your doing chained animations that lead to other
events it's a must have.  Or the whole thing repeats again.


----- Original Message -----
From: "Richard :o" <ma...@ri...>
To: <dyn...@li...>
Sent: Tuesday, January 09, 2001 2:32 AM
Subject: Re: [Dynapi-Dev] [Bug #128263] PathAnimation.stopAnimation() bug (&
patch)


> Hi,
> I'm glad to see pathanim tackled at last.What you're sayng is correct, and
> is what I did in my modified pathanim (see:
> http://www.resass.f2s.com/?menu=examples&node=1  or
> http://www.richardinfo.f2s.com/?menu=examples&node=1
> under "dynapi.util.pathanim.chained.html" when the servers up again)
>
> But I also had to add the method stopAnimation now() , which is identical,
> but without
> invoke-ing pathstop, so the animation could be manually halted without
> starting the next one from the onpathstop listener:
>
> PathAnimation.prototype.stopAnimationNow = function () {
>  if (this.pathPlaying && this.pathPlaying.resets && !this.cancelThread &&
> this.dlyr!=null)
this.dlyr.moveTo(this.pathPlaying[0],this.pathPlaying[1]);
>  this.stop();
>  delete this.pathPlaying;  // only deletes unstored path
>  this.playing = false;
> };
>  And I don't understand why slideBy() has been scrapped just because you
can
> do that manually aswell; you can do most of the
> API manually, but it's easier if you can just call a method. And we still
> use moveBy(), so slideBy() keeps consistency, and saves changing existing
> webcode when you upgrade you're API.
>
> DynLayer.prototype.slideBy = function(x,y,inc,ms) {
>  if (x==null) x = 0;
>  if (y==null) y = 0;
>  this.slideTo(this.x+x,this.y+y,inc,ms);
> };
>
> Cheers,
> Richard  :o
>
> ma...@ri...
> www.richardinfo.com
> (Everything running on, and ported to the 19/12/2000 snapshot of DynAPI2)
>
> ----- Original Message -----
> From: <no...@so...>
> To: <ma...@ab...>; <no...@so...>;
> <dyn...@so...>
> Sent: Wednesday, January 10, 2001 9:30 AM
> Subject: [Dynapi-Dev] [Bug #128263] PathAnimation.stopAnimation() bug (&
> patch)
>
>
> > Bug #128263, was updated on 2001-Jan-10 00:30
> > Here is a current snapshot of the bug.
> >
> > Project: DynAPI 2
> > Category: API Extentions
> > Status: Open
> > Resolution: None
> > Bug Group: None
> > Priority: 5
> > Submitted by: marstr
> > Assigned to : nobody
> > Summary: PathAnimation.stopAnimation() bug (& patch)
> >
> > Details: dynapi/util/pathanim.js
> >
> > a bug in PathAnimation.stopAnimation() causes the onpathstop-event to
fire
> > before some values were set
> > (this.playing=false)
> > before you couldn't slide a DynLayer directly in a
> > onpathstop eventListener (while this.playing still
> > was true)
> >
> > i've moved the line
> > if (this.dlyr != null) this.dlyr.invokeEvent("pathstop");
> >
> > last in the method which solved the problem
> >
> > PathAnimation.prototype.stopAnimation = function () {
> > if (this.pathPlaying && this.pathPlaying.resets && !this.cancelThread &&
> > this.dlyr!=null)
> > this.dlyr.moveTo(this.pathPlaying[0],this.pathPlaying[1]);
> > this.stop();
> > delete this.pathPlaying;  // only deletes unstored path
> > this.playing = false;
> > if (this.dlyr != null) this.dlyr.invokeEvent("pathstop");
> > };
> >
> > For detailed info, follow this link:
> > http://sourceforge.net/bugs/?func=detailbug&bug_id=128263&group_id=5757
> >
> > _______________________________________________
> > Dynapi-Dev mailing list
> > Dyn...@li...
> > http://lists.sourceforge.net/mailman/listinfo/dynapi-dev
> > ____________________________________________________________
> > Get your free domain name and domain-based e-mail from
> > Namezero.com. New!  Namezero Plus domains now available.
> > Find out more at: http://www.namezero.com
> >
>
>
>
> _______________________________________________
> Dynapi-Dev mailing list
> Dyn...@li...
> http://lists.sourceforge.net/mailman/listinfo/dynapi-dev
>