search

inc2734 / smart-custom-fields

78 stars 33 forks source link

feat: Implement datetime picker with flatpickr #73

Closed mirucon closed 6 years ago

mirucon commented 6 years ago

Hi there,

Thank you for developing this awesome plugin, I love this :) Now I think that having datetime picker would be great as well as a date picker, so I've added it in this PR. Since jQuery UI does not support datetime picker, I've implemented this with flatpickr and native JavaScript way. Although we should actually use NPM and module bundler, let's go with CDN for now. I would be sending another PR to use them if I could have time.

mirucon commented 6 years ago

あータイムゾーン周りのこと完全に忘れてたのでマージちょっと待ってください。

inc2734 commented 6 years ago

おーありがとうございます！待ちます！

mirucon commented 6 years ago

@inc2734 あーだめだ、いい具合のタイムゾーンの解決法が思い浮かばない… 今回使った flatpickr 側でタイムゾーンのサポート特にないっぽくて自前でどうにかしないといけないのですけど、このプラグインの構成的に完全にフロントとサーバーサイドがくっついてる構成になってるから、サーバー側で保存のタイミングでタイムゾーンハンドルすることになると思うんだけど、ちょっと保存側のコード読みきれてないので、一旦タイムゾーンの件は保留でお願いします…。 これ一旦マージしてもらっても良いですし (そしたらタイムゾーンの件は Issue 化しておく)、解決するまで待っても (ただいつになるか分からないけど…) 良いので好きにして下さい ! とりあえず1タイムゾーンにしか対応してないよっていうのは注釈入れときます.