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

Fixed invalid negative input #14

Closed
wants to merge 3 commits into
base: bug-fixes
from

Conversation

Projects
None yet
2 participants
@Benzolitz

Benzolitz commented Sep 17, 2015

Currently the script only checkes if the input has a bigger length then maxValue. If you want to enter a negative Value which is longer than the maxValue, the script stops you from doing so. Only possibility was to use the mousewheel.

EmKayDK and others added some commits May 4, 2015

Fixed bug from Github (Issue 9).
Updated tests to include this.
Added Syn to the project instead of including it directly from Github.
Lucas Engel
Fixed invalid negative input
Script only checked if input has a bigger length then maxValue. If you wanted to enter a negative Value which is longer than the maxValue, the script stoped you from doing so. Only possibility was to use the mousewheel.
@Benzolitz

This comment has been minimized.

Show comment
Hide comment
@Benzolitz

Benzolitz Sep 19, 2015

$('#txtTesting').jStepper({minValue: -9999, maxValue: 20, overflowMode: 'ignore'});

I can write -9, but I can't add another 9. The script will stop me because the only check is if the Value.length is bigger than maxValue.length.

Benzolitz commented Sep 19, 2015

$('#txtTesting').jStepper({minValue: -9999, maxValue: 20, overflowMode: 'ignore'});

I can write -9, but I can't add another 9. The script will stop me because the only check is if the Value.length is bigger than maxValue.length.

@Benzolitz Benzolitz closed this Sep 19, 2015

@Benzolitz

This comment has been minimized.

Show comment
Hide comment
@Benzolitz

Benzolitz Sep 19, 2015

Didn't want to close :D

Benzolitz commented Sep 19, 2015

Didn't want to close :D

@Benzolitz Benzolitz reopened this Sep 19, 2015

EmKayDK added a commit that referenced this pull request Sep 19, 2015

@EmKayDK

This comment has been minimized.

Show comment
Hide comment
@EmKayDK

EmKayDK Sep 19, 2015

Owner

Fixed!
Thanks for the contribution :)

Owner

EmKayDK commented Sep 19, 2015

Fixed!
Thanks for the contribution :)

@EmKayDK EmKayDK closed this Sep 19, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment