From cbfc279edd1eff51c973090413522b8ca99a1d10 Mon Sep 17 00:00:00 2001
From: Miroslav Pejic <miroslav.pejic.85@gmail.com>
Date: Fri, 28 Jul 2023 20:18:28 +0200
Subject: [PATCH] [mirotalksfu] - #112 keep settings on local storage

---
 public/js/LocalStorage.js | 17 +++++++++++
 public/js/Room.js         | 59 ++++++++++++++++++++++++++++-----------
 public/js/RoomClient.js   | 10 +++----
 public/views/Room.html    |  6 ++--
 4 files changed, 67 insertions(+), 25 deletions(-)

diff --git a/public/js/LocalStorage.js b/public/js/LocalStorage.js
index 08e7adb1..4176dfc9 100644
--- a/public/js/LocalStorage.js
+++ b/public/js/LocalStorage.js
@@ -15,6 +15,14 @@ class LocalStorage {
             audioVideo: true,
         };
 
+        this.SFU_SETTINGS = {
+            video_obj_fit: 'cover',
+            video_controls: 'off',
+            theme: 'dark',
+            buttons_bar: 'vertical',
+            pin_grid: 'vertical',
+        };
+
         this.DEVICES_COUNT = {
             audio: 0,
             speaker: 0,
@@ -52,6 +60,11 @@ class LocalStorage {
         localStorage.setItem(name, JSON.stringify(object));
     }
 
+    setSettings(settings) {
+        this.SFU_SETTINGS = settings;
+        this.setObjectLocalStorage('SFU_SETTINGS', this.SFU_SETTINGS);
+    }
+
     setInitConfig(type, status) {
         switch (type) {
             case this.MEDIA_TYPE.audio:
@@ -94,6 +107,10 @@ class LocalStorage {
     // GET LOCAL STORAGE
     // ####################################################
 
+    getSettings() {
+        return this.getObjectLocalStorage('SFU_SETTINGS');
+    }
+
     getInitConfig() {
         return this.getObjectLocalStorage('INIT_CONFIG');
     }
diff --git a/public/js/Room.js b/public/js/Room.js
index 2b1c6136..3f41322d 100644
--- a/public/js/Room.js
+++ b/public/js/Room.js
@@ -55,12 +55,16 @@ const wbHeight = 600;
 const swalImageUrl = '../images/pricing-illustration.svg';
 
 const lS = new LocalStorage();
-const localStorageSettings = lS.getObjectLocalStorage('MIROTALK_SFU_SETTINGS');
+// Get Settings from localStorage
+const localStorageSettings = lS.getObjectLocalStorage('SFU_SETTINGS');
 const lsSettings = localStorageSettings
     ? localStorageSettings
     : {
-          theme: 'dark',
-          //...
+          video_obj_fit: 2, // cover
+          video_controls: BtnVideoControls.selectedIndex,
+          theme: selectTheme.selectedIndex,
+          buttons_bar: BtnsBarPosition.selectedIndex,
+          pin_grid: pinVideoPosition.selectedIndex,
       };
 
 // ####################################################
@@ -131,7 +135,7 @@ let initStream = null;
 // ####################################################
 
 function initClient() {
-    setTheme(lsSettings.theme);
+    setTheme();
     if (!DetectRTC.isMobileDevice) {
         setTippy('shareButton', 'Share room', 'right');
         setTippy('hideMeButton', 'Toggle hide me', 'right');
@@ -837,6 +841,7 @@ function roomIsReady() {
     BUTTONS.settings.lobbyButton && show(lobbyButton);
     BUTTONS.main.aboutButton && show(aboutButton);
     if (!DetectRTC.isMobileDevice) show(pinUnpinGridDiv);
+    loadSettings();
     handleButtons();
     handleSelects();
     handleInputs();
@@ -920,6 +925,21 @@ function stopRecordingTimer() {
     clearInterval(recTimer);
 }
 
+// ####################################################
+// LOAD SETTINGS
+// ####################################################
+
+function loadSettings() {
+    BtnVideoObjectFit.selectedIndex = lsSettings.video_obj_fit;
+    BtnVideoControls.selectedIndex = lsSettings.video_controls;
+    BtnsBarPosition.selectedIndex = lsSettings.buttons_bar;
+    pinVideoPosition.selectedIndex = lsSettings.pin_grid;
+    rc.handleVideoObjectFit(BtnVideoObjectFit.value);
+    rc.handleVideoControls(BtnVideoControls.value);
+    rc.changeBtnsBarPosition(BtnsBarPosition.value);
+    rc.togglePin(pinVideoPosition.value);
+}
+
 // ####################################################
 // HTML BUTTONS
 // ####################################################
@@ -1347,25 +1367,33 @@ function handleSelects() {
         rc.lobbyToggle();
     };
     // styling
-    BtnsAspectRatio.onchange = () => {
-        setAspectRatio(BtnsAspectRatio.value);
+    BtnAspectRatio.onchange = () => {
+        setAspectRatio(BtnAspectRatio.value);
     };
     BtnVideoObjectFit.onchange = () => {
         rc.handleVideoObjectFit(BtnVideoObjectFit.value);
+        lsSettings.video_obj_fit = BtnVideoObjectFit.selectedIndex;
+        lS.setSettings(lsSettings);
     }; // cover
-    BtnVideoObjectFit.selectedIndex = 2;
-
     BtnVideoControls.onchange = () => {
         rc.handleVideoControls(BtnVideoControls.value);
+        lsSettings.video_controls = BtnVideoControls.selectedIndex;
+        lS.setSettings(lsSettings);
     };
     selectTheme.onchange = () => {
-        setTheme(selectTheme.value);
+        lsSettings.theme = selectTheme.selectedIndex;
+        lS.setSettings(lsSettings);
+        setTheme();
     };
     BtnsBarPosition.onchange = () => {
         rc.changeBtnsBarPosition(BtnsBarPosition.value);
+        lsSettings.buttons_bar = BtnsBarPosition.selectedIndex;
+        lS.setSettings(lsSettings);
     };
     pinVideoPosition.onchange = () => {
         rc.togglePin(pinVideoPosition.value);
+        lsSettings.pin_grid = pinVideoPosition.selectedIndex;
+        lS.setSettings(lsSettings);
     };
     // chat
     showChatOnMsg.onchange = (e) => {
@@ -1387,8 +1415,7 @@ function handleSelects() {
     };
     whiteboardGhostButton.onclick = (e) => {
         wbIsBgTransparent = !wbIsBgTransparent;
-        //setWhiteboardBgColor(wbIsBgTransparent ? 'rgba(0, 0, 0, 0.100)' : wbBackgroundColorEl.value);
-        wbIsBgTransparent ? wbCanvasBackgroundColor('rgba(0, 0, 0, 0.100)') : setTheme(lsSettings.theme);
+        wbIsBgTransparent ? wbCanvasBackgroundColor('rgba(0, 0, 0, 0.100)') : setTheme();
     };
 }
 
@@ -2346,7 +2373,9 @@ function getParticipantAvatar(peerName) {
 // SET THEME
 // ####################################################
 
-function setTheme(theme) {
+function setTheme() {
+    selectTheme.selectedIndex = lsSettings.theme;
+    const theme = selectTheme.value;
     switch (theme) {
         case 'dark':
             swalBackground = 'radial-gradient(#393939, #000000)';
@@ -2420,8 +2449,6 @@ function setTheme(theme) {
             break;
         //...
     }
-    lsSettings.theme = theme;
-    lS.setObjectLocalStorage('MIROTALK_SFU_SETTINGS', lsSettings);
     wbIsBgTransparent = false;
     if (rc) rc.isChatBgTransparent = false;
 }
@@ -2492,8 +2519,8 @@ function adaptAspectRatio(participantsCount) {
         desktop = 1; // (4:3)
         mobile = 3; // (1:1)
     }
-    BtnsAspectRatio.selectedIndex = DetectRTC.isMobileDevice ? mobile : desktop;
-    setAspectRatio(BtnsAspectRatio.selectedIndex);
+    BtnAspectRatio.selectedIndex = DetectRTC.isMobileDevice ? mobile : desktop;
+    setAspectRatio(BtnAspectRatio.selectedIndex);
 }
 
 // ####################################################
diff --git a/public/js/RoomClient.js b/public/js/RoomClient.js
index 9eb16b09..7909e6f4 100644
--- a/public/js/RoomClient.js
+++ b/public/js/RoomClient.js
@@ -2504,7 +2504,7 @@ class RoomClient {
     }
 
     handleVideoControls(value) {
-        isVideoControlsOn = value == 'On' ? true : false;
+        isVideoControlsOn = value == 'on' ? true : false;
         let cameras = this.getEcN('Camera');
         for (let i = 0; i < cameras.length; i++) {
             let cameraId = cameras[i].id.replace('__video', '');
@@ -3153,11 +3153,9 @@ class RoomClient {
 
     chatToggleBg() {
         this.isChatBgTransparent = !this.isChatBgTransparent;
-        if (this.isChatBgTransparent) {
-            document.documentElement.style.setProperty('--msger-bg', 'rgba(0, 0, 0, 0.100)');
-        } else {
-            setTheme(lsSettings.theme);
-        }
+        this.isChatBgTransparent
+            ? document.documentElement.style.setProperty('--msger-bg', 'rgba(0, 0, 0, 0.100)')
+            : setTheme();
     }
 
     chatClean() {
diff --git a/public/views/Room.html b/public/views/Room.html
index 2d572191..949e0136 100644
--- a/public/views/Room.html
+++ b/public/views/Room.html
@@ -298,7 +298,7 @@ access to use this app.
                     <br />
                     <p>Aspect ratio:</p>
                     <br />
-                    <select id="BtnsAspectRatio" class="form-select text-light bg-dark">
+                    <select id="BtnAspectRatio" class="form-select text-light bg-dark">
                         <option value="0">default</option>
                         <option value="1">4:3</option>
                         <option value="2">16:9</option>
@@ -319,8 +319,8 @@ access to use this app.
                     <p>Video Controls:</p>
                     <br />
                     <select id="BtnVideoControls" class="form-select text-light bg-dark">
-                        <option value="Off">Off</option>
-                        <option value="On">On</option>
+                        <option value="off">off</option>
+                        <option value="on">on</option>
                     </select>
                     <br />
                 </div>