search

fgelinas / timepicker

A jQuery UI Time Picker
http://fgelinas.com/code/timepicker
GNU General Public License v2.0
266 stars 84 forks source link

Date() support added for setTime, getTimeAsDate added #53

Closed AAverin closed 11 years ago

AAverin commented 11 years ago

setTime can now have Date() object as an argument getTimeAsDate method was added to retrieve time as date inst.minuts renamed to inst.minutes

and btw, why not use underlying Date object to store all of the time settings?

fgelinas commented 11 years ago

Thanks for the pull request, I'll check it later. For the Date object question, at first I did not think of it and if I was to write the plugin again, I would use it. But using the Date object has some limitations too like when only minutes or hours are selected. Also the timepicker is not limited to valid minutes / hours so one can set options to enter minutes ranging from 1 to 100.