If I understand the issue correctly changing this line (#318) in fx.js:
this.curAnim = prop;
to this;
this.curAnim = jQuery.extend({}, prop);
Should fix the problem. However, I have no test case to run this against.
--
Brandon Aaron
On 10/10/06, Klaus Hartl <[EMAIL PROTECTED]> wrote:
>
> > i was also thinking about copying the object but in a more basic version :)
> >
> > var anihide = options.anihide;
> > var anihide2 = anihide;
> > $(img).siblings("img:visible").animate(anihide2,
> > anitime).end().animate(anishow, anitime);
> > ;
> >
> > But that didn't work and i regret to say your workaround didn't work
> > either. :(
> >
> > I downloaded the uncompressed jquery and according to firebug the error is
> > situated in lines 91, 344, 979, 1350, 1359, 1370 and 1380. But that is
> > after the anihide object is set to true so i don't know if it's useful
> > information.
> >
> > If i find another way to avoid this i will let you know.
> >
>
> Maybe you didn't copy it in the right place... see it has to be copied
> every time animate is called.
>
> In the tabs plugin that looks something like:
>
> jQuery.fn.tabs = function(options) {
>
> // copying here from options is not sufficient...
>
> return this.each(function() {
> jQuery('a').click(function() {
> // 1. copy objects
> // 2. call animate...
> });
> });
>
> };
>
> Would be great if this bug is fixed, because I could completely move
> that out from the click event making the whole thing perform better I
> guess...
>
> -- Klaus
>
> _______________________________________________
> jQuery mailing list
> [email protected]
> http://jquery.com/discuss/
>
_______________________________________________
jQuery mailing list
[email protected]
http://jquery.com/discuss/
