From d77319e328137743bb0b024abba2d562017955e2 Mon Sep 17 00:00:00 2001 From: Alex Yatskov Date: Mon, 28 Mar 2016 13:00:48 -0700 Subject: WIP --- ext/client.js | 48 +++++++++++++++++++++++++++++++++--------------- ext/util.js | 4 ++-- 2 files changed, 35 insertions(+), 17 deletions(-) (limited to 'ext') diff --git a/ext/client.js b/ext/client.js index 74b246f3..76e4ceed 100644 --- a/ext/client.js +++ b/ext/client.js @@ -21,36 +21,55 @@ class Client { constructor() { this.popup = $('
'); this.popupOffset = 10; + this.lastMosePos = null; this.enabled = false; - $('body').append(this.popup).click(() => this.hidePopup()); + $('body').append(this.popup); chrome.runtime.onMessage.addListener(this.onMessage.bind(this)); - window.addEventListener('mousemove', this.onMouseMove.bind(this)); - window.addEventListener('keydown', this.onKeyDown.bind(this)); + window.addEventListener('mousedown', this.onMouseAction.bind(this)); + window.addEventListener('mousemove', this.onMouseAction.bind(this)); + window.addEventListener('keydown', this.onKeyAction.bind(this)); getState((state) => this.setEnabled(state === 'enabled')); } - onKeyDown(e) { - if (e.keyCode === 16 || e.charCode === 16) { - this.hidePopup(); + onKeyAction(e) { + if (!this.enabled) { + return; + } + + if (this.lastMousePos !== null && (e.keyCode === 16 || e.charCode === 16)) { + this.searchAtPoint(this.lastMousePos); } } - onMouseMove(e) { - if (!this.enabled || (!e.shiftKey && e.which !== 2)) { + onMouseAction(e) { + this.lastMousePos = {x: e.clientX, y: e.clientY}; + + if (!this.enabled) { return; } - const range = getRangeAtCursor(e, 10); + if (e.shiftKey || e.which === 2) { + this.searchAtPoint(this.lastMousePos); + } + } + + onMessage(request, sender, callback) { + this.setEnabled(request === 'enabled'); + callback(); + } + + searchAtPoint(point) { + const range = getRangeAtPoint(point, 10); if (range === null) { this.hidePopup(); return; } const rect = getRangePaddedRect(range); - if (e.clientX < rect.left || e.clientX > rect.right) { + if (point.x < rect.left || point.x > rect.right) { this.hidePopup(); return; } @@ -68,11 +87,6 @@ class Client { }); } - onMessage(request, sender, callback) { - this.setEnabled(request === 'enabled'); - callback(); - } - showPopup(range) { const selection = window.getSelection(); selection.removeAllRanges(); @@ -83,6 +97,10 @@ class Client { } hidePopup() { + if (this.popup.css('visibility') === 'hidden') { + return; + } + const selection = window.getSelection(); selection.removeAllRanges(); diff --git a/ext/util.js b/ext/util.js index 76579cac..8db4f297 100644 --- a/ext/util.js +++ b/ext/util.js @@ -17,8 +17,8 @@ */ -function getRangeAtCursor(e, lookAhead) { - const range = document.caretRangeFromPoint(e.clientX, e.clientY); +function getRangeAtPoint(point, lookAhead) { + const range = document.caretRangeFromPoint(point.x, point.y); if (range === null) { return null; } -- cgit v1.2.3