From dafa166364ff9a30b0cc2f0d160fcbcfc01cc61e Mon Sep 17 00:00:00 2001
From: mrmr1993
Date: Sun, 2 Nov 2014 05:39:07 +0000
Subject: 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)).
---
 lib/dom_utils.coffee | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

(limited to 'lib')

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
-- 
cgit v1.2.3