search

mugifly / jquery-simple-datetimepicker

Date & time picker for jQuery, It's simple & clean.
http://mugifly.github.com/jquery-simple-datetimepicker
Other
260 stars 182 forks source link

Changing months should not update the picked date #162

Closed royzrj closed 8 years ago

royzrj commented 8 years ago

As described in issue #151 when a user changes the viewed month the picked date is also changed, which on environments where a month change triggers an AJAX call could cause severe problems. This PR fixes that, by storing another variable ('shownDate') into $picker.data array, and using the 'keepPickedDate' option when drawing the calendar after month-changing trigger is activated.

royzrj commented 8 years ago

@mugifly I'm not sure how to proceed now, as the Travis run failed but I think the tests should be done in some other way

mugifly commented 8 years ago

I got it. I will repair the test code.

mugifly commented 8 years ago

I merged it now. Thank you!