From 99d784ff593378be41b5e6eb87867826fee9e921 Mon Sep 17 00:00:00 2001
From: Stephen Blott
Date: Mon, 22 Dec 2014 16:56:29 +0000
Subject: Add comment regarding .blur() on embeds.

Including embeds for .blur() etc. here is experimental.  It appears to
be the right thing to do for most common use cases.  However, it could
also cripple flash-based sites and games.  See discussion in #1211
and #1194.
---
 content_scripts/vimium_frontend.coffee | 3 +++
 1 file changed, 3 insertions(+)

(limited to 'content_scripts')

diff --git a/content_scripts/vimium_frontend.coffee b/content_scripts/vimium_frontend.coffee
index eacba682..071e87a6 100644
--- a/content_scripts/vimium_frontend.coffee
+++ b/content_scripts/vimium_frontend.coffee
@@ -458,6 +458,9 @@ onKeydown = (event) ->
     if isEditable(event.srcElement) or isEmbed(event.srcElement)
       # Remove focus so the user can't just get himself back into insert mode by typing in the same input
       # box.
+      # NOTE(smblott, 2014/12/22) Including embeds for .blur() etc. here is experimental.  It appears to be
+      # the right thing to do for most common use cases.  However, it could also cripple flash-based sites and
+      # games.  See discussion in #1211 and #1194.
       event.srcElement.blur()
     exitInsertMode()
     DomUtils.suppressEvent event
-- 
cgit v1.2.3