Fixes notification sounds

README.md

@@ -4,4 +4,4 @@ Break Helper
 [Break Helper on Chrome Webstore](https://chrome.google.com/webstore/detail/dllmmaniogaffdnagcenlnnlbahmlmld)
 This is a Chrome extension that periodically reminds you to take a break.
 
-Notification sound credits: _VTK Productions (vtkproductions.com)_
+Notification sound credits: _Notification Sounds (notificationsounds.com)_

js/background.js

@@ -78,7 +78,7 @@ if (typeof localStorage.statistics === "undefined") {
 } else {
     statistics = $.parseJSON(localStorage["statistics"]);
 }
-var notification, breakTime, workTime, rule, enableSound, iCloseNotification = 0, timer, nextBreak, breakTimeLeft;
+var notification, breakTime, workTime, rule, iCloseNotification = 0, timer, nextBreak, breakTimeLeft;
 resetTimes();
 $(document).ready(function () {
     chrome.notifications.onButtonClicked.addListener(handleNotificationInteraction);
@@ -111,6 +111,7 @@ function displayNotification() {
         ],
         requireInteraction: true,
     });
+    doSoundNotification();
     log("Displaying notification");
 }
 
@@ -300,6 +301,12 @@ function playSound ( url ) {
     (new Audio(url)).play();
 }
 
+function doSoundNotification() {
+  if (settings.playSound) {
+    playSound('notification.mp3');
+  }
+}
+
 function secondsToClock(leftSeconds) {
     var h, m, s, tempLeft;
     h = Math.floor(leftSeconds / 3600);