Skip to content

Commit 3be567e

Browse files
Update comments for trentrichardson#644 fix
1 parent 953cd46 commit 3be567e

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

dist/jquery-ui-timepicker-addon.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -752,13 +752,13 @@
752752
// If the update was done in the input field, the input field should not be updated.
753753
// If the update was done using the sliders, update the input field.
754754
var hasChanged = (
755-
hour !== parseInt(this.hour,10) ||
755+
hour !== parseInt(this.hour,10) || // sliders should all be numeric
756756
minute !== parseInt(this.minute,10) ||
757757
second !== parseInt(this.second,10) ||
758758
millisec !== parseInt(this.millisec,10) ||
759759
microsec !== parseInt(this.microsec,10) ||
760760
(this.ampm.length > 0 && (hour < 12) !== ($.inArray(this.ampm.toUpperCase(), this.amNames) !== -1)) ||
761-
(this.timezone !== null && timezone !== this.timezone.toString())
761+
(this.timezone !== null && timezone !== this.timezone.toString()) // could be numeric or "EST" format, so use toString()
762762
);
763763

764764
if (hasChanged) {

src/jquery-ui-timepicker-addon.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -758,13 +758,13 @@
758758
// If the update was done in the input field, the input field should not be updated.
759759
// If the update was done using the sliders, update the input field.
760760
var hasChanged = (
761-
hour !== parseInt(this.hour,10) ||
761+
hour !== parseInt(this.hour,10) || // sliders should all be numeric
762762
minute !== parseInt(this.minute,10) ||
763763
second !== parseInt(this.second,10) ||
764764
millisec !== parseInt(this.millisec,10) ||
765765
microsec !== parseInt(this.microsec,10) ||
766766
(this.ampm.length > 0 && (hour < 12) !== ($.inArray(this.ampm.toUpperCase(), this.amNames) !== -1)) ||
767-
(this.timezone !== null && timezone !== this.timezone.toString())
767+
(this.timezone !== null && timezone !== this.timezone.toString()) // could be numeric or "EST" format, so use toString()
768768
);
769769

770770
if (hasChanged) {

0 commit comments

Comments
 (0)