From 862ac3fed864704c2a1cf50d2da059f4b108ea4c Mon Sep 17 00:00:00 2001
From: RossAsscends <124905043+RossAscends@users.noreply.github.com>
Date: Wed, 22 Mar 2023 22:47:10 +0900
Subject: [PATCH 1/2] right nav (now on top) lock and open/close preferences
 are now remembered between page reloads. lock prevents right nav drawer from
 closing when clicking away or when opening other drawers.

---
 public/index.html                  |  2 +-
 public/script.js                   | 30 +++++++-----------
 public/scripts/RossAscends-mods.js | 50 ++++++++++++++++++++++--------
 public/scripts/f-localStorage.js   | 22 +++++++------
 4 files changed, 62 insertions(+), 42 deletions(-)

diff --git a/public/index.html b/public/index.html
index 8d3e810c5..bf741ec93 100644
--- a/public/index.html
+++ b/public/index.html
@@ -784,7 +784,7 @@
 
         <div id="rightNavHolder" class="drawer" style="z-index:3001;">
             <div id="unimportantYes" class="drawer-toggle drawer-header">
-                <div class="drawer-icon icon-idcard closedIcon"></div>
+                <div id="rightNavDrawerIcon" class="drawer-icon icon-idcard closedIcon"></div>
             </div>
             <nav id="right-nav-panel" class="drawer-content closedDrawer">
                 <div id="right-nav-panel-tabs">
