From ee5090959b1bff33f8e70b6f440ccf5152bc563b Mon Sep 17 00:00:00 2001
From: Stefan Malzner <stefan@adlk.io>
Date: Sun, 2 Dec 2018 21:19:30 +0100
Subject: Fix set initial state for dark mode

---
 src/config.js               | 3 ++-
 src/stores/SettingsStore.js | 7 ++-----
 2 files changed, 4 insertions(+), 6 deletions(-)

(limited to 'src')

diff --git a/src/config.js b/src/config.js
index 08dbb85f3..c612d1e2b 100644
--- a/src/config.js
+++ b/src/config.js
@@ -2,6 +2,7 @@ import electron from 'electron';
 import path from 'path';
 
 const app = process.type === 'renderer' ? electron.remote.app : electron.app;
+const systemPreferences = process.type === 'renderer' ? electron.remote.systemPreferences : electron.systemPreferences;
 
 export const CHECK_INTERVAL = 1000 * 3600; // How often should we perform checks
 export const LOCAL_API = 'http://localhost:3000';
@@ -18,7 +19,7 @@ export const DEFAULT_APP_SETTINGS = {
   showMessageBadgeWhenMuted: true,
   enableSpellchecking: true,
   spellcheckerLanguage: 'en-us',
-  darkMode: false,
+  darkMode: process.platform === 'darwin' ? systemPreferences.isDarkMode() : false, // We can't use refs from `./environment` at this time
   locale: '',
   fallbackLocale: 'en-US',
   beta: false,
diff --git a/src/stores/SettingsStore.js b/src/stores/SettingsStore.js
index 4a42ed924..61528ba2c 100644
--- a/src/stores/SettingsStore.js
+++ b/src/stores/SettingsStore.js
@@ -1,4 +1,4 @@
-import { remote, ipcRenderer } from 'electron';
+import { ipcRenderer } from 'electron';
 import { action, computed, observable } from 'mobx';
 import localStorage from 'mobx-localstorage';
 
@@ -10,7 +10,6 @@ import { getLocale } from '../helpers/i18n-helpers';
 import { DEFAULT_APP_SETTINGS, FILE_SYSTEM_SETTINGS_TYPES } from '../config';
 import { SPELLCHECKER_LOCALES } from '../i18n/languages';
 
-const { systemPreferences } = remote;
 const debug = require('debug')('Franz:SettingsStore');
 
 export default class SettingsStore extends Store {
@@ -37,7 +36,7 @@ export default class SettingsStore extends Store {
     });
 
     ipcRenderer.on('appSettings', (event, resp) => {
-      debug('Get appSettings resolves', resp, resp.type, resp.data);
+      debug('Get appSettings resolves', resp.type, resp.data);
 
       this._fileSystemSettingsCache[resp.type] = resp.data;
     });
@@ -156,7 +155,6 @@ export default class SettingsStore extends Store {
       debug('Migrated settings to split stores');
     }
 
-    // Enable dark mode once
     if (!this.all.migration['5.0.0-beta.19-settings']) {
       const spellcheckerLanguage = getLocale({
         locale: this.stores.settings.app.locale,
@@ -168,7 +166,6 @@ export default class SettingsStore extends Store {
       this.actions.settings.update({
         type: 'app',
         data: {
-          darkMode: systemPreferences.isDarkMode(),
           spellcheckerLanguage,
         },
       });
-- 
cgit v1.2.3-70-g09d2


From b089a6dc52cbfbbcd6ad1fb93d8009d7795fee3a Mon Sep 17 00:00:00 2001
From: Stefan Malzner <stefan@adlk.io>
Date: Sun, 2 Dec 2018 21:19:54 +0100
Subject: Make settings debug logs less confusing

---
 src/electron/Settings.js | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

(limited to 'src')

diff --git a/src/electron/Settings.js b/src/electron/Settings.js
index 6ac3b9177..ed9733bfe 100644
--- a/src/electron/Settings.js
+++ b/src/electron/Settings.js
@@ -42,12 +42,12 @@ export default class Settings {
 
   _hydrate() {
     this.store = this._merge(readJsonSync(this.settingsFile));
-    debug('Hydrate store', toJS(this.store));
+    debug('Hydrate store', this.type, toJS(this.store));
   }
 
   _writeFile() {
     outputJsonSync(this.settingsFile, this.store);
-    debug('Write settings file', toJS(this.store));
+    debug('Write settings file', this.type, toJS(this.store));
   }
 
   get settingsFile() {
-- 
cgit v1.2.3-70-g09d2