Replace electron-json-storage with electron-settings

Fixes electron#251

Author: nathanbuchar

assets/demo-btns.js

@@ -1,20 +1,26 @@
-const storage = require('electron-json-storage')
+const settings = require('electron-settings')
 
 const demoBtns = document.querySelectorAll('.js-container-target')
 // Listen for demo button clicks
 Array.prototype.forEach.call(demoBtns, function (btn) {
   btn.addEventListener('click', function (event) {
-    event.target.parentElement.classList.toggle('is-open')
+    const parent = event.target.parentElement;
 
-    // Save currently active demo button in localStorage
-    storage.set('activeDemoButtonId', event.target.getAttribute('id'), function (err) {
-      if (err) return console.error(err)
-    })
+    // Toggles the "is-open" class on the demo's parent element.
+    parent.classList.toggle('is-open')
+
+    // Saves the active demo if it is open, or clears it if the demo was user
+    // collapsed by the user
+    if (parent.classList.contains('is-open')) {
+      settings.set('activeDemoButtonId', event.target.getAttribute('id'))
+    } else {
+      settings.delete('activeDemoButtonId')
+    }
   })
 })
 
 // Default to the demo that was active the last time the app was open
-storage.get('activeDemoButtonId', function (err, id) {
-  if (err) return console.error(err)
-  if (id && id.length) document.getElementById(id).click()
-})
+const buttonId = settings.get('activeDemoButtonId');
+if (buttonId) {
+  document.getElementById(buttonId).click()
+}

assets/nav.js

@@ -1,18 +1,4 @@
-const storage = require('electron-json-storage')
-
-// Default to the view that was active the last time the app was open
-storage.get('activeSectionButtonId', function (err, id) {
-  if (err) return console.error(err)
-
-  if (id && id.length) {
-    showMainContent()
-    const section = document.getElementById(id)
-    if (section) section.click()
-  } else {
-    activateDefaultSection()
-    displayAbout()
-  }
-})
+const settings = require('electron-settings')
 
 document.body.addEventListener('click', function (event) {
   if (event.target.dataset.section) {
@@ -36,9 +22,7 @@ function handleSectionTrigger (event) {
 
   // Save currently active button in localStorage
   const buttonId = event.target.getAttribute('id')
-  storage.set('activeSectionButtonId', buttonId, function (err) {
-    if (err) return console.error(err)
-  })
+  settings.set('activeSectionButtonId', buttonId);
 }
 
 function activateDefaultSection () {
@@ -79,3 +63,14 @@ function hideAllSectionsAndDeselectButtons () {
 function displayAbout () {
   document.querySelector('#about-modal').classList.add('is-shown')
 }
+
+// Default to the view that was active the last time the app was open
+const sectionId = settings.get('activeSectionButtonId')
+if (sectionId) {
+  showMainContent()
+  const section = document.getElementById(sectionId)
+  if (section) section.click()
+} else {
+  activateDefaultSection()
+  displayAbout()
+}

package.json

@@ -44,7 +44,7 @@
     "standard": "^8.2.0"
   },
   "dependencies": {
-    "electron-json-storage": "^2.0.0",
+    "electron-settings": "^3.0.5",
     "electron-shortcut-normalizer": "^1.0.0",
     "glob": "^7.1.0",
     "highlight.js": "^9.3.0"