diff --git a/public/script.js b/public/script.js
index 46c316fe9..aeca0cb57 100644
--- a/public/script.js
+++ b/public/script.js
@@ -4039,24 +4039,18 @@ $(document).ready(function () {
     });
 
     $('.drawer-toggle').click(function () {
-        //$('#top-settings-holder').find('.open').slideToggle(0, "swing");
         var icon = $(this).find('.drawer-icon');
         var drawer = $(this).parent().find('.drawer-content');
         var drawerWasOpenAlready = $(this).parent().find('.drawer-content').hasClass('openDrawer');
-        var iconWasOpenAlready = $(this).find('.drawer-icon').hasClass('openicon');
-        console.log('Draw Open? ' + drawerWasOpenAlready + ' Icon Open? ' + iconWasOpenAlready);
 
         if (!drawerWasOpenAlready) {
-            $('.openDrawer').slideToggle(200, "swing")                                      //close all open drawers
-            $('.openIcon').toggleClass('closedIcon openIcon');                  //set all icons   to closed state
-            $('.openDrawer').toggleClass('closedDrawer openDrawer');            //set all drawers to closed state
-            console.log('drawer was closed, so opening');
+            $('.openDrawer').not('.pinnedOpen').slideToggle(200, "swing");
+            $('.openIcon').toggleClass('closedIcon openIcon');
+            $('.openDrawer').not('.pinnedOpen').toggleClass('closedDrawer openDrawer');
             icon.toggleClass('openIcon closedIcon');
-            drawer.toggleClass('openDrawer closedDrawer');                      //mark selected drawer as open if closed
-            $(this).closest('.drawer').find('.drawer-content').slideToggle(200, "swing");   //open selected drawer
-
+            drawer.toggleClass('openDrawer closedDrawer');
+            $(this).closest('.drawer').find('.drawer-content').slideToggle(200, "swing");
         } else if (drawerWasOpenAlready) {
-            console.log('found the drawer was already open, so closing');
             icon.toggleClass('closedIcon openIcon');
             $('.openDrawer').slideToggle(200, "swing");
             drawer.toggleClass('closedDrawer openDrawer');
@@ -4066,14 +4060,14 @@ $(document).ready(function () {
     $("html").click(function (e) {
         var clickTarget = $(e.target);
         var targetParentHasOpenDrawer = clickTarget.parents('.openDrawer').length;
-        console.log('clicking ' + clickTarget + ' parents have OpenDrawer? ' + targetParentHasOpenDrawer);
-        if (clickTarget.hasClass('drawer-icon') == false && !clickTarget.hasClass('openDrawer')) {                         // don't run this check on drawer icons, as causes insta-closes
-            if (jQuery.find('.openDrawer').length !== 0) {                          // only run if opendrawers actually exist
-                if (targetParentHasOpenDrawer === 0) {                              // check if clicked target is NOT part of an OpenDrawer, and if so...
+        if (clickTarget.hasClass('drawer-icon') == false && !clickTarget.hasClass('openDrawer')) {
+            if (jQuery.find('.openDrawer').length !== 0) {
+                if (targetParentHasOpenDrawer === 0) {
+                    console.log($('.openDrawer').not('.pinnedOpen').length);
+                    $('.openDrawer').not('.pinnedOpen').slideToggle(200, "swing");
+                    $('.openIcon').toggleClass('closedIcon openIcon');
+                    $('.openDrawer').not('.pinnedOpen').toggleClass('closedDrawer openDrawer');
 
-                    $('.openDrawer').slideToggle(200, "swing")                          //close all open drawers
-                    $('.openIcon').toggleClass('closedIcon openIcon');                  //set all icons   to closed state
-                    $('.openDrawer').toggleClass('closedDrawer openDrawer');            //set all drawers to closed state
                 }
             }
 
diff --git a/public/scripts/RossAscends-mods.js b/public/scripts/RossAscends-mods.js
index f361f28f8..a6789c769 100644
--- a/public/scripts/RossAscends-mods.js
+++ b/public/scripts/RossAscends-mods.js
@@ -262,9 +262,7 @@ function isUrlOrAPIKey(string) {
 
 $("document").ready(function () {
 
-	// read the state of Nav Lock and whether the nav was open or not before page load.
-	$(PanelPin).prop('checked', LoadLocalBool("NavLockOn"));
-	if (LoadLocalBool("NavLockOn") == true) { $(NavToggle).prop("checked", LoadLocalBool("NavOpened")); }
+
 	// read the state of AutoConnect and AutoLoadChat.
 	$(AutoConnectCheckbox).prop("checked", LoadLocalBool("AutoConnectEnabled"));
 	$(AutoLoadChatCheckbox).prop("checked", LoadLocalBool("AutoLoadChatEnabled"));
@@ -275,23 +273,49 @@ $("document").ready(function () {
 	$("#main_api").change(function () { RA_autoconnect(); });
 	$("#api_button").click(function () { setTimeout(RA_checkOnlineStatus, 100); });
 
-	//save NavLock prefs and record state of the Nav being open or closed
-	$(RightNavPanel).on("change", function () {
-		if ($(RightNavPanel).hasClass('openDrawer')) {
-			SaveLocal("NavOpened", true);
-		} else { SaveLocal("NavOpened", false); }
+	//toggle pin class when lock toggle clicked
+	$(PanelPin).on("click", function () {
+		SaveLocal("NavLockOn", $(PanelPin).prop("checked"));
+		if ($(PanelPin).prop("checked") == true) {
+			console.log('adding pin class to right nav');
+			$(RightNavPanel).addClass('pinnedOpen');
+		} else {
+			console.log('removing pin class from right nav');
+			$(RightNavPanel).removeClass('pinnedOpen');
+		}
 	});
 
-	$(PanelPin).on("change", function () { SaveLocal("NavLockOn", $(PanelPin).prop("checked")); });
+	// read the state of Nav Lock and apply to rightnav classlist
+	$(PanelPin).prop('checked', LoadLocalBool("NavLockOn"));
+	if (LoadLocalBool("NavLockOn") == true) {
+		console.log('setting pin class via local var');
+		$(RightNavPanel).addClass('pinnedOpen');
+	}
+	if ($(PanelPin).prop('checked' == true)) {
+		console.log('setting pin class via checkbox state');
+		$(RightNavPanel).addClass('pinnedOpen');
+	}
+
+	//save state of nav being open or closed
+	$("#rightNavDrawerIcon").on("click", function () {
+		if (!$("#rightNavDrawerIcon").hasClass('openIcon')) {
+			SaveLocal('NavOpened', 'true');
+		} else { SaveLocal('NavOpened', 'false'); }
+	});
+
+	if (LoadLocalBool("NavLockOn") == true && LoadLocalBool("NavOpened") == true) {
+		$("#rightNavDrawerIcon").click();
+	} else {
+		console.log('didnt see reason to open nav on load: ' +
+			LoadLocalBool("NavLockOn")
+			+ ' nav open pref' +
+			LoadLocalBool("NavOpened" == true));
+	}
 
 	//save AutoConnect and AutoLoadChat prefs
 	$(AutoConnectCheckbox).on("change", function () { SaveLocal("AutoConnectEnabled", $(AutoConnectCheckbox).prop("checked")); });
 	$(AutoLoadChatCheckbox).on("change", function () { SaveLocal("AutoLoadChatEnabled", $(AutoLoadChatCheckbox).prop("checked")); });
 
-	$("#rm_button_extensions").click(function () {
-		SaveLocal('SelectedNavTab', 'rm_button_extensions');
-	});
-	$("#rm_button_settings").click(function () { SaveLocal('SelectedNavTab', 'rm_button_settings'); });
 	$(SelectedCharacterTab).click(function () { SaveLocal('SelectedNavTab', 'rm_button_selected_ch'); });
 	$("#rm_button_characters").click(function () {			//if char list is clicked, in addition to saving it...
 		SaveLocal('SelectedNavTab', 'rm_button_characters');
diff --git a/public/scripts/f-localStorage.js b/public/scripts/f-localStorage.js
index 52bc6a7bb..de387ff14 100644
--- a/public/scripts/f-localStorage.js
+++ b/public/scripts/f-localStorage.js
@@ -2,24 +2,26 @@
 
 export function SaveLocal(target, val) {
 	localStorage.setItem(target, val);
-	console.log('SaveLocal -- '+target+' : '+val);
-  }
+	console.log('SaveLocal -- ' + target + ' : ' + val);
+}
 export function LoadLocal(target) {
+	console.log('LoadLocal -- ' + target);
 	return localStorage.getItem(target);
-  }
-export function LoadLocalBool(target){
-	  let result = localStorage.getItem(target) === 'true';
-	  return result;
-  }
+
+}
+export function LoadLocalBool(target) {
+	let result = localStorage.getItem(target) === 'true';
+	return result;
+}
 export function CheckLocal() {
 	console.log("----------local storage---------");
 	var i;
 	for (i = 0; i < localStorage.length; i++) {
-	  console.log(localStorage.key(i) +" : " +localStorage.getItem(localStorage.key(i)));
+		console.log(localStorage.key(i) + " : " + localStorage.getItem(localStorage.key(i)));
 	}
 	console.log("------------------------------");
-  }
+}
 
-export function ClearLocal() {localStorage.clear();console.log('Removed All Local Storage');}
+export function ClearLocal() { localStorage.clear(); console.log('Removed All Local Storage'); }
 
 /////////////////////////////////////////////////////////////////////////
\ No newline at end of file

From b9da16ed9501cd3e67294f65ef4492e8d633cc0e Mon Sep 17 00:00:00 2001
From: RossAsscends <124905043+RossAscends@users.noreply.github.com>
Date: Wed, 22 Mar 2023 23:06:18 +0900
Subject: [PATCH 2/2] fixed regenerate hotkey to work with groups - it now
 clicks the regenerate button - previously it directly called Regenerate()

---
 public/scripts/RossAscends-mods.js | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/public/scripts/RossAscends-mods.js b/public/scripts/RossAscends-mods.js
index a6789c769..7e462ba8c 100644
--- a/public/scripts/RossAscends-mods.js
+++ b/public/scripts/RossAscends-mods.js
@@ -361,7 +361,10 @@ $("document").ready(function () {
 			// Ctrl+Enter for Regeneration Last Response
 			if (is_send_press == false) {
 
-				Generate("regenerate");
+				$('#option_regenerate').click();
+				$('#options').hide();
+				//setTimeout(function () { $('#chat').click(); }, 50) //needed to remove the options menu popping up..
+				//Generate("regenerate");
 			}
 		}
 		if (event.ctrlKey && event.key == "ArrowUp") {