Skip to content

Commit daa38f5

Browse files
author
Alexander Gitter
committed
Fix issue trentrichardson#177, keyboard input not working when datepicker wants us to
accept all characters
1 parent f80d8c1 commit daa38f5

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

jquery-ui-timepicker-addon.js

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -804,7 +804,7 @@ $.datepicker._updateDatepicker = function(inst) {
804804
};
805805

806806
//#######################################################################################
807-
// third bad hack :/ override datepicker so it allows spaces and colan in the input field
807+
// third bad hack :/ override datepicker so it allows spaces and colon in the input field
808808
//#######################################################################################
809809
$.datepicker._base_doKeyPress = $.datepicker._doKeyPress;
810810
$.datepicker._doKeyPress = function(event) {
@@ -814,6 +814,7 @@ $.datepicker._doKeyPress = function(event) {
814814
if (tp_inst) {
815815
if ($.datepicker._get(inst, 'constrainInput')) {
816816
var ampm = tp_inst._defaults.ampm,
817+
dateChars = $.datepicker._possibleChars($.datepicker._get(inst, 'dateFormat')),
817818
datetimeChars = tp_inst._defaults.timeFormat.toString()
818819
.replace(/[hms]/g, '')
819820
.replace(/TT/g, ampm ? 'APM' : '')
@@ -825,9 +826,9 @@ $.datepicker._doKeyPress = function(event) {
825826
" " +
826827
tp_inst._defaults.separator +
827828
tp_inst._defaults.timeSuffix +
828-
$.datepicker._possibleChars($.datepicker._get(inst, 'dateFormat')),
829+
dateChars,
829830
chr = String.fromCharCode(event.charCode === undefined ? event.keyCode : event.charCode);
830-
return event.ctrlKey || (chr < ' ' || !datetimeChars || datetimeChars.indexOf(chr) > -1);
831+
return event.ctrlKey || (chr < ' ' || !dateChars || datetimeChars.indexOf(chr) > -1);
831832
}
832833
}
833834

0 commit comments

Comments
 (0)