diff options
author | Vladislav Zarakovsky <vlad.zar@gmail.com> | 2016-01-25 11:52:09 +0300 |
---|---|---|
committer | Vladislav Zarakovsky <vlad.zar@gmail.com> | 2016-01-25 11:52:09 +0300 |
commit | 7560f0d33af67a64dca8c3c4fe1ebd1cd0537823 (patch) | |
tree | 001555b1ce14ea0f0c1b3bfc4dad9834bf49bf72 | |
parent | 481eb6acd950de35e2f40a16cd7ae67f94ef082b (diff) | |
download | awesomplete-7560f0d33af67a64dca8c3c4fe1ebd1cd0537823.zip awesomplete-7560f0d33af67a64dca8c3c4fe1ebd1cd0537823.tar.gz awesomplete-7560f0d33af67a64dca8c3c4fe1ebd1cd0537823.tar.bz2 |
Use dispatchEvent return value to get defaultPrevented
Standard `element.dispatchEvent` already provides infromation of
wether event was prevented or not. From MDN:
The return value is false if at least one of the event handlers
which handled this event called Event.preventDefault(). Otherwise
it returns true.
Instead of implementing our own `preventDefault()` use existing.
-rw-r--r-- | awesomplete.js | 11 |
1 files changed, 3 insertions, 8 deletions
diff --git a/awesomplete.js b/awesomplete.js index 2be360d..b4bf9af 100644 --- a/awesomplete.js +++ b/awesomplete.js @@ -195,17 +195,12 @@ _.prototype = { selected = selected || this.ul.children[this.index]; if (selected) { - var prevented; - - $.fire(this.input, "awesomplete-select", { + var allowed = $.fire(this.input, "awesomplete-select", { text: selected.textContent, - preventDefault: function () { - prevented = true; - }, origin: origin || selected }); - if (!prevented) { + if (allowed) { this.replace(selected.textContent); this.close(); $.fire(this.input, "awesomplete-selectcomplete"); @@ -349,7 +344,7 @@ $.fire = function(target, type, properties) { evt[j] = properties[j]; } - target.dispatchEvent(evt); + return target.dispatchEvent(evt); }; $.regExpEscape = function (s) { |