Skip to content

alignValue.toFixed is not a function (suggested fix included) #374

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

Open
nyndesigns opened this issue May 30, 2012 · 0 comments
Open

alignValue.toFixed is not a function (suggested fix included) #374

nyndesigns opened this issue May 30, 2012 · 0 comments

Comments

@nyndesigns
Copy link

I was testing out an implementation of the time picker with some new code -- when my firebug blew up with error messages (alignValue.toFixed is not a function) while sliding the minute field.

The site is using jQuery ui 1.8.17 and jQuery 1.6.4 -- running Firefox 12 on Ubuntu 12.04.

Anyways, after a quick search I came upon this thread (http://forum.jquery.com/topic/alignvalue-tofixed-is-not-a-function) in the jQuery forums. As suggested, I wrapped each value (value, min, max, step) within the slider function (line numbers: 469, 484, 496, 508, 768, 770, 772, 774) with parseFloat().. This seems to have corrected this issue.

I'm pretty busy atm.. so I haven't had a chance to look at an optimization (primarily, to see where adding parseFloat is necessary and where it is not), to test other versions / combinations of jQuery, jQuery UI, etc.... and to obviously push these suggested changes.. in the meantime, if someone else has a chance to look at this, can confirm it.. or just stumbles upon it from a web search.... great :)

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

No branches or pull requests

1 participant