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

2-digit hour #235

Open
wants to merge 1 commit into
base: gh-pages
Choose a base branch
from
Open

2-digit hour #235

wants to merge 1 commit into from

Conversation

narcisbodea
Copy link

I tried to use bootstrap-timepicker with Masked Input Plugin but I had no success because the hour should be returned always with two digits, otherwise it cannot be validated by mask input, so the value it is canceled. All the time when the user clicks the input, the value it is reset to empty and has a strange behavior. I did not find any settings in the timepicker to force to return always 2 digits for the hour. So I decided to make changes in the code.
This commit contains the change, a new option "hourTwoDigits" in the settings, which is default set to false, to not affect the behavior for those who already implemented timepicker.
$('#timepicker').timepicker({
hourTwoDigits:true
});

I tried to use bootstrap-timepicker  with Masked Input Plugin but I had no success because the hour should be return always wit two digits, otherwise cannot be validet by mask input, so the value it is canceled. All the time when the user clicks the input, the value it is reset to empty and has a strange behaviour. I did not find any settings in timepicker to force to return always 2 digits for the hour. So I decided to make changes in the code. 
I this commit are the changes with a new option "hourTwoDigits" which is default set to false, to not affect the behaviour for those who already imlpemented timepicker.
 $('#timepicker5').timepicker({
        hourTwoDigits:true
});
@mrhota mrhota changed the title Update bootstrap-timepicker.js 2-digit hour Aug 10, 2015
@mrhota
Copy link
Collaborator

mrhota commented Aug 10, 2015

Related to #203

@lusever
Copy link

lusever commented Aug 29, 2015

Leading zero notation is used to indicate 24 hour time. It may be necessary to use a flag for showMeridian?

@mrhota
Copy link
Collaborator

mrhota commented Aug 29, 2015

Some 24-hour locales zero-pad the hour, others don't. Time libraries like Java and Moment allow you to specify. See Moment's docs:
http://momentjs.com/docs/#/displaying/format/

I want to allow the user to provide a localization config object which will contain the time format. This will be part of the larger i18n effort I'm currently planning.

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

Successfully merging this pull request may close these issues.

3 participants