Datepicker: Date range demo bug when clearing date

[mattac]

I'd like to suggest an improvement to the date range limiting implementation listed here:

​http://jqueryui.com/demos/datepicker/#event-search

The current implementation has a bug. If a date is selected in one field and then cleared out, the date limiting will remain for the other field. Using an onClose handler rather than onSelect enables removing the min/maxDate setting on the other field.

See the following:

    onClose: function(selectedDate) {
        if (selectedDate) {
            var option = this.id == "from" ? "minDate" : "maxDate";
	    var instance = $(this).data("datepicker");
            var date = $.datepicker.parseDate(instance.settings.dateFormat || $.datepicker._defaults.dateFormat, selectedDate, instance.settings);
	    dates.not(this).datepicker("option", option, date);
        }
        else {
            dates.not(this).datepicker("option", option, null);
        }
    }

[Scott González]

Datepicker demo: Fixed date range restriction when clearing dates. Fixes #5473 - Datepicker: Date range demo bug when clearing date.

Changeset: a50516f77e82206f2d1f4188c9f5a242442ace09