From 2424b8ecc1eeb3b1728f7d4a30db77fccdc8fa05 Mon Sep 17 00:00:00 2001
From: mrmr1993
Date: Wed, 17 Dec 2014 14:39:52 +0000
Subject: Ensure cursor is hidden on scroll

---
 content_scripts/vimium_frontend.coffee | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

(limited to 'content_scripts/vimium_frontend.coffee')

diff --git a/content_scripts/vimium_frontend.coffee b/content_scripts/vimium_frontend.coffee
index 6dd244de..fcf5dac1 100644
--- a/content_scripts/vimium_frontend.coffee
+++ b/content_scripts/vimium_frontend.coffee
@@ -1129,7 +1129,7 @@ CursorHider =
     @cursorHideStyle = document.createElement("style")
     @cursorHideStyle.innerHTML = """
       body * {pointer-events: none !important; cursor: none !important;}
-      body {cursor: none !important;}
+      body, html {cursor: none !important;}
     """
     window.addEventListener "mousemove", @onMouseMove
     window.addEventListener "scroll", @onScroll
-- 
cgit v1.2.3


From 107cef3ecd0bdbcaba157b53cc3d20ff5e06b94f Mon Sep 17 00:00:00 2001
From: mrmr1993
Date: Wed, 17 Dec 2014 20:21:33 +0000
Subject: Disable cursor hiding for Chrome versions < 39.0.2171.71

There is a bug for earlier versions of Chrome which leaves
`pointer-events: none` stuck enabled for Google+ after a certain amount
of scrolling.
---
 content_scripts/vimium_frontend.coffee | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

(limited to 'content_scripts/vimium_frontend.coffee')

diff --git a/content_scripts/vimium_frontend.coffee b/content_scripts/vimium_frontend.coffee
index fcf5dac1..6f099e54 100644
--- a/content_scripts/vimium_frontend.coffee
+++ b/content_scripts/vimium_frontend.coffee
@@ -1107,7 +1107,8 @@ Tween =
 
 CursorHider =
   #
-  # Hides the cursor when the browser scrolls, and prevent mouse from hovering while invisible
+  # Hides the cursor when the browser scrolls, and prevent mouse from hovering while invisible.
+  # Disabled for Chrome versions less than 39.0.2171.71 due to a browser error.
   # NOTE(smblott) onScroll and onMouseMove events come in pairs.
   #
   cursorHideStyle: null
@@ -1124,8 +1125,10 @@ CursorHider =
     CursorHider.isScrolling = false
 
   init: ->
-    # NOTE(smblott) CursorHider is currently disabled pending a fix for #1345.
-    return
+    # Disable for Chrome versions less than 39.0.2171.71 due to a browser error.
+    chromeVersion = navigator.appVersion.match(/Chrome\/(.*?) /)?[1] || ""
+    return if 0 <= Utils.compareVersions "39.0.2171.71", chromeVersion
+
     @cursorHideStyle = document.createElement("style")
     @cursorHideStyle.innerHTML = """
       body * {pointer-events: none !important; cursor: none !important;}
-- 
cgit v1.2.3


From 226cfb8011e83e48b9c723ebc891ad0e9a7c82ec Mon Sep 17 00:00:00 2001
From: Stephen Blott
Date: Fri, 19 Dec 2014 05:38:29 +0000
Subject: Tidy cursor hider, move haveChromeVersion to utils.

---
 content_scripts/vimium_frontend.coffee | 10 ++++------
 1 file changed, 4 insertions(+), 6 deletions(-)

(limited to 'content_scripts/vimium_frontend.coffee')

diff --git a/content_scripts/vimium_frontend.coffee b/content_scripts/vimium_frontend.coffee
index 6f099e54..21a05de6 100644
--- a/content_scripts/vimium_frontend.coffee
+++ b/content_scripts/vimium_frontend.coffee
@@ -1107,9 +1107,7 @@ Tween =
 
 CursorHider =
   #
-  # Hides the cursor when the browser scrolls, and prevent mouse from hovering while invisible.
-  # Disabled for Chrome versions less than 39.0.2171.71 due to a browser error.
-  # NOTE(smblott) onScroll and onMouseMove events come in pairs.
+  # Hide the cursor when the browser scrolls, and prevent mouse from hovering while invisible.
   #
   cursorHideStyle: null
   isScrolling: false
@@ -1125,9 +1123,9 @@ CursorHider =
     CursorHider.isScrolling = false
 
   init: ->
-    # Disable for Chrome versions less than 39.0.2171.71 due to a browser error.
-    chromeVersion = navigator.appVersion.match(/Chrome\/(.*?) /)?[1] || ""
-    return if 0 <= Utils.compareVersions "39.0.2171.71", chromeVersion
+    # Disable cursor hiding for Chrome versions less than 39.0.2171.71 due to a suspected browser error.
+    # See #1345 and #1348.
+    return unless Utils.haveChromeVersion "39.0.2171.71"
 
     @cursorHideStyle = document.createElement("style")
     @cursorHideStyle.innerHTML = """
-- 
cgit v1.2.3