Persist the logged-in window state (#534)

atomiclabs · Sep 29, 2018 · 420a443 · 420a443
1 parent 20b2834
commit 420a443
Show file tree

Hide file tree

Showing 3 changed files with 24 additions and 4 deletions.
diff --git a/app/index.js b/app/index.js
@@ -19,6 +19,7 @@ const config = require('./config');
 const marketmaker = require('./marketmaker');
 const {loginWindowSize} = require('./constants');
 const {isDevelopment} = require('./util-common');
+const rendererState = require('./renderer-state');
 
 require('electron-unhandled')({
 	showDialog: !isDevelopment,
@@ -182,8 +183,9 @@ app.on('window-all-closed', () => {
 });
 
 app.on('before-quit', () => {
-	// TODO: Only save this when logged in.
-	// config.set('windowState', mainWindow.getBounds());
+	if (rendererState.isLoggedIn) {
+		config.set('windowState', mainWindow.getBounds());
+	}
 });
 
 ipc.answerRenderer('start-marketmaker', async seedPhrase => {

diff --git a/app/renderer-state.js b/app/renderer-state.js
@@ -0,0 +1,13 @@
+'use strict';
+const electron = require('electron');
+
+const {ipcMain: ipc} = electron;
+
+const rendererState = {};
+
+ipc.on('app-container-state-updated', (event, state) => {
+	rendererState.appContainer = state;
+	rendererState.isLoggedIn = state.activeView !== 'Login' && state.activeView !== 'AppSettings';
+});
+
+module.exports = rendererState;
diff --git a/app/renderer/containers/Login.js b/app/renderer/containers/Login.js
@@ -18,8 +18,13 @@ const setAppWindowBounds = () => {
 	win.setMaximizable(true);
 	win.setFullScreenable(true);
 	win.setMinimumSize(minWindowSize.width, minWindowSize.height);
-	setWindowBounds(config.get('windowState'));
-	win.center(); // TODO: Remove this when `setWindowBounds` handles positioning the window inside the window bounds
+
+	const windowState = config.get('windowState');
+	setWindowBounds(windowState);
+	const hasPositionState = Reflect.has(windowState, 'x');
+	if (!hasPositionState) {
+		win.center();
+	}
 };
 
 const initApi = async seedPhrase => {