Adjust timer use to alleviate flicker on scroll up #20
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I reworked the timer logic to have one timer running constantly running against the physics state. I believe that doing so reduces overheads/delays caused by repeatedly scheduling callbacks as in the original code. To keep the timer overhead light, the callback function is modified to short-circuit in cases where scrolling is not expected to take place.
On my machine, this resolves issue #17 (bad flickering scrolling up), which I experienced with vim in xterm ($TERM set to xterm-256color). I'd be interested to see feedback from others.
By the way, thanks for the cool plugin!