feat[WIP]: handle tab's URL change, need a way to remove current tab'…

…s setting for #292
ffoodd · May 9, 2019 · 1322db8 · 1322db8
1 parent e7ef404
commit 1322db8
Show file tree

Hide file tree

Showing 3 changed files with 16 additions and 1 deletion.
diff --git a/a11ycss-webextension-chrome.zip b/a11ycss-webextension-chrome.zip
diff --git a/a11ycss-webextension-firefox.zip b/a11ycss-webextension-firefox.zip
diff --git a/webextension/scripts/textspacing.js b/webextension/scripts/textspacing.js
@@ -32,13 +32,28 @@ btnTextspacing.addEventListener('click', function () {
 	storeTextSpacingStatus(!checked);
 });
 
+function textSpacingOnUpdated(tabId, changeInfo, tabInfo) {
+  if (changeInfo.url) {
+    let getStatus = browser.storage.local.get("textSpacingStatus");
+	getStatus.then(
+		// when we got something
+		(item) => {
+			if (item && item.textSpacingStatus && item.textSpacingStatus[tabId]) {
+				// Need to find a way to only remove current tab's value
+			}
+		}, onError
+	);
+  }
+}
+
+browser.tabs.onUpdated.addListener(textSpacingOnUpdated);
+
 function textSpacingOnload() {
 	let getStatus = browser.storage.local.get("textSpacingStatus");
 	getStatus.then(
 		// when we got something
 		(item) => {
 			if (item && item.textSpacingStatus) {
-				console.log(item.textSpacingStatus);
 				browser.tabs.query({ active: true, currentWindow: true }).then((tabs) => {
 					// If a setting is found for this tab
 					if (item.textSpacingStatus[tabs[0].id]) {