From ee87defcca81d7741758b620b85bd1e70d94cfae Mon Sep 17 00:00:00 2001 From: Scott González Date: Fri, 22 Jan 2010 04:13:36 +0000 Subject: Datepicker: Use null instead of empty string when showing/hiding without a duration. Fixes #5086 - no animation still animates. --- ui/jquery.ui.datepicker.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'ui') diff --git a/ui/jquery.ui.datepicker.js b/ui/jquery.ui.datepicker.js index a80ffba0d..45d514a33 100644 --- a/ui/jquery.ui.datepicker.js +++ b/ui/jquery.ui.datepicker.js @@ -644,7 +644,7 @@ $.extend(Datepicker.prototype, { if ($.effects && $.effects[showAnim]) inst.dpDiv.show(showAnim, $.datepicker._get(inst, 'showOptions'), duration, postProcess); else - inst.dpDiv[showAnim || 'show']((showAnim ? duration : ''), postProcess); + inst.dpDiv[showAnim || 'show']((showAnim ? duration : null), postProcess); if (!showAnim) postProcess(); if (inst.input[0].type != 'hidden') @@ -754,7 +754,7 @@ $.extend(Datepicker.prototype, { inst.dpDiv.hide(showAnim, $.datepicker._get(inst, 'showOptions'), duration, postProcess); else inst.dpDiv[(showAnim == 'slideDown' ? 'slideUp' : - (showAnim == 'fadeIn' ? 'fadeOut' : 'hide'))]((showAnim ? duration : ''), postProcess); + (showAnim == 'fadeIn' ? 'fadeOut' : 'hide'))]((showAnim ? duration : null), postProcess); if (!showAnim) postProcess(); var onClose = this._get(inst, 'onClose'); -- cgit v1.2.3