Fixed the hide all button not doing anything in the menu bar

Author: Jonathan-Greve

SourceFiles/GuiGlobalConstants.h

@@ -98,6 +98,38 @@ class GuiGlobalConstants
 		is_window_controller_open = prev_is_window_controller_open;
 	}
 
+	// Method to set the hide_all state and update panels accordingly
+	static void SetHideAll(bool hide)
+	{
+		if (hide)
+		{
+			if (!hide_all) {
+				SaveCurrentStates();
+			}
+			hide_all = true;
+			is_dat_browser_open = false;
+			is_dat_browser_resizeable = false;
+			is_dat_browser_movable = false;
+			is_left_panel_open = false;
+			is_right_panel_open = false;
+			is_hex_editor_open = false;
+			is_text_panel_open = false;
+			is_audio_controller_open = false;
+			is_texture_panel_open = false;
+			is_picking_panel_open = false;
+			is_compare_panel_open = false;
+			is_custom_file_info_editor_open = false;
+			is_extract_panel_open = false;
+			is_byte_search_panel_open = false;
+			is_pathfinding_panel_open = false;
+		}
+		else
+		{
+			RestorePreviousStates();
+			hide_all = false;
+		}
+	}
+
 	// Method to reset all panels to default visibility
 	static void ResetToDefaults()
 	{

SourceFiles/draw_gui_window_controller.cpp

@@ -47,33 +47,10 @@ void draw_gui_window_controller()
 
 	// Checkbox to hide or show all windows
 	CheckAndResetHideAll();
-	if (ImGui::Checkbox("Hide All", &GuiGlobalConstants::hide_all))
+	bool temp_hide_all = GuiGlobalConstants::hide_all;
+	if (ImGui::Checkbox("Hide All", &temp_hide_all))
 	{
-		if (GuiGlobalConstants::hide_all)
-		{
-			GuiGlobalConstants::SaveCurrentStates();
-
-			GuiGlobalConstants::is_dat_browser_open = !GuiGlobalConstants::hide_all;
-			GuiGlobalConstants::is_dat_browser_movable = !GuiGlobalConstants::hide_all;
-			GuiGlobalConstants::is_dat_browser_resizeable = !GuiGlobalConstants::hide_all;
-			GuiGlobalConstants::is_dat_browser_open = !GuiGlobalConstants::hide_all;
-			GuiGlobalConstants::is_left_panel_open = !GuiGlobalConstants::hide_all;
-			GuiGlobalConstants::is_right_panel_open = !GuiGlobalConstants::hide_all;
-			GuiGlobalConstants::is_hex_editor_open = !GuiGlobalConstants::hide_all;
-			GuiGlobalConstants::is_text_panel_open = !GuiGlobalConstants::hide_all;
-			GuiGlobalConstants::is_audio_controller_open = !GuiGlobalConstants::hide_all;
-			GuiGlobalConstants::is_texture_panel_open = !GuiGlobalConstants::hide_all;
-			GuiGlobalConstants::is_picking_panel_open = !GuiGlobalConstants::hide_all;
-			GuiGlobalConstants::is_compare_panel_open = !GuiGlobalConstants::hide_all;
-			GuiGlobalConstants::is_custom_file_info_editor_open = !GuiGlobalConstants::hide_all;
-			GuiGlobalConstants::is_extract_panel_open = !GuiGlobalConstants::hide_all;
-			GuiGlobalConstants::is_byte_search_panel_open = !GuiGlobalConstants::hide_all;
-			GuiGlobalConstants::is_pathfinding_panel_open = !GuiGlobalConstants::hide_all;
-		}
-		else
-		{
-			GuiGlobalConstants::RestorePreviousStates();
-		}
+		GuiGlobalConstants::SetHideAll(temp_hide_all);
 		GuiGlobalConstants::SaveSettings();
 	}
 

SourceFiles/draw_ui.cpp

@@ -84,13 +84,7 @@ void draw_ui(std::map<int, std::unique_ptr<DATManager>>& dat_managers, int& dat_
 				}
 				ImGui::Separator();
 				if (ImGui::MenuItem("Hide All", NULL, GuiGlobalConstants::hide_all)) {
-					if (GuiGlobalConstants::hide_all) {
-						GuiGlobalConstants::RestorePreviousStates();
-						GuiGlobalConstants::hide_all = false;
-					} else {
-						GuiGlobalConstants::SaveCurrentStates();
-						GuiGlobalConstants::hide_all = true;
-					}
+					GuiGlobalConstants::SetHideAll(!GuiGlobalConstants::hide_all);
 					GuiGlobalConstants::SaveSettings();
 				}
 				ImGui::EndMenu();