Skip to content
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_separator option #5

Open
kakoni opened this issue Apr 23, 2013 · 2 comments
Open

time_separator option #5

kakoni opened this issue Apr 23, 2013 · 2 comments

Comments

@kakoni
Copy link

kakoni commented Apr 23, 2013

Perhaps you should mention the importance of time_seperator option. Otherwise there is this extra colon outside the select field (wow, did I spent some time finding this;)

@excid3
Copy link
Owner

excid3 commented Apr 23, 2013

Ha! I noticed that a while back but totally forgot about it. Good idea.

I almost feel like making a new time selector method that automatically sets some of these options for you. Would that make more sense?

@kakoni
Copy link
Author

kakoni commented Apr 24, 2013

Well since you're thinking about it;)->
Could you also make am/pm vs 24hour option?
And yes, setting time_seperator automatically/completely hiding it makes sense

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants