-
Notifications
You must be signed in to change notification settings - Fork 158
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Timepicker #25
Comments
Love to know if there's one |
There are no plans for it at the moment.Since it's highly likely to need fundamental design changes, the possibility to have a plan in the near future is not so high, unfurtunately. |
Hi @mymth, did you consider making Timepicker a separate project instead? |
@stsrki, |
What about timepicker as part of datepicker? Same functionality as https://developer.mozilla.org/en-US/docs/Web/HTML/Element/input/datetime-local If anyone wants, I manage to cooked in the timepicker here https://github.com/newlogic-digital/ui/blob/main/src/scripts/Ui/Input.js#L102 it uses native type="time" input, can be used also with https://github.com/epavanello/input-time-polyfill |
I personally kinda like how the air datepicker is handling time selection: It also has an option to show only the timepicker (without the datepicker part): @mymth Would a solution like this also mean "fundamental design changes"? |
@xdev1 This is why I think "fundamental"
All dates given to date picker are converted to the time values of their local time 00:00:00.000 for internal use. This is to make these possible, to use Array.prototype.includes() on selected/disabled dates arrays and to compare dates with simple arithmetic comparison. Other hurdles for this library but not for Air Datepicker are... |
@mymth Thank you very much for the detailed information. |
this library is perfect until I realize I also need time instead just date, damn I wish I could still use this clean simple tiny vanilla js library @mymth for the second part I think you should not focus on it, let the community update their own locale |
Hey, love your vanillajs date picker!
Are there any plans to include a time picker?
The text was updated successfully, but these errors were encountered: