Add basic persistent settings

package.json

@@ -37,6 +37,7 @@
   "dependencies": {
     "electron-is-dev": "^1.2.0",
     "electron-log": "^4.1.1",
+    "electron-settings": "^3.2.0",
     "electron-updater": "^4.3.1",
     "play-sound": "^1.1.3"
   }

src/ContextMenu.js

@@ -1,10 +1,11 @@
 const EventEmitter = require("events");
+const settings = require("electron-settings");
 
 const { getIconsByTheme } = require("./utils");
 const { installUpdateAndRestart } = require("./updater");
 
 class ContextMenu extends EventEmitter {
-  constructor() {
+  constructor(userSettings) {
     super();
 
     const { playIcon, stopIcon, updateIcon } = getIconsByTheme();
@@ -38,12 +39,15 @@ class ContextMenu extends EventEmitter {
       },
       { type: "separator" },
       {
-        id: "settings",
-        label: "Settings",
+        id: "sound",
+        label: "Play Sound",
+        type: "checkbox",
+        checked: userSettings.playSound,
         click: () => {
-          this.emit("showSettings");
+          settings.set("playSound", !userSettings.playSound);
         }
       },
+      { type: "separator" },
       { label: "Quit", role: "quit" }
     ];
   }

src/main.js

@@ -7,10 +7,11 @@ const {
   Tray
 } = require("electron");
 const log = require("electron-log");
+const settings = require("electron-settings");
 
 const { showNotification } = require("./notifications");
 const { checkForUpdates } = require("./updater");
-const { setProductionAppPreferences } = require("./utils");
+const { setProductionAppPreferences, loadSettings } = require("./utils");
 
 setProductionAppPreferences();
 
@@ -21,44 +22,56 @@ const ContextMenu = require("./ContextMenu");
 let player, contextMenu, menuTemplate;
 
 app.on("ready", () => {
-  function buildMenu(tray) {
-    contextMenu = new ContextMenu();
+  const userSettings = loadSettings();
+
+  function buildMenu(tray, userSettings) {
+    contextMenu = new ContextMenu(userSettings);
     menuTemplate = Menu.buildFromTemplate(contextMenu.menuItems);
     tray.setContextMenu(menuTemplate);
   }
 
+  function handlePlay() {
+    if (settings.get("playSound")) {
+      player.play();
+    }
+
+    timer.start();
+
+    contextMenu.toggleStartStop(menuTemplate);
+
+    showNotification({
+      title: "Time to Work 👨‍💻",
+      body: "Get it done. It'll feel good later."
+    });
+  }
+
   function handleStop({ shouldReset }) {
+    if (settings.get("playSound")) {
+      player.stop();
+    }
+
     timer.resetTimer(shouldReset);
-    player.stop();
     contextMenu.toggleStartStop(menuTemplate);
   }
 
   function handleBreak() {
-    player.stop();
+    if (settings.get("playSound")) {
+      player.stop();
+    }
+
     timer.resetTimer();
     timer.start();
 
     showNotification({
       title: "Break Time 🧘‍♀️",
-      body: "10 push ups then chill."
-    });
-  }
-
-  function handlePlay() {
-    timer.start();
-    player.play();
-    contextMenu.toggleStartStop(menuTemplate);
-
-    showNotification({
-      title: "Time to Work 👨‍💻",
-      body: "Get it done. It'll feel good later."
+      body: "15 pushups then chill."
     });
   }
 
   const tray = new Tray(nativeImage.createEmpty());
   const timer = new Timer(tray);
   player = new Player(tray);
-  buildMenu(tray);
+  buildMenu(tray, userSettings);
 
   contextMenu.on("play", handlePlay);
   contextMenu.on("stop", handleStop);

src/utils.js

@@ -1,7 +1,19 @@
 const { app, dialog, nativeTheme } = require("electron");
+const settings = require("electron-settings");
 const path = require("path");
 const isDev = require("electron-is-dev");
 
+const loadSettings = () => {
+  // Default play sound to true
+  if (!settings.has("playSound")) {
+    settings.set("playSound", true);
+  }
+
+  return {
+    playSound: settings.get("playSound")
+  };
+};
+
 const joinPath = (relativePath, { extraResource = false } = {}) => {
   // Prod: assets is in the same directory (as Resources) so remove the ../
   if (extraResource && !isDev) {
@@ -66,6 +78,7 @@ const TIMES = {
 
 module.exports = {
   TIMES,
+  loadSettings,
   joinPath,
   getTimerDisplay,
   getIconsByTheme,