Merge pull request #350 from Lausselloic/webextension-fixchrome

catch error when sending message looks to fix error
ffoodd · May 9, 2019 · 52e8672 · 52e8672
2 parents feba69e + 783e784
commit 52e8672
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 2 deletions.
diff --git a/webextension/scripts/checkalts.js b/webextension/scripts/checkalts.js
@@ -39,11 +39,14 @@ btnCheckalts.addEventListener('click', function () {
 			icons: icons,
 			strings: strings
 		});
-	});
+	}).catch(onError);
 	var checked = this.getAttribute('aria-checked') === 'true' || false;
 	this.setAttribute('aria-checked', !checked);
 	storeCheckAltsStatus(!checked);
 });
+function onError(error) {
+    console.error(`Error: ${error}`);
+}
 
 function checkAltsOnload() {
 	let getStatus = browser.storage.local.get("checkAltsStatus");

diff --git a/webextension/scripts/textspacing.js b/webextension/scripts/textspacing.js
@@ -27,12 +27,16 @@ btnTextspacing.addEventListener('click', function () {
 		browser.tabs.sendMessage(tabs[0].id, {
 			a11ycss_action: "textspacing"
 		});
-	});
+	}).catch(onError);
 	var checked = this.getAttribute('aria-checked') === 'true' || false;
 	this.setAttribute('aria-checked', !checked);
 	storeTextSpacingStatus(!checked);
 });
 
+function onError(error) {
+    console.error(`Error: ${error}`);
+}
+
 function textSpacingOnload() {
 	let getStatus = browser.storage.local.get("textSpacingStatus");
 	getStatus.then(