Looks like a couple of colons where causing the problem. Can you try
it again and let me know if it is still giving you an error?
--
Brandon Aaron
On 1/8/07, Dotan Dimet <[EMAIL PROTECTED]> wrote:
> Brandon, I just did an "svn update" and tried to parse the docs for the
> plugins,
>
> and I get an error in parse.js when it tries to parse dimensions.js:
>
> parsing file ../plugins/dimensions/dimensions.js
> js: "build/js/parse.js", line 41: uncaught JavaScript runtime exception:
> TypeError: Cannot set property "result" of undefined to "{ top: 100,
> left: 100 }"
>
> Any idea what's up with that?
>
> I think the line in parse.js is line 40:
>
> 40 if ( n == 'desc' || n == 'before' || n ==
> 'after' || n == 'result' ) {
> 41 ret.examples[ ret.examples.length -
> 1 ][ n ] = v;
>
>
> Erik Beeson wrote:
>
> > Thanks Brandon! For the lazy, get the latest version here:
> > http://jquery.com/dev/svn/trunk/plugins/dimensions/dimensions.js?format=raw
> > Or all the files related to the plugin here:
> > http://jquery.com/dev/svn/trunk/plugins/dimensions/
> >
> > --Erik
> >
> > On 1/7/07, Brandon Aaron <[EMAIL PROTECTED]> wrote:
> >
> >> I just finished checking in some major updates to the dimensions plugin.
> >>
> >> * Added unit tests
> >> * Fixed a bug in .innerHeight() and .innerWidth()
> >> * Major refactor of .offset() method to be faster and fix a couple of bugs
> >> * Added a test/example/benchmark page for the .offset() method
> >>
> >> All of this can be found in SVN but you can try the unit tests for
> >> yourself here:
> >> http://brandon.jquery.com/plugins/dimensions/test/unit.html
> >> http://brandon.jquery.com/plugins/dimensions/test/offset.html
> >>
> >> The .offset() method has changed a lot in the name of performance.
> >> First it takes an optional param called, includeScrollOffsets. By
> >> default this param is true and will calculate the scroll offsets. The
> >> returned object will have two extra properties called scrollTop and
> >> scrollLeft. These two properties are the cumulative scroll offset. If
> >> the includeScrollOffsets param is set to false, then the scroll offset
> >> calculation is ignored and that should provide a performance boost.
> >>
> >> Even with the changes made getting the offset of an element is a slow
> >> process. So try and cache the offset value when you can instead of
> >> calling it lots of times in succession.
> >>
> >> There is a bug in Firefox with the .scrollWidth property where it only
> >> returns width of the visible area, hence the one failure in Firefox. I
> >> could use some help with this nasty little bug. :)
> >>
> >> If you find any bugs or have a feature request feel free to log them
> >> and set the component to dimensions. http://jquery.com/dev/bugs/new/
> >>
> >> --
> >> Brandon Aaron
> >>
> >> _______________________________________________
> >> jQuery mailing list
> >> [email protected]
> >> http://jquery.com/discuss/
> >>
> >>
> >
> > _______________________________________________
> > jQuery mailing list
> > [email protected]
> > http://jquery.com/discuss/
> >
>
>
> _______________________________________________
> jQuery mailing list
> [email protected]
> http://jquery.com/discuss/
>
_______________________________________________
jQuery mailing list
[email protected]
http://jquery.com/discuss/
