From 103c3395ddfe60967f921955f6a398daf3d53215 Mon Sep 17 00:00:00 2001
From: Pavel Djundik <github@xpaw.me>
Date: Thu, 15 Mar 2018 15:11:15 +0200
Subject: [PATCH] Store visibility status of sidebar menus

Fixes #51
Closes #1606
---
 client/js/lounge.js             | 22 ++++++++++++++++++----
 client/js/socket-events/init.js | 11 +++++++++--
 client/js/utils.js              |  2 ++
 3 files changed, 29 insertions(+), 6 deletions(-)

diff --git a/client/js/lounge.js b/client/js/lounge.js
index 3f35d757..c93223ec 100644
--- a/client/js/lounge.js
+++ b/client/js/lounge.js
@@ -32,18 +32,29 @@ $(function() {
 	const contextMenuContainer = $("#context-menu-container");
 	const contextMenu = $("#context-menu");
 
+	function storeSidebarVisibility(name, state) {
+		if ($(window).outerWidth() < utils.mobileViewportPixels) {
+			return;
+		}
+
+		storage.set(name, state);
+	}
+
 	$("#windows").on("click", function(e) {
 		const isOpen = slideoutMenu.isOpen();
 
-		if (isOpen || $(e.target).is(".lt")) {
+		if ((isOpen && $(window).outerWidth() < utils.mobileViewportPixels) || $(e.target).is(".lt")) {
 			slideoutMenu.toggle(!isOpen);
+			storeSidebarVisibility("thelounge.state.sidebar", !isOpen);
 		}
 	});
 
 	viewport.on("click", ".rt", function() {
-		const self = $(this);
-		viewport.toggleClass(self.prop("class"));
+		const isOpen = !viewport.hasClass("rt");
+
+		viewport.toggleClass("rt", isOpen);
 		chat.find(".chan.active .chat").trigger("keepToBottom");
+		storeSidebarVisibility("thelounge.state.userlist", isOpen);
 
 		return false;
 	});
@@ -374,7 +385,10 @@ $(function() {
 			}
 
 			utils.scrollIntoViewNicely(self[0]);
-			slideoutMenu.toggle(false);
+
+			if ($(window).outerWidth() < utils.mobileViewportPixels) {
+				slideoutMenu.toggle(false);
+			}
 		}
 
 		const lastActive = $("#windows > .active");
diff --git a/client/js/socket-events/init.js b/client/js/socket-events/init.js
index 009f7a1a..11d8b3e6 100644
--- a/client/js/socket-events/init.js
+++ b/client/js/socket-events/init.js
@@ -42,12 +42,19 @@ socket.on("init", function(data) {
 
 		webpush.configurePushNotifications(data.pushSubscription, data.applicationServerKey);
 
+		slideoutMenu.enable();
+
+		const viewport = $("#viewport");
+
+		if ($(window).outerWidth() >= utils.mobileViewportPixels) {
+			slideoutMenu.toggle(storage.get("thelounge.state.sidebar") === "true");
+			viewport.toggleClass("rt", storage.get("thelounge.state.userlist") === "true");
+		}
+
 		$(document.body).removeClass("signed-out");
 		$("#loading").remove();
 		$("#sign-in").remove();
 
-		slideoutMenu.enable();
-
 		if (window.g_LoungeErrorHandler) {
 			window.removeEventListener("error", window.g_LoungeErrorHandler);
 			window.g_LoungeErrorHandler = null;
diff --git a/client/js/utils.js b/client/js/utils.js
index 9e0bc37c..3761e4da 100644
--- a/client/js/utils.js
+++ b/client/js/utils.js
@@ -9,6 +9,8 @@ var serverHash = -1; // eslint-disable-line no-var
 var lastMessageId = -1; // eslint-disable-line no-var
 
 module.exports = {
+	// Same value as media query in CSS that forces sidebars to become overlays
+	mobileViewportPixels: 768,
 	inputCommands: {collapse, expand, join},
 	findCurrentNetworkChan,
 	serverHash,