fwahlqvist / simile-widgets

Automatically exported from code.google.com/p/simile-widgets
0 stars 0 forks source link

TIMELINE: RFE--check that a band is not asked to syncWith itself. #35

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Move top timeline.  See sample files.

What is the expected output? What do you see instead?
Stack overflow.

What version of the product are you using? On what operating system?
TimeLine 2.2
IE 6.0.2900.2180.xpsp_sp2_qfe.070227-2300
Windows XP.
Call stack error does not occur with firefox, however lagged freezing
occurs with firefox.

Please provide any additional information below.
Appears to get caught in a loop in the Timeline._Band.prototype._onChanging
method.
First Timeline._Band.prototype._onMouseMove gets called, which calls 
this._moveEther(this._timeline.isHorizontal() ? diffX : diffY);
which calls Timeline._Band.prototype._onChanging
which calls _fireOnScroll()
which calls a scroll listener called (_syncWithBandHandler)
which calls _onHighlightBandScroll
which calls _moveEther
Loop repeats form this point back up to moveEther 6 steps above.
Call stack exceeds IE's capacity, IE craps out.

Original issue reported on code.google.com by tim.dims...@gmail.com on 24 Oct 2008 at 9:10

Attachments:

GoogleCodeExporter commented 9 years ago
It occurred to me what the problem was after I submitted it, sorry about this.

I had 
bandInfos[0].syncWith = 0; 
for whatever reason instead of 
bandInfos[0].syncWith = 1;

Since it was listening to itself it caused it's own loop.
My fault, not a bug, configuration error.  Not sure how to delete this bug.

Thanks.

Original comment by tim.dims...@gmail.com on 24 Oct 2008 at 9:14

GoogleCodeExporter commented 9 years ago
Thanks for all of the sleuthing and follow-up! I will change to an rfe to check 
that 
a Timeline is not being asked to sync with itself. 

Original comment by larryklu...@gmail.com on 26 Oct 2008 at 8:40

GoogleCodeExporter commented 9 years ago

Original comment by stefano.mazzocchi@gmail.com on 25 Mar 2009 at 7:01