diff --git a/src/js/background.js b/src/js/background.js index a7fdaa340..8cb846d13 100644 --- a/src/js/background.js +++ b/src/js/background.js @@ -191,8 +191,9 @@ return { noopFunc: function(){}, apiErrorCount: 0, - contextMenuClientX: -1, - contextMenuClientY: -1, + mouseX: -1, + mouseY: -1, + mouseURL: '', epickerTarget: '', epickerEprom: null, diff --git a/src/js/contentscript-end.js b/src/js/contentscript-end.js index 2996839e4..152339a73 100644 --- a/src/js/contentscript-end.js +++ b/src/js/contentscript-end.js @@ -897,19 +897,20 @@ var uBlockCollapser = (function() { if ( window !== window.top ) { return; } - var onContextMenu = function(ev) { + var onMouseClick = function(ev) { messager.send({ - what: 'contextMenuEvent', - clientX: ev.clientX, - clientY: ev.clientY + what: 'mouseClick', + x: ev.clientX, + y: ev.clientY, + url: ev.target && ev.target.localName === 'a' ? ev.target.href : '' }); }; - window.addEventListener('contextmenu', onContextMenu, true); + window.addEventListener('contextmenu', onMouseClick, true); // https://github.com/gorhill/uMatrix/issues/144 vAPI.shutdown.add(function() { - document.removeEventListener('contextmenu', onContextMenu, true); + document.removeEventListener('contextmenu', onMouseClick, true); }); })(); diff --git a/src/js/messaging.js b/src/js/messaging.js index 47886d5e3..07a2bc69e 100644 --- a/src/js/messaging.js +++ b/src/js/messaging.js @@ -96,9 +96,10 @@ var onMessage = function(request, sender, callback) { var response; switch ( request.what ) { - case 'contextMenuEvent': - µb.contextMenuClientX = request.clientX; - µb.contextMenuClientY = request.clientY; + case 'mouseClick': + µb.mouseX = request.x; + µb.mouseY = request.y; + µb.mouseURL = request.url; break; case 'cosmeticFiltersInjected': @@ -133,7 +134,7 @@ var onMessage = function(request, sender, callback) { case 'launchElementPicker': // Launched from some auxiliary pages, clear context menu coords. - µb.contextMenuClientX = µb.contextMenuClientY = -1; + µb.mouseX = µb.mouseY = -1; µb.elementPickerExec(request.tabId, request.targetURL); break; @@ -633,14 +634,14 @@ var onMessage = function(request, sender, callback) { callback({ frameContent: this.responseText.replace(reStrings, replacer), target: µb.epickerTarget, - clientX: µb.contextMenuClientX, - clientY: µb.contextMenuClientY, + clientX: µb.mouseX, + clientY: µb.mouseY, eprom: µb.epickerEprom }); µb.epickerTarget = ''; - µb.contextMenuClientX = -1; - µb.contextMenuClientY = -1; + µb.mouseX = -1; + µb.mouseY = -1; }; xhr.send(); return; diff --git a/src/js/tab.js b/src/js/tab.js index 6625bff26..d7ce86308 100644 --- a/src/js/tab.js +++ b/src/js/tab.js @@ -500,7 +500,10 @@ vAPI.tabs.onPopup = function(details) { var loggerEnabled = µb.logger.isEnabled(); // Check user switch first - if ( µb.hnSwitches.evaluateZ('no-popups', openerHostname) ) { + if ( + targetURL !== µb.mouseURL && + µb.hnSwitches.evaluateZ('no-popups', openerHostname) + ) { result = 'ub:no-popups: ' + µb.hnSwitches.z + ' true'; }