diff options
| author | Tobias Brunner <tobias@strongswan.org> | 2011-02-03 08:51:50 -0500 |
|---|---|---|
| committer | Scott González <scott.gonzalez@gmail.com> | 2011-02-03 08:51:50 -0500 |
| commit | 39cf7d5bb4f9dfe1ada5dc2499f5c91e536ec96b (patch) | |
| tree | 940bab699be137911c63746a9b8d66172f77885b | |
| parent | 8ac5da23159292225068c82caee99d5308e2be22 (diff) | |
| download | jquery-ui-39cf7d5bb4f9dfe1ada5dc2499f5c91e536ec96b.tar.gz jquery-ui-39cf7d5bb4f9dfe1ada5dc2499f5c91e536ec96b.zip | |
Datepicker: only set the focus explicitly if it is not already on the target field. Fixed #6694 – datepicker does not fire change event in IE8
jQuery stores the current content of a field whenever it gets the focus. Since the fixed function is also called for KeyUp events the change event is not triggered anymore when the input looses focus.
| -rw-r--r-- | ui/jquery.ui.datepicker.js | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/ui/jquery.ui.datepicker.js b/ui/jquery.ui.datepicker.js index 6b46dcfe2..018c6cf84 100644 --- a/ui/jquery.ui.datepicker.js +++ b/ui/jquery.ui.datepicker.js @@ -700,7 +700,9 @@ $.extend(Datepicker.prototype, { inst.dpDiv[(this._get(inst, 'isRTL') ? 'add' : 'remove') + 'Class']('ui-datepicker-rtl'); if (inst == $.datepicker._curInst && $.datepicker._datepickerShowing && inst.input && - inst.input.is(':visible') && !inst.input.is(':disabled')) + // #6694 - don't focus the input if it's already focused + // this breaks the change event in IE + inst.input.is(':visible') && !inst.input.is(':disabled') && inst.input[0] != document.activeElement) inst.input.focus(); // deffered render of the years select (to avoid flashes on Firefox) if( inst.yearshtml ){ |