diff --git a/manifest.json b/manifest.json index 75ed273106634..9c59f1dbbc56e 100644 --- a/manifest.json +++ b/manifest.json @@ -1,5 +1,5 @@ { - "version": "3.20.0", + "version": "3.20.1.1", "key": "ilkggpgmkemaniponkfgnkonpajankkm", "author": "Daniel C. Howe", "background": { diff --git a/src/css/menu.css b/src/css/menu.css index 43d65d6f53fbe..eedb7cb4c7b88 100755 --- a/src/css/menu.css +++ b/src/css/menu.css @@ -764,6 +764,10 @@ body.en .btn_radio_label { opacity: 1; } +#disable:not(.popup_open):checked + .btn_radio_label .popup_arrow { + opacity: 0; +} + #on_domain-lbl { text-align: center; } diff --git a/src/js/adn/core.js b/src/js/adn/core.js index ff51544cf344a..bcf216b8c9036 100644 --- a/src/js/adn/core.js +++ b/src/js/adn/core.js @@ -2375,7 +2375,10 @@ const adnauseam = (function () { if (typeof adnauseam[request.what] === 'function') { - request.url && (request.url = trimChar(request.url, '/')); // no trailing slash + if (request.what !== 'toggleEnabled') { // fix for https://github.com/dhowe/AdNauseam/issues/2516 + // Why do we do this? + request.url && (request.url = trimChar(request.url, '/')); // no trailing slash + } callback(adnauseam[request.what](request, pageStore, tabId, frameId)); adnauseam.markUserAction(); // assume user-initiated and thus no longer 'idle' diff --git a/src/js/adn/menu.js b/src/js/adn/menu.js index 9e6e6db0ab4a6..1d3e6b7412d6b 100644 --- a/src/js/adn/menu.js +++ b/src/js/adn/menu.js @@ -634,6 +634,7 @@ import { broadcast, onBroadcast } from '../broadcast.js'; break; case 'active': toggleStrictAlert(popupData.pageURL, false) + uDom("#on_domain").prop('checked', true); toggleEnabled(evt, true) if (initialButtonState === 'strict') { updateRefreshNotification(currentNotifications) @@ -657,14 +658,14 @@ import { broadcast, onBroadcast } from '../broadcast.js'; const onChangeDisabledScope = function (evt) { // check if url is domain home let url = new URL(popupData.pageURL) - let isDomainHome = url.pathname === '/' || url.pathname === '/index.html' || url.pathname === '/index.php' - var scope = uDom(".disable_type_radio:checked") && !isDomainHome ? uDom(".disable_type_radio:checked").val() : '' + // let isDomainHome = url.pathname === '/' || url.pathname === '/index.html' || url.pathname === '/index.php' + var scope = uDom(".disable_type_radio:checked") ? uDom(".disable_type_radio:checked").val() : '' // first remove previous whichever previous scope from whitelist vAPI.messaging.send( 'adnauseam', { what: 'toggleEnabled', url: popupData.pageURL, - scope: scope, + scope: scope == '' ? 'page' : '', // remove the inverted scope state: true, tabId: popupData.tabId }).then(() => { @@ -699,7 +700,7 @@ import { broadcast, onBroadcast } from '../broadcast.js'; 'adnauseam', { what: 'toggleEnabled', url: popupData.pageURL, - scope: (evt.altKey || evt.metaKey) && !isDomainHome ? 'page' : '', + scope: '', state: state, tabId: popupData.tabId }); @@ -716,6 +717,7 @@ import { broadcast, onBroadcast } from '../broadcast.js'; } const closePopup = function () { + uDom("#disable").removeClass("popup_open") uDom(".popup_arrow").removeClass("open") uDom(".inner-popup_wrapper").addClass("hidden") document.removeEventListener('click', onAnyClickAfterOpen) @@ -724,6 +726,7 @@ import { broadcast, onBroadcast } from '../broadcast.js'; } const openPopup = function () { + uDom("#disable").addClass("popup_open") uDom(".popup_arrow").addClass("open") uDom(".inner-popup_wrapper").removeClass("hidden"); document.addEventListener('click', onAnyClickAfterOpen)