search

davist11 / jQuery-One-Page-Nav

Smooth scrolling and smart navigation when user scrolls on one-page sites.
http://davist11.github.com/jQuery-One-Page-Nav/
MIT License
1.52k stars 501 forks source link

Uncaught TypeError: Cannot read property 'top' of null #6

Closed saltcod closed 12 years ago

saltcod commented 12 years ago

Hi Trevor,

I've been using and loving your plugin. But I stumbled across an issue earlier today. I can't even point to anything I did—the ScrollTop still works, jQuery still loads, but your plugin calls an UncaughtType Error. Just out of the blue.

I can make the error go away by adding a +1 to line 65—don't know why that happens.... topPos = (divPos +1).top;

I'm probably just going to start from scratch, but I wonder if its something I did or if its an error.

Full Error Text:

Uncaught TypeError: Cannot read property 'top' of null $.fn.onePageNav.onePageNav.getPositionsjquery.nav.js:65 e.extend.eachjquery.min.js:2 e.fn.e.eachjquery.min.js:2 $.fn.onePageNav.onePageNav.getPositionsjquery.nav.js:62 $.fn.onePageNav.onePageNav.initjquery.nav.js:102 $.fn.onePageNavjquery.nav.js:120 e.extend.eachjquery.min.js:2 e.fn.e.eachjquery.min.js:2 $.fn.onePageNavjquery.nav.js:116 (anonymous function) And error screenshot: http://cl.ly/2d3A0j15372p3e1G4325

davist11 commented 12 years ago

Do you have a link to your page?

saltcod commented 12 years ago

Hey Trevor,

Thanks for the reply.

The page is here for the moment: http://singlepage.waterstreetgm.org/index.htm

And a screen of what I see: http://cl.ly/1M1X072f0b0s473e1L2k

Thanks again!

Terry

On 2011-11-18, at 2:24 PM, Trevor Davis wrote:

Do you have a link to your page?

Reply to this email directly or view it on GitHub: https://github.com/davist11/jQuery-One-Page-Nav/issues/6#issuecomment-2792082

saltcod commented 12 years ago

It would probably help if I just gave you the exact code I'm working from: http://singlepage.waterstreetgm.org/onepage.zip

thanks again.

davist11 commented 12 years ago

Did you modify the plugin to start with? Line 65 of the plugin doesn't have the parenthesis, so not sure why yours does.

saltcod commented 12 years ago

Humm.weird. Even without the brackets I still get the error "Uncaught TypeError: Cannot read property 'top' of null"

I'm going to start from scratch with this. There must be something I've done accidentally.

Thanks

On 2011-11-18, at 9:16 PM, Trevor Davis wrote:

Did you modify the plugin to start with? Line 65 of the plugin doesn't have the parenthesis, so not sure why yours does.

Reply to this email directly or view it on GitHub: https://github.com/davist11/jQuery-One-Page-Nav/issues/6#issuecomment-2796490