search

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 502 forks source link

Requested animation frame should be cancelled #247

Closed b1rdex closed 10 years ago

b1rdex commented 10 years ago

The problem is described is #237.

Details: pane animation is behind content animation by 1 step. That's because you just do nothing if there is planned animation frame. But it was planned step before, so you get wrong animation.

kristerkari commented 10 years ago

Thanks! #246 has the same fix, so I think I'll merge that one once I have tested it out.

b1rdex commented 10 years ago

OK, it's your decision. But it's better to have 1 feature/bugfix in 1 PR.

kristerkari commented 10 years ago

@b1rdex I agree. Let's reopen this if something comes up with that PR.