Skip to content
Permalink
Browse files

Datepicker: Use null instead of empty string when showing/hiding with…

…out a duration.

Fixes #5086 - no animation still animates.
  • Loading branch information
scottgonzalez committed Jan 22, 2010
1 parent 1e95c7f commit ee87defcca81d7741758b620b85bd1e70d94cfae
Showing with 2 additions and 2 deletions.
  1. +2 −2 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');

0 comments on commit ee87def

Please sign in to comment.
You can’t perform that action at this time.