From 20a570ebe8e4a7573e71ac854563124657e39408 Mon Sep 17 00:00:00 2001
From: Raymond Hill <rhill@raymondhill.net>
Date: Mon, 9 Dec 2024 09:42:46 -0500
Subject: [PATCH] [mv3] Make strict-blocking checkbox dependent on filtering
 mode

Related feedback:
https://github.com/uBlockOrigin/uBOL-home/issues/214#issuecomment-2528072997
---
 platform/mv3/extension/css/settings.css | 5 +++--
 platform/mv3/extension/dashboard.html   | 4 ++--
 platform/mv3/extension/js/settings.js   | 7 ++++++-
 3 files changed, 11 insertions(+), 5 deletions(-)

diff --git a/platform/mv3/extension/css/settings.css b/platform/mv3/extension/css/settings.css
index 02eb60037..36f9c3d6f 100644
--- a/platform/mv3/extension/css/settings.css
+++ b/platform/mv3/extension/css/settings.css
@@ -30,8 +30,9 @@ body[data-forbid~="filteringMode"] section[data-pane="settings"] > div:has(> h3[
     display: none;
     }
 
-#showBlockedCount:has(input[type="checkbox"][disabled]) {
-    opacity: 0.5;
+label:has(input[type="checkbox"][disabled]),
+label:has(input[type="checkbox"][disabled]) + legend {
+    filter: var(--checkbox-disabled-filter);
     }
 
 #defaultFilteringMode {
diff --git a/platform/mv3/extension/dashboard.html b/platform/mv3/extension/dashboard.html
index 5ac63f3d0..8ef979904 100644
--- a/platform/mv3/extension/dashboard.html
+++ b/platform/mv3/extension/dashboard.html
@@ -31,8 +31,6 @@
             <p><label id="autoReload" data-i18n="autoReloadLabel"><span class="input checkbox"><input type="checkbox"><svg viewBox="0 0 24 24"><path d="M1.73,12.91 8.1,19.28 22.79,4.59"/></svg></span>_</label>
             </p>
             <p><label id="showBlockedCount" data-i18n="showBlockedCountLabel"><span class="input checkbox"><input type="checkbox"><svg viewBox="0 0 24 24"><path d="M1.73,12.91 8.1,19.28 22.79,4.59"/></svg></span>_</label>
-            <p><label id="strictBlockMode" data-i18n="enableStrictBlockLabel"><span class="input checkbox"><input type="checkbox"><svg viewBox="0 0 24 24"><path d="M1.73,12.91 8.1,19.28 22.79,4.59"/></svg></span>_</label><legend data-i18n="enableStrictBlockLegend"></legend>
-            <p id="developerMode" hidden><label><span class="input checkbox"><input type="checkbox"><svg viewBox="0 0 24 24"><path d="M1.73,12.91 8.1,19.28 22.79,4.59"/></svg></span>Developer mode</label>
         </div>
 
         <div class="firstRun">
@@ -90,6 +88,8 @@
                     <div data-i18n="completeFilteringModeDescription"></div>
                 </label>
             </div>
+            <p><label id="strictBlockMode" data-i18n="enableStrictBlockLabel"><span class="input checkbox"><input type="checkbox"><svg viewBox="0 0 24 24"><path d="M1.73,12.91 8.1,19.28 22.79,4.59"/></svg></span>_</label><legend data-i18n="enableStrictBlockLegend"></legend>
+            <p id="developerMode" hidden><label><span class="input checkbox"><input type="checkbox"><svg viewBox="0 0 24 24"><path d="M1.73,12.91 8.1,19.28 22.79,4.59"/></svg></span>Developer mode</label>
         </div>
 
         <div>
diff --git a/platform/mv3/extension/js/settings.js b/platform/mv3/extension/js/settings.js
index 5b73144cb..d924d6f59 100644
--- a/platform/mv3/extension/js/settings.js
+++ b/platform/mv3/extension/js/settings.js
@@ -67,7 +67,12 @@ function renderWidgets() {
         }
     }
 
-    qs$('#strictBlockMode input[type="checkbox"]').checked = cachedRulesetData.strictBlockMode;
+    {
+        const input = qs$('#strictBlockMode input[type="checkbox"]');
+        const canStrictBlock = cachedRulesetData.defaultFilteringMode > 1;
+        input.checked = canStrictBlock && cachedRulesetData.strictBlockMode;
+        dom.attr(input, 'disabled', canStrictBlock ? null : '');
+    }
 
     {
         dom.prop('#developerMode input[type="checkbox"]', 'checked',