search

PitPik / tinyDatePicker

NEW: Tiny javascript and jQuery date / time picker [datepicker]
http://www.dematte.at/tinyDatePicker/
MIT License
57 stars 14 forks source link

Using 24 hours time gives too many hours #8

Closed bgiaccio closed 7 years ago

bgiaccio commented 7 years ago

When using 24 hour time format the drop down shows 0 - 24 meaning I can select 24:30 as a time. Trying to turn that into a date object will fail. The drop down should stop at 23

I believe you want to change line 288 of datePicker.js to be 23 vice 24.

PitPik commented 7 years ago

Hey @bgiaccio , thanks for this finding (and sorry for the late response). I just updated. Cheers

umbundu commented 7 years ago

If you have a datepicker on the page body, it works fine. If you have it inside a tab control nothing happens.