nazar-pc / PickMeUp

Really simple, powerful, customizable and lightweight standalone datepicker
BSD Zero Clause License
615 stars 191 forks source link

default_date, date, #110

Closed gzhegow2all closed 8 years ago

gzhegow2all commented 8 years ago

Image https://i.gyazo.com/ba5368ad114be15d29704b1dff00997e.png

Please check, it's not working. When load i have my default value, when i click at the field the pickmeup is loading, and replace date to MIN, i feel it as ignoring date and default_date. if i set default_date to false, the date field is ignored, even if i remove default value and set "new Date()".

i checked when min is bigger than selected, and converselly - still not working

as i think, it may be works next:

nazar-pc commented 8 years ago

Can you prepare live demo, please? I would be easier to debug with minimal reliable reproducible example.

gzhegow2all commented 8 years ago

http://www.fakeltour.ru/widget/

2016-03-10 0:46 GMT+03:00 Nazar Mokrynskyi notifications@github.com:

Can you prepare live demo, please? I would be easier to debug with minimal reliable reproducible example.

— Reply to this email directly or view it on GitHub https://github.com/nazar-pc/PickMeUp/issues/110#issuecomment-194523270.

nazar-pc commented 8 years ago

Looks like there is some bug with date parsing when format doesn't have any separators. Here is kind of example I was asking for: https://jsfiddle.net/z4fmvuzb/199/

gzhegow2all commented 8 years ago

lol, my code creates correct Date string about 1 may, i give Date object to pickmeup

2016-03-10 0:53 GMT+03:00 Nazar Mokrynskyi notifications@github.com:

Looks like there is some bug with date parsing when format doesn't have any separators. Here is kind of example I was asking for: https://jsfiddle.net/z4fmvuzb/199/

— Reply to this email directly or view it on GitHub https://github.com/nazar-pc/PickMeUp/issues/110#issuecomment-194526113.

nazar-pc commented 8 years ago

As workaround you can open pickmeup manually on click, handle change event and set date when necessary. This is awkward, but it will help you to get job done until proper fix released.

gzhegow2all commented 8 years ago

ok i correct, still not working, date is dropping to today. http://take.ms/MkP7O

2016-03-10 0:53 GMT+03:00 Nazar Mokrynskyi notifications@github.com:

Looks like there is some bug with date parsing when format doesn't have any separators. Here is kind of example I was asking for: https://jsfiddle.net/z4fmvuzb/199/

— Reply to this email directly or view it on GitHub https://github.com/nazar-pc/PickMeUp/issues/110#issuecomment-194526113.

nazar-pc commented 8 years ago

Yes, this is a bug, use workaround instead

gzhegow2all commented 8 years ago

ok, thx for your attention.

nazar-pc commented 8 years ago

I don't think we'll support format without separators. For such cases I recommend to use events and manage things manually.