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

Two small fixes #9

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

Two small fixes #9

wants to merge 2 commits into from

Conversation

zefei
Copy link

@zefei zefei commented Jun 12, 2014

  • changed input type requirement from text to text or number (html5 input type)
    this doesn't change any behavior but it allows libraries such as angularjs to use the correct input type
  • sometimes the steps array misses the last value due to float point calculation error
    changed how steps are calculated by adding a small number to the end condition

zefei added 2 commits June 12, 2014 16:11
changed input type requirement from text to text or number (html5 input type)
sometimes the steps array misses the max value due to float point error
changed how steps are calculated by adding a small number to the end condition
@zefei zefei changed the title two small fixes Two small fixes Jun 12, 2014
@startswithaj
Copy link

+1

3 similar comments
@slobdell
Copy link

+1

@sja
Copy link

sja commented Apr 6, 2015

+1

@RicardoGeek
Copy link

+1

@abpetkov
Copy link
Owner

abpetkov commented Nov 4, 2016

@zefei Can you provide a specific example of the steps issue?

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.

6 participants