Debug-Option per Einstellungsoberfläche konfigurierbar gemacht

cadeyrn · Jan 11, 2017 · 20fd790 · 20fd790
1 parent ea1342e
commit 20fd790
Show file tree

Hide file tree

Showing 7 changed files with 81 additions and 5 deletions.
diff --git a/src/_locales/de/messages.json b/src/_locales/de/messages.json
@@ -68,6 +68,9 @@
   "omnibox_command_check_unknowns": {
     "message": "Prüfe deine Lesezeichen nur auf welche, deren Status nicht ermittelt werden kann"
   },
+  "option_debug_label": {
+    "message": "Debug-Option aktivieren"
+  },
   "search_label": {
     "message" : "Titel oder URL…"
   }

diff --git a/src/_locales/en/messages.json b/src/_locales/en/messages.json
@@ -68,6 +68,9 @@
   "omnibox_command_check_unknowns": {
     "message": "Prüfe deine Lesezeichen nur auf welche, deren Status nicht ermittelt werden kann"
   },
+  "option_debug_label": {
+    "message": "Debug-Option aktivieren"
+  },
   "search_label": {
     "message" : "Titel oder URL…"
   }

diff --git a/src/css/ui.css b/src/css/ui.css
@@ -81,6 +81,15 @@ footer a {
   display: none;
 }
 
+.vertical-align > div {
+  display: table-cell;
+  vertical-align: middle;
+}
+
+.vertical-align label {
+  margin-left: 5px;
+}
+
 #counters {
   margin-bottom: 20px;
   font-weight: bold;

diff --git a/src/html/options.html b/src/html/options.html
@@ -0,0 +1,32 @@
+<!DOCTYPE html>
+<html>
+<head>
+  <meta charset='utf-8' />
+  <title>__MSG_extension_name__</title>
+  <link rel='stylesheet' href='../css/ui.css' />
+  <link rel='icon' type='image/png' href='../images/icon-32.png'>
+</head>
+<body>
+  <header>
+    <a href='ui.html'><h1>__MSG_extension_name__</h1></a>
+  </header>
+  <progress id='progress' value='0'></progress>
+  <main>
+    <section>
+      <form>
+        <div class='vertical-align'>
+          <div><input type='checkbox' name='debug' id='debug'></div>
+          <div><label for='debug'>__MSG_option_debug_label__</label></div>
+        </div>
+      </form>
+    </section>
+  </main>
+  <footer>
+    <a href='https://www.soeren-hentzschel.at' target='_blank' rel='noopener'>
+      __MSG_extension_name__ © 2017, soeren-hentzschel.at
+    </a>
+  </footer>
+  <script src='../js/i18n.js'></script>
+  <script src='../js/options.js'></script>
+</body>
+</html>
diff --git a/src/js/background.js b/src/js/background.js
@@ -2,10 +2,10 @@
 
 const bookmarkchecker = {
   UI_PAGE : 'html/ui.html',
-  DEBUG_MODE : false,
   LIMIT : 0,
   TIMEOUT : 0,
   ATTEMPTS : 2,
+  debug_enabled : false,
   inProgress : false,
   internalCounter : 0,
   totalBookmarks : 0,
@@ -109,6 +109,10 @@ const bookmarkchecker = {
     bookmarkchecker.bookmarksResult = [];
     bookmarkchecker.debug = [];
 
+    browser.storage.local.get('debug_enabled', function (result) {
+      bookmarkchecker.debug_enabled = result.debug_enabled;
+    });
+
     browser.bookmarks.getTree().then((bookmarks) => {
       bookmarkchecker.checkBookmarks(bookmarks[0], 'check', type);
     });
@@ -233,7 +237,7 @@ const bookmarkchecker = {
         bookmark.status = response.status;
       }
 
-      if (bookmarkchecker.DEBUG_MODE) {
+      if (bookmarkchecker.debug_enabled) {
         bookmarkchecker.debug.push({
           bookmark : {
             id : bookmark.id,
@@ -262,7 +266,7 @@ const bookmarkchecker = {
         bookmark.status = STATUS.NOT_FOUND;
       }
 
-      if (bookmarkchecker.DEBUG_MODE) {
+      if (bookmarkchecker.debug_enabled) {
         bookmarkchecker.debug.push({
           bookmark : {
             id : bookmark.id,

diff --git a/src/js/options.js b/src/js/options.js
@@ -0,0 +1,20 @@
+'use strict';
+
+const elDebug = document.querySelector('#debug');
+
+const options = {
+  load : function () {
+    function setDebugOption(result) {
+      elDebug.checked = result.debug_enabled || false;
+    }
+
+    let debug = browser.storage.local.get('debug_enabled');
+    debug.then(setDebugOption);
+  }
+};
+
+document.addEventListener('DOMContentLoaded', options.load);
+
+elDebug.addEventListener('change', function () {
+  browser.storage.local.set({ 'debug_enabled' : this.checked });
+});
diff --git a/src/manifest.json b/src/manifest.json
@@ -26,10 +26,15 @@
   "omnibox": {
     "keyword" : "bookmarks"
   },
+  "options_ui": {
+    "page": "html/options.html",
+    "open_in_tab": true
+  },
   "permissions": [
+    "<all_urls>",
     "bookmarks",
-    "tabs",
-    "<all_urls>"
+    "storage",
+    "tabs"
   ],
   "default_locale": "en",
   "applications": {