OK, re-checked out the trunk and it's all good. Sorry for the false alarm. Odd...
On Mon, Feb 23, 2009 at 1:40 PM, Larry Kluger <[email protected]> wrote: > Hi Scott, > > I don't understand your email. The current version of Timeline, the trunk > version, does include the checks for the variable type of band.width. (The > issue is that the field is used for both distributing the relative sizes of > the bands, using a string such as 20% and for a minimum amount, as integer > number of pixels, when autowidth is turned on.) > > Anyhow, as far as I can tell, it is all working correctly in the trunk > version. Eg see the test for autowidth, > > http://simile-widgets.googlecode.com/svn/timeline/trunk/src/webapp/site/examples/test_example2/test_example2.html > > I suggest that you download a fresh copy from the trunk. > > Are you maintaining your own svn for storing your project? Maybe you've > updated to the version of Timeline that you are referencing in your own > repository.... > > Regards, > > Larry > > ------------------------------ > *From:* Scott Thomson <[email protected]> > *To:* [email protected] > *Sent:* Sunday, February 22, 2009 8:27:49 PM > *Subject:* AutoWidth, Zoom, Scroll, other issues > > Hi there, > > I updated the latest version of timeline (SVN 9475) this morning and found > an error: "F.IndexOf(%) is not a function". > > This turned out to be a bundled issue with timeline.js > > This code: > Timeline._Impl.prototype._distributeWidths = function() { > : > var x = widthString.indexOf("%"); > if (x > 0) { > var percent = parseInt(widthString.substr(0, x)); > var bandWidth = percent * width / 100; > } else { > var bandWidth = parseInt(widthString); > } > : > }; > > Had changed from this code: (differences bolded) > Timeline._Impl.prototype._distributeWidths = function() { > : > * if (typeof widthString == 'string') { > * var x = widthString.indexOf("%"); > if (x > 0) { > var percent = parseInt(widthString.substr(0, x)); > var bandWidth = percent * width / 100; > } else { > var bandWidth = parseInt(widthString); > } > * } else { > // was given an integer > bandWidth = widthString; > } > *: > }; > > Further to "fixing" this so I could get a timeline to show I seem to have > lost theme based autowidth, scrolling, zooming and my custom tape-painter is > no longer working. > > Can someone confirm that autowidth, scrolling and zooming still work after > the latest changes? I can look at my custom tape-painter. > > Thanks, > Scott > > > > > > > -- Scott Thomson 0401 726 889 --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "SIMILE Widgets" group. To post to this group, send email to [email protected] To unsubscribe from this group, send email to [email protected] For more options, visit this group at http://groups.google.com/group/simile-widgets?hl=en -~----------~----~----~----~------~----~------~--~---
