Force our key event handlers to have the highest possible priority

* The `window` object receives key events before the `document` object,
and so any event listeners on `window` get priority. This commit
switches from binding `keydown`, `keypress`, `keyup` on `document` to
on `window`.
* We were using `event.stopPropagation()` to prevent other event
listeners from firing if we had handled an event. This stopped the event
from propagating to other elements/objects and triggering their event
listeners, but didn't block event listeners registered on the same
object as ours. Switching to `event.stopImmediatePropagation()` ensures
that our event listener is the last to run for the event.

Fixing these issues allows Vimium to regain control over key events in
Google Groups (eg. the [vimium-dev
group](https://groups.google.com/forum/vimium-dev)).

1 files changed, 1 insertions, 1 deletions

diff --git a/lib/dom_utils.coffee b/lib/dom_utils.coffee
index dcdd5518..8c0445c5 100644
--- a/lib/dom_utils.coffee
+++ b/lib/dom_utils.coffee
@@ -129,7 +129,7 @@ DomUtils =
 
   suppressEvent: (event) ->
     event.preventDefault()
-    event.stopPropagation()
+    event.stopImmediatePropagation()
 
 root = exports ? window
 root.DomUtils = DomUtils