This repository has been archived by the owner on May 29, 2019. It is now read-only.
Update datepicker.js #2932
Closed
Update datepicker.js #2932
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Now the model is a date object so when the inline datepicker is used, the date actually gets formatted. Saving millions of people tons of time and hassle with this fix.
|
I'm guessing this is trying to resolve issues with AngularJS 1.3 ? I'm not sure what this is trying to fix. Is there particular issue number this is trying to address? Working on AngularJS 1.3 compatibility is going to require quite an amount of work, feel free to tackle it and make the tests pass. |
shwei
added a commit
to shwei/bootstrap
that referenced
this pull request
Dec 10, 2014
Add AngularJS 1.3 compatibility - datepicker display issue reported in angular-ui#2932 and angular-ui#3047.
|
Add a truthy test: |
|
#2943 contains a slightly better fix -- handles the case when the value is |
|
This commit should have fixed it: 5f9afe5 see the working plunkr: http://plnkr.co/edit/0EEK4JpvW7GXqbhAgVwJ?p=preview regrads |
DKnodel
pushed a commit
to DKnodel/bootstrap
that referenced
this pull request
Apr 9, 2015
…-ui#2659 Closes angular-ui#3293 Closes angular-ui#3279 Closes angular-ui#2440 Closes angular-ui#2932 Closes angular-ui#3074 Closes angular-ui#2943 Closes angular-ui#2733 Fixes angular-ui#3047 Fixes angular-ui#2659 Fixes angular-ui#2681
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Now the model is a date object so when the inline datepicker is used, the date actually gets formatted. Saving millions of people tons of time and hassle with this fix.