Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix for mousewheel scrolling bug with regards to decimal increments #309

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

andrewleek
Copy link

Here are a pair of fixes for mousewheel scrolling when a knob is set to use decimal increments (it appears the issue doesnt exists if it is set to the default or whole number increments). The first commit fixes an issue where NaN was being generated. The second commit is more an esthetic fix so that numbers like 2.00000000001 are not generated when the desired output would be something like 2.0 or 2.1 for example.

fixes and issue where s.o.step was being treated as a string instead of a number
This fixes an issue where a decimal being incremented by .1 for example would manifest itself as 1.10000000005 and display as such in the text field of the knob - mostly and esthetic issue
Added on change notifications to mousewheel scrolling and arrow key interactions.  Also minified.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant