jtsage / jtsage-datebox

A multi-mode date and time picker for Bootstrap (3&4), jQueryMobile, Foundation, Bulma, FomanticUI, and UIKit (or others)
http://datebox.jtsage.dev/
Other
474 stars 166 forks source link

Default value for time #239

Closed erikverheij closed 11 years ago

erikverheij commented 11 years ago

Hi.. Thank you very much for this wonderfull library.

I encountered a problem by using the DefaulValue for a timebox.

The documentation says you can provide a string like "13:42". But "_makeDate -> case: string" expects 3 something like 13:42:00.

Maybe you can default the third argument to 0 if it is not set.

jtsage commented 11 years ago

Whoops - 2 element got left out somehow, yes, it expects 3. Also note that you can use the array method for times too ([13,42,0])

Fixed in just a second.