jamesflorentino / nanoScrollerJS

A jQuery plugin that offers a simplistic way of implementing Lion OS scrollbars.
http://jamesflorentino.github.com/nanoScrollerJS/
MIT License
2.69k stars 503 forks source link

upward scrolling from end tiggers "scrollend" event #224

Open chipat opened 10 years ago

chipat commented 10 years ago

In some cases the contentScrollTop > maxScrollTop (by 1px) and therefore (_this.prevScrollTop > _this.maxScrollTop) when scrolling all the way to bottom.

Changing line 517 to if (_this.prevScrollTop < _this.maxScrollTop) { _this.$el.trigger('scrollend'); }

seems to fix the issue.

kristerkari commented 10 years ago

Thanks for reporting, sounds like a bug yeah.

kristerkari commented 10 years ago

@chipat Do you have an example of this happening? I don't really know how to reproduce this bug.