Add toggle for music (#17)

js/background.js

@@ -1,5 +1,6 @@
 // The active background music track is stored here instead of themeAudio.src
 var currentMusic = ''
+var musicEnabled = true
 
 // Set MediaSession API info for Chrome media player popup
 if ('mediaSession' in navigator) {
@@ -15,17 +16,28 @@ themeAudio.loop = true
 
 // Get stored settings
 chrome.storage.local.get({
-    music: 'wii-shop-theme'
+    music: 'wii-shop-theme',
+    musicEnabled: true
 }, function (data) {
     currentMusic = chrome.extension.getURL('music/' + data.music + '.ogg')
+    console.log('Music enabled:', data.musicEnabled)
+    musicEnabled = data.musicEnabled
 })
 
-// Change music after settings change
+// Update settings after storage change
 chrome.storage.onChanged.addListener(function (changes, area) {
+    if (changes.musicEnabled) {
+        musicEnabled = changes.musicEnabled.newValue
+        if (!changes.musicEnabled) {
+            themeAudio.src = ''
+        }
+    }
     if (changes.music) {
         currentMusic = chrome.extension.getURL('music/' + changes.music.newValue + '.ogg')
-        themeAudio.src = chrome.extension.getURL('music/' + changes.music.newValue + '.ogg')
-        themeAudio.play()
+        if (musicEnabled) {
+            themeAudio.src = chrome.extension.getURL('music/' + changes.music.newValue + '.ogg')
+            themeAudio.play()
+        }
     }
 })
 
@@ -33,8 +45,8 @@ chrome.storage.onChanged.addListener(function (changes, area) {
 function checkMusic(tabs) {
     var url = new URL(tabs[0].url)
     var domain = url.hostname.toString().replace('www.', '')
-    console.log(domain)
-    if (siteList.includes(domain)) {
+    console.log(domain, musicEnabled)
+    if (siteList.includes(domain) && musicEnabled) {
         themeAudio.src = currentMusic
         themeAudio.play()
     } else {

js/popup.js

@@ -6,8 +6,38 @@ document.querySelector('#music-picker').addEventListener('change', function () {
 })
 
 // Get stored settings
-chrome.storage.local.get(function (data) {
-    document.querySelector('#music-picker').value = data.music;
+chrome.storage.local.get({
+    music: 'wii-shop-theme',
+    musicEnabled: 'true'
+}, function (data) {
+    document.querySelector('#music-picker').value = data.music
+    if (data.musicEnabled) {
+        document.getElementById('music-toggle').innerText = 'Turn off music'
+    } else {
+        document.getElementById('music-toggle').innerText = 'Turn on music'
+    }
+})
+
+// Music on/off button
+document.getElementById('music-toggle').addEventListener('click', function() {
+    chrome.storage.local.get({
+        musicEnabled: true
+    }, function (data) {
+        console.log(data)
+        if (data.musicEnabled) {
+            // Turn off music
+            document.getElementById('music-toggle').innerText = 'Turn on music'
+            chrome.storage.local.set({
+                musicEnabled: false
+            })
+        } else {
+            // Turn on music
+            document.getElementById('music-toggle').innerText = 'Turn off music'
+            chrome.storage.local.set({
+                musicEnabled: true
+            })
+        }
+    })
 })
 
 // Button link functionality