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

paper-calender _tapDay issue #203

Open
cvPriyanka opened this issue Oct 31, 2017 · 2 comments
Open

paper-calender _tapDay issue #203

cvPriyanka opened this issue Oct 31, 2017 · 2 comments

Comments

@cvPriyanka
Copy link

cvPriyanka commented Oct 31, 2017

Hi,

Suppose the previous date is Oct 31 and we have to set the new date to Nov 1.

    var newDate = new Date(this.date.getTime());
    newDate.setYear(item.year);
    newDate.setMonth(item.month);
    newDate.setDate(item.day);

Here while setting month to Nov, since Nov 31st doesn't exist, it is set to Dec 1.

This can be avoided by setting month as:

newDate.setMonth(item.month, item.day);

I am not sure if its the right way. Please help

@admwx7
Copy link
Collaborator

admwx7 commented Nov 1, 2017

@cvPriyanka how are you triggering this case in the UI? The calendar element shouldn't allow you to select a date that's not actually valid, however if it is and you have steps to reproduce I can debug it and address the issue. Just don't want to make changes if I can't test that I actually fixed it...

@cvPriyanka
Copy link
Author

@admwx7 Select Oct 31st and save it. Then reopen paper-date-picker element , now the current value is Oct 31. Try to select Nov 1

@admwx7 admwx7 modified the milestones: 2.0.0, 2.0.1 Nov 2, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants