Update audio cue settings

ion201 · Feb 7, 2021 · 32fd26d · 32fd26d
1 parent 609b6a8
commit 32fd26d
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 3 deletions.
diff --git a/main-webext.js b/main-webext.js
@@ -15,6 +15,14 @@ var notified_dls = []
 // Note: This only stores a hash, not any details
 var RECENT_DOWNLOADS_MAX_SIZE = 32
 
+
+var SOUND_NAME_TO_FILE = {
+    bell: 'bell.ogg',
+    ding: 'ding.oga',
+    pop: 'pop.oga'
+}
+
+
 function loadPrefs(callback)
 {
     var prefs = Object();
@@ -77,10 +85,11 @@ function checkConfigVer(prefs)
 
 function playSoundAsset(target)
 {
-    if (target.toLowerCase() != 'none')
+    target = target.toLowerCase();
+    if (SOUND_NAME_TO_FILE.hasOwnProperty(target))
     {
         var audioElem = document.createElement('audio');
-        audioElem.src = browser.extension.getURL('assets/' + target);
+        audioElem.src = browser.extension.getURL('assets/' + SOUND_NAME_TO_FILE[target]);
         audioElem.play();
     }
 }

diff --git a/pref/config.json b/pref/config.json
@@ -49,7 +49,7 @@
     {
         "type": "select",
         "defaultValue": "None",
-        "options": ["None", "bell.ogg", "ding.oga", "pop.oga"],
+        "options": ["None", "bell", "ding", "pop"],
         "name": "Audio Notifications",
         "description": "Play an audio cue with every notification"
     }