Re: [jQuery] Dimension plugin updates
Thanks, Brandon! That's fixed it! Brandon Aaron wrote: > 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 discuss@jquery.com http://jquery.com/discuss/ >>> ___ >>> jQuery mailing list >>> discuss@jquery.com >>> http://jquery.com/discuss/ >>> >>> >> ___ >> jQuery mailing list >> discuss@jquery.com >> http://jquery.com/discuss/ >> >> > > ___ > jQuery mailing list > discuss@jquery.com > http://jquery.com/discuss/ > ___ jQuery mailing list discuss@jquery.com http://jquery.com/discuss/
Re: [jQuery] Dimension plugin updates
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 > >> discuss@jquery.com > >> http://jquery.com/discuss/ > >> > >> > > > > ___ > > jQuery mailing list > > discuss@jquery.com > > http://jquery.com/discuss/ > > > > > ___ > jQuery mailing list > discuss@jquery.com > http://jquery.com/discuss/ > ___ jQuery mailing list discuss@jquery.com http://jquery.com/discuss/
Re: [jQuery] Dimension plugin updates
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 >> discuss@jquery.com >> http://jquery.com/discuss/ >> >> > > ___ > jQuery mailing list > discuss@jquery.com > http://jquery.com/discuss/ > ___ jQuery mailing list discuss@jquery.com http://jquery.com/discuss/
Re: [jQuery] Dimension plugin updates
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 > discuss@jquery.com > http://jquery.com/discuss/ > ___ jQuery mailing list discuss@jquery.com http://jquery.com/discuss/
[jQuery] Dimension plugin updates
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 discuss@jquery.com http://jquery.com/discuss/