0x2620/oxjs
2
0
Fork 1
Code Issues 172 Pull requests Releases Wiki Activity

fix a bug in Ox.Input that would make its change event fire more than once

Browse source
This commit is contained in:
rlx 2012-01-13 16:28:01 +05:30
parent 923ed92bd7
commit 8de69b3173
1 changed files with 13 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

14
source/Ox.UI/js/Form/Ox.Input.js
View file

@ -596,6 +596,15 @@ Ox.Input = function(options, self) {
self.options.placeholder && setPlaceholder(); self.options.placeholder && setPlaceholder();
self.options.validate && validate(); self.options.validate && validate();
self.bindKeyboard && Ox.UI.$document.unbind('keydown', keydown); self.bindKeyboard && Ox.UI.$document.unbind('keydown', keydown);
if (!self.cancelled && !self.submitted) {
that.triggerEvent('blur', {
value: self.options.value
});
self.options.value !== self.originalValue && that.triggerEvent('change', {
value: self.options.value
});
}
/*
self.options.value !== self.originalValue && that.triggerEvent('change', { self.options.value !== self.originalValue && that.triggerEvent('change', {
value: self.options.value value: self.options.value
}); });
@ -604,17 +613,20 @@ Ox.Input = function(options, self) {
!self.cancelled && !self.submitted && that.triggerEvent('blur', { !self.cancelled && !self.submitted && that.triggerEvent('blur', {
value: self.options.value value: self.options.value
}); });
*/
} }
function cancel() { function cancel() {
self.cancelled = true; self.cancelled = true;
self.$input.blur().val(self.originalValue); self.$input.val(self.originalValue).blur();
self.cancelled = false; self.cancelled = false;
that.triggerEvent('cancel'); that.triggerEvent('cancel');
} }
function change() { function change() {
// change gets invoked before blur
self.options.value = self.$input.val(); self.options.value = self.$input.val();
self.originalValue = self.options.value;
that.triggerEvent('change', { that.triggerEvent('change', {
value: self.options.value value: self.options.value
}); });