Add a settings page.

pe1uca · pe1uca · commit b1f0b456af2f · 2023-06-15T16:33:17.000-04:00
Quick option to turn on and off the redirection
diff --git a/app/_locales/en/messages.json b/app/_locales/en/messages.json
@@ -10,5 +10,9 @@
   "appDescription": {
     "message": "Always use WayBack machine (archive.org) for reddit links",
     "description": "The description of the application"
+  },
+  "actionTitle": {
+    "message": "Reddit to WBM",
+    "description": "Tooltip for browser action icon"
   }
 }
diff --git a/app/html/settings.html b/app/html/settings.html
@@ -0,0 +1,14 @@
+<!DOCTYPE html>
+<html>
+	<head>
+        <meta charset="utf-8">
+    </head>
+
+	<body>
+		<div>
+			<input type="checkbox" id="enableRWBM" name="enableRWBM" value="enabled" />
+			<label for="enableRWBM">Enable redirect to WBM</label>
+		</div>
+		<script src="../scripts/settings.js"></script>
+	</body>
+</html>
diff --git a/app/manifest.json b/app/manifest.json
@@ -2,7 +2,7 @@
   "name": "__MSG_appName__",
   "short_name": "__MSG_appShortName__",
   "description": "__MSG_appDescription__",
-  "version": "1.0.0",
+  "version": "2.0.0",
   "manifest_version": 2,
   "default_locale": "en",
   "icons": {
@@ -11,13 +11,23 @@
   },
   "permissions": [
     "*://*.reddit.com/*",
-    "https://web.archive.org/*",
+    "*://web.archive.org/*",
+    "storage",
     "webRequest",
     "webRequestBlocking"
   ],
   "background": {
     "scripts": [
       "scripts/background.js"
     ]
+  },
+  "page_action": {
+    "default_title": "__MSG_actionTitle__",
+    "default_popup": "html/settings.html",
+    "default_icon": "images/icon-128.png",
+    "show_matches": ["*://*.reddit.com/*", "https://web.archive.org/*"]
+  },
+  "options_ui": {
+    "page": "html/settings.html"
   }
 }
diff --git a/app/scripts/background.ts b/app/scripts/background.ts
@@ -1,15 +1,29 @@
-browser.webRequest.onBeforeRequest.addListener(
-  checkWBM,
-  { urls: ["*://*.reddit.com/*"], types: ["main_frame"] },
-  ["blocking"]
-);
-
 browser.webRequest.onBeforeRequest.addListener(
   (details) => { return { upgradeToSecure: true } },
   { urls: ["http://*.archive.org/*"], types: ["main_frame"] },
   ["blocking"]
 );
 
+browser.storage.local.get('enableRWBM').then((value) => {
+  if (!value.enableRWBM) setRWBMEnabled(true);
+});
+
+browser.storage.local.onChanged.addListener((changes) => {
+  if(changes.enableRWBM) setRWBMEnabled(changes.enableRWBM.newValue);
+});
+
+function setRWBMEnabled(enabled: boolean) {
+  browser.webRequest.onBeforeRequest.removeListener(checkWBM);
+
+  if (enabled) {
+    browser.webRequest.onBeforeRequest.addListener(
+      checkWBM,
+      { urls: ["*://*.reddit.com/*"], types: ["main_frame"] },
+      ["blocking"]
+    );
+  }
+}
+
 async function checkWBM(details: browser.webRequest._OnBeforeRequestDetails): Promise<browser.webRequest.BlockingResponse> {
   if (details.tabId === -1 || details.method !== "GET") {
     return {};
diff --git a/app/scripts/settings.ts b/app/scripts/settings.ts
@@ -0,0 +1,14 @@
+
+(() => {
+	const enableRWBM = document.getElementById('enableRWBM') as HTMLInputElement;
+	if (!enableRWBM) return;
+	enableRWBM.addEventListener('change', (event) => {
+		browser.storage.local.set({
+			enableRWBM: (event.target as HTMLInputElement).checked
+		});
+	});
+
+	browser.storage.local.get('enableRWBM').then((value) => {
+		enableRWBM.checked = value.enableRWBM ?? true;
+	});
+})();

Original file line number	Diff line number	Diff line change
`@@ -10,5 +10,9 @@`
`10`	`10`	`"appDescription": {`
`11`	`11`	`"message": "Always use WayBack machine (archive.org) for reddit links",`
`12`	`12`	`"description": "The description of the application"`
	`13`	`+ },`
	`14`	`+ "actionTitle": {`
	`15`	`+ "message": "Reddit to WBM",`
	`16`	`+ "description": "Tooltip for browser action icon"`
`13`	`17`	`}`
`14`	`18`	`}`