-
Notifications
You must be signed in to change notification settings - Fork 1k
Date not read from field when using jQuery UI v1.8.14 #197
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
Comments
I've been fighting this issue all day. Great timing! I thought I was going crazy. No amount of fidgeting with datepicker options seems to get around this issue. Worth noting, perhaps, that setting a defaultDate to Still an issue on jQuery 1.6.2, as well (jmabey's test case uses 1.6.1 it looks like). |
I just put in a pull request for a fix. I'm very, very new to jQuery and github, so I'm not sure I've followed all the proper procedures to get the fix in. FWIW, here is what I requested be added: //####################################################################################### |
Patch does not help to me :) Date does not get recognized correctly. |
What is an example date that is not recognized?
|
jquery: 1.5.1 Datetimepicker settings: dateFormat: %Y/%m/%d Steps to reproduce:
|
I reproduced it in jsfiddle and it seems the patch indeed fixes the problem. Thanks! |
There is validation in _parseDate which throw exception and set current date to 0. To override it it is necessary to send only date to the '_parseDate'. I'll provide patch in an hour. |
If you click on a field that already holds a date/time value, the date/time picker displays today's date instead of the date in the field's value. The time is read correctly. This only happens when using jQuery UI v1.8.14. Tested in Chrome 12 and Firefox 5.
Test case: http://jsfiddle.net/jmabey/zrhfd/
Notice the date is set to 01/02/2012, but when you click the field, today's date is selected.
The text was updated successfully, but these errors were encountered: