-
Notifications
You must be signed in to change notification settings - Fork 415
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
Time picker - better UX + a11y #6387
Comments
Yeah, my suggestion was to provide a time picker that is based on the combobox with a list of time presets in 15-minute increments, similar to how it works in Google Calendar. |
Will a combobox work for this purpose though? I think it may work OK if you can include all the times, but I'm sure you don't want to show 720 entries for every minute, and even w/ 5/15 minute steps you'd still have 288/96 individual items. (^ that part was written before the last comment from @foxbunny) I typically just type the time - but something like All that said, I personally never found it annoying/unpleasant to enter a time on Google Calendar, and considering how popular that tool is, maybe the same kind of behavior is good enough for us and our users. :) |
Since I don't think it makes much sense to start a longer discussion in the comment thread of #6371, let's use this issue instead.
My initial comment:
And @tomasr8's comment:
cc @foxbunny
The text was updated successfully, but these errors were encountered: