When using disableBodyScroll, set stored scrollTo value to the initia…

[bwshen]

…l scrolled position, as well as upon subsequent calls to init.

Hi Gromo, didn't find any information on the best way to submit proposed changes. If you'd prefer that i open up an issue or something else instead let me know.

I have the use case where i programatically scroll to a position initially. however when i mousewheel with disableBodyScroll: true, the mousewheel, assumes that initial scroll is 0, instead of the actual value and scrolls there.

thxs!

[gromo]

How do you change scroll position? Do you use jQuery scrollTop/scrollLeft? Can you prepare example with wrong behavior on jsFiddle.net using original jquery.scrollbar.js file?

[bwshen]

here's a jsfiddle:
https://jsfiddle.net/tjqgxkd9/3/

When you attempt to use the mousewheel, if you mousewheel down, you are scrolled to the top, if you mousewheel up, it doesn't scroll.

[bwshen]

apologies, try https://jsfiddle.net/tjqgxkd9/5/

[gromo]

Pretty interesting. If you use mousewheel over scrollbar it works correctly. Idea was that disableBodyScroll handles mousewheel and trigger the same event for scrollbars. I will look at the code in next few days

Upd. fix should be like this: trigger mouseenter event on scrollbar and only then use mousewheel event

[bwshen]

something like this?

[gromo]

https://jsfiddle.net/tjqgxkd9/6/ - quick hack

[gromo]

But much better if you set scrollTop offset first and only then initialize scrollbar