From 8c94d354dffd468553bc84fb4a51114288984877 Mon Sep 17 00:00:00 2001 From: Espen Antonsen Date: Wed, 13 Apr 2011 17:46:44 +0800 Subject: [PATCH 1/3] added setting for onBeforeHide event --- jquery.timePicker.js | 1 + 1 file changed, 1 insertion(+) diff --git a/jquery.timePicker.js b/jquery.timePicker.js index 41b35c1..65f765b 100644 --- a/jquery.timePicker.js +++ b/jquery.timePicker.js @@ -227,6 +227,7 @@ elm.focus(); } // Hide picker + settings.onBeforeHide() || $.noop(); $tpDiv.hide(); } From 23824fcd7fc7a3c169f891cb9976c39584da6bba Mon Sep 17 00:00:00 2001 From: Espen Antonsen Date: Wed, 13 Apr 2011 17:53:04 +0800 Subject: [PATCH 2/3] added setting for onBeforeShow event --- jquery.timePicker.js | 1 + 1 file changed, 1 insertion(+) diff --git a/jquery.timePicker.js b/jquery.timePicker.js index 65f765b..dd438d3 100644 --- a/jquery.timePicker.js +++ b/jquery.timePicker.js @@ -88,6 +88,7 @@ if ($tpDiv.is(":visible")) { return false; } + settings.onBeforeShow() || $.noop(); $("li", $tpDiv).removeClass(selectedClass); // Position From 30d780959a5837d1c62587f776f233cf7e39693c Mon Sep 17 00:00:00 2001 From: Espen Antonsen Date: Sun, 17 Apr 2011 21:56:38 +0800 Subject: [PATCH 3/3] helper function for hidePicker() --- jquery.timePicker.js | 16 ++++++++++------ 1 file changed, 10 insertions(+), 6 deletions(-) diff --git a/jquery.timePicker.js b/jquery.timePicker.js index dd438d3..8d6a187 100644 --- a/jquery.timePicker.js +++ b/jquery.timePicker.js @@ -121,7 +121,7 @@ // Hide timepicker on blur $(elm).blur(function() { if (!tpOver) { - $tpDiv.hide(); + $.timePicker(this).hidePicker(); } }); // Keypress doesn't repeat on Safari for non-text keys. @@ -183,7 +183,7 @@ return false; break; case 27: // Esc - $tpDiv.hide(); + hidePicker(); return false; break; } @@ -204,6 +204,12 @@ // Trigger element's change events. $(elm).change(); }; + + this.hidePicker = function() { + // Hide picker + settings.onBeforeHide() || $.noop(); + $tpDiv.hide(); + } }; // End fn; @@ -227,9 +233,7 @@ if (!$.browser.msie) { elm.focus(); } - // Hide picker - settings.onBeforeHide() || $.noop(); - $tpDiv.hide(); + $.timePicker(elm).hidePicker(); } function formatTime(time, settings) { @@ -276,4 +280,4 @@ return time; } -})(jQuery); +})(jQuery); \ No newline at end of file