refactor: use util's onError() for #292

ffoodd · May 9, 2019 · e7ef404 · e7ef404
1 parent 41c02c0
commit e7ef404
Show file tree

Hide file tree

Showing 5 changed files with 3 additions and 9 deletions.
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/checkalts.js b/webextension/scripts/checkalts.js
@@ -39,14 +39,11 @@ 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
@@ -26,16 +26,12 @@ 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(

diff --git a/webextension/scripts/utils.js b/webextension/scripts/utils.js
@@ -17,6 +17,7 @@ function _t() {
 function onError(error) {
 	console.error(`a11y.css error: ${error}`);
 }
+
 /**
  * Documents what we got from a STORAGE.get()
  * @param {Object} item