From 3e8cb47abed8638f48ff31d70437b0dfa14a1f6a Mon Sep 17 00:00:00 2001 From: Richard van der Hoff Date: Fri, 12 Feb 2016 10:19:01 +0000 Subject: [PATCH] Consider mouse-clicking as user activity It's a bit non-sensical that moving the mouse constitutes user activity, but clicking it does not. Fix it. --- src/UserActivity.js | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/UserActivity.js b/src/UserActivity.js index 669b007934..384dc23059 100644 --- a/src/UserActivity.js +++ b/src/UserActivity.js @@ -30,6 +30,7 @@ class UserActivity { * Start listening to user activity */ start() { + document.onmousedown = this._onUserActivity.bind(this); document.onmousemove = this._onUserActivity.bind(this); document.onkeypress = this._onUserActivity.bind(this); // can't use document.scroll here because that's only the document @@ -46,6 +47,7 @@ class UserActivity { * Stop tracking user activity */ stop() { + document.onmousedown = undefined; document.onmousemove = undefined; document.onkeypress = undefined; window.removeEventListener('wheel', this._onUserActivity.bind(this), true);