Skip to content

Dev #379

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

Closed
wants to merge 39 commits into from
Closed

Dev #379

wants to merge 39 commits into from

Conversation

kdeng
Copy link
Contributor

@kdeng kdeng commented Jun 26, 2012

Fixed a bug when the input field is empty, and the date picker will never update it with selected date.

trentrichardson and others added 30 commits February 28, 2012 14:42
Fixed errors when call public methods from input without datepicker instance
Fixed typo for iso8601
Add $.datepicker.parseDateTime(), $.datepicker.parseTime() functions
Pick the current timezone of the browser by default.
* New config field 'defaulTimezone'.
…ings).

* Only do this, when using the localTimezone.
Use defaultTimezone, when a time is already set.
Fix : Input date is not selected
…al timezone.

* This switches the timezone automatically, when changing the date to a (non) daylight saving time.
Make sure useLocalTimezone variable actually works
@trentrichardson
Copy link
Owner

Hey kdeng, Thanks for the code, but this pull request will need to be issued from the "dev" branch

@kdeng
Copy link
Contributor Author

kdeng commented Jun 27, 2012

Hi Trent, Thanks for your comment. I am new for using github, even I own my account for a long time.So, now what should I do to submit my changes? do I need to create another fork, pull the codes from DEV branch, and do the changes, and then push my changes to the DEV branch of the new fork, and then pull request again? Many thanks

@kdeng
Copy link
Contributor Author

kdeng commented Jun 27, 2012

I have created new fork, and done the changes again. A new pull request has been submitted to the dev branch. Hopefully, it works. Thanks

@kdeng kdeng closed this Jun 27, 2012
@kdeng
Copy link
Contributor Author

kdeng commented Jun 27, 2012

you can ignore this one.

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.

8 participants