Add comments as to why we should stop propagating the keyup event

1 files changed, 4 insertions, 4 deletions

diff --git a/content_scripts/vimium_frontend.coffee b/content_scripts/vimium_frontend.coffee
index b66a7990..c7e5f2ef 100644
--- a/content_scripts/vimium_frontend.coffee
+++ b/content_scripts/vimium_frontend.coffee
@@ -433,10 +433,10 @@ onKeydown = (event) ->
       isValidFirstKey(KeyboardUtils.getKeyChar(event))))
     event.stopPropagation()
 
-onKeyup = (event) -> 
-  return unless handlerStack.bubbleEvent('keyup', event)
-  if (!isInsertMode())
-    event.stopPropagation();
+onKeyup = (event) ->
+  return unless handlerStack.bubbleEvent("keyup", event)
+  # Don't propagate the keyup to the underlying page, since Vimium has handled it. See #733.
+  event.stopPropagation() unless isInsertMode()
 
 checkIfEnabledForUrl = ->
   url = window.location.toString()