Skip to content
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

calendar displays wrong days with minDate #126

Closed
edshadi opened this issue Jun 12, 2015 · 6 comments · Fixed by #277
Closed

calendar displays wrong days with minDate #126

edshadi opened this issue Jun 12, 2015 · 6 comments · Fixed by #277
Labels

Comments

@edshadi
Copy link

edshadi commented Jun 12, 2015

I have a DatePicker that looks like this:

<DatePicker
        key="somekey"
        selected={moment()}
        onChange={this.setDeterminationDate}
        minDate={moment().subtract(10, 'days')}
        maxDate={moment().add(10, 'days')}
        placeholderText="Determination Date" />

this causes the calendar to display the wrong dates like this

Friday is the 5th not Saturday.

@skurger
Copy link

skurger commented Jun 29, 2015

I'm getting "Request has expired" from the link you provided which I think was a screenshot.

I looked at the demo which has an example using minDate but the dates are correct. Can you give some more info on how to reproduce?

@martijnrusschen
Copy link
Member

Is this still an issue? If so, can you post a POC?

@edshadi
Copy link
Author

edshadi commented Jul 29, 2015

I can't reproduce it now, I will reopen an issue if/when I do.

@edshadi edshadi closed this as completed Jul 29, 2015
@morgante
Copy link

I'm able to reproduce this using these options:

<DatePicker minDate={moment().add(1, 'day')} />

Initially, the calendar appears fine. But once I select a date (ex. November 1), it appears incorrectly, with November 1 occurring on a Monday.

@GalacticWeb
Copy link

+1 !

@edshadi
Copy link
Author

edshadi commented Nov 6, 2015

@morgante thanks for reproducing this, I'm not using min/maxDate anymore so I've moved passed this issue. The issue you're having is the same as this original one for sure.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants