Skip to content
Permalink
Browse files

Datepicker: Added onSelect.apply() call to _doKeyDown method when ent…

…er button pushed if onSelect is defined. Fixed #7124 - Datepicker: onSelect does not fire when enter button is pushed
  • Loading branch information...
marcneuwirth authored and gnarf committed Jun 28, 2011
1 parent 5c34cea commit a56aa861b1deb6d71ec74a77647d12d50570ea57
Showing with 12 additions and 0 deletions.
  1. +5 −0 tests/unit/datepicker/datepicker_events.js
  2. +7 −0 ui/jquery.ui.datepicker.js
@@ -41,6 +41,11 @@ test('events', function() {
simulate('keydown', {keyCode: $.simulate.VK_ESC});
equals(selectedDate, $.datepicker.formatDate('mm/dd/yy', date),
'Callback selected date - esc');
var dateStr = '02/04/2008';
inp.val(dateStr).datepicker('show').
simulate('keydown', {keyCode: $.simulate.VK_ENTER});
equals(dateStr, selectedDate,
'onSelect is called after enter keydown');
// onChangeMonthYear
inp.datepicker('option', {onChangeMonthYear: callback2, onSelect: null}).
val('').datepicker('show');
@@ -529,6 +529,13 @@ $.extend(Datepicker.prototype, {
$.datepicker._currentClass + ')', inst.dpDiv);
if (sel[0])
$.datepicker._selectDay(event.target, inst.selectedMonth, inst.selectedYear, sel[0]);
var onSelect = $.datepicker._get(inst, 'onSelect');
if (onSelect) {
var dateStr = $.datepicker._formatDate(inst);

// trigger custom callback
onSelect.apply((inst.input ? inst.input[0] : null), [dateStr, inst]);
}
else
$.datepicker._hideDatepicker();
return false; // don't submit the form

0 comments on commit a56aa86

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