Fullpage.js: "getPaddings is not defined" in scrolloverflow.js after resize

[arthurdewolf]

I'm using fullPage (2.9.6) with scrollOverflow (5.2.0) and am getting the error getPaddings is not defined after resizing the window.

I was able to fix it by moving the getPaddings function out of and above the scrollBarHandler function.

[alvarotrigo]

I'm using fullPage (2.9.6) with scrollOverflow (5.2.0) and am getting the error getPaddings is not defined after resizing the window.

The issue was fixed here thanks to @Odrin that detected it.

Make sure to use the unminified version of the dev branch. (you can minifiy it yourself if you wish, just not updated in the dev branch)

[AbdulkaderTahhan]

I'm take the fixed file and update it successfully, but this error still exist i don't know what can i do for that,
thanks a lot

[alvarotrigo]

@AbdulkaderTahhan you'll have to provide an isolated reproduction of it in jsfiddle or codepen to prove it.

[AbdulkaderTahhan]

Here the example for this issue http://websitk.com/public/main/,
i use this file https://goo.gl/mtGNBk but when click F12 on the browser you will find the issue

[alvarotrigo]

@AbdulkaderTahhan your reproduction is not isolated. So it is not valid for me to check.
And, as I said, you should be using the file from the dev folder.

[petebarr]

Greetings!

Yeah, unfortunately, I'm getting the same issue when using the un-minified/minified cdn version. This example is pretty isolated: https://codepen.io/petebarr/pen/9dfa27828a12af469bf67da410346d09

Cheers!

[alvarotrigo]

@petebarr see my comment above yours. You should be using the dev version as the change was not yet merged into any public release. You are not using the dev version. Use the link i posted.

[petebarr]

May sound like a dumb question but is that the dev version of fullscreen.js or scrolloverflow.js?
Thanks again!

[alvarotrigo]

Use the latest of both. You'll get less bugs this way ;)

[petebarr]

Thanks, that works in Codepen now using the following URLs in "Add External Scripts/Pens":

https://rawgit.com/alvarotrigo/fullPage.js/dev/vendors/scrolloverflow.js
https://rawgit.com/alvarotrigo/fullPage.js/dev/dist/jquery.fullpage.js

Appreciate it!

[mexn]

Hey,

are there any plans, when this fix will be available over npm?

[alvarotrigo]

@mexn soon! In the next release. Hopefully by the end of this week!

[mexn]

That's great news, thanks!

[alvarotrigo]

@mexn I've just published the new release 2.9.7!

[hortonelectric]

@alvarotrigo This isn't fixed in 2.9.7 for me. However if I use the code from the dev branch IE https://rawgit.com/alvarotrigo/fullPage.js/dev/dist/jquery.fullpage.js then it works fine.

Uncaught ReferenceError: getPaddings is not defined
    at Object.update (scrolloverflow.js:2736)
    at n (scrolloverflow.js:2490)
    at HTMLDivElement.<anonymous> (scrolloverflow.js:2524)
    at Function.each (jquery.min.js:2)
    at w.fn.init.each (jquery.min.js:2)
    at r (scrolloverflow.js:2516)
    at s [as createScrollBarForAll] (scrolloverflow.js:2438)
    at Ke (jquery.fullpage.js:466)
    at jquery.fullpage.js:2151
update @ scrolloverflow.js:2736
n @ scrolloverflow.js:2490
(anonymous) @ scrolloverflow.js:2524
each @ jquery.min.js:2
each @ jquery.min.js:2
r @ scrolloverflow.js:2516
s @ scrolloverflow.js:2438
Ke @ jquery.fullpage.js:466
(anonymous) @ jquery.fullpage.js:2151
setTimeout (async)
Mo @ jquery.fullpage.js:2150
dispatch @ jquery.min.js:2
y.handle @ jquery.min.js:2

[alvarotrigo]

@hortonelectric please provide an isolated reproduction. Working as expected on the demo page.

[alvarotrigo]

@hortonelectric ok, I've noticed I forgot to update the minified version. The unminified one should work as expected. Merging the minified one now, but you won't get it on npm. So please get it from here until the next release gets pushed to npm. https://github.com/alvarotrigo/fullPage.js/pull/3192