From 1b13b94095d24feba4461aac3c5071861ccc9cf5 Mon Sep 17 00:00:00 2001
From: Unknown <studienricky89@googlemail.com>
Date: Sun, 3 Dec 2017 18:30:41 +0100
Subject: [PATCH] Qt: Fix icon size actions

---
 rpcs3/rpcs3qt/main_window.cpp | 46 +++++++++++++++++++++--------------
 rpcs3/rpcs3qt/main_window.h   |  1 +
 2 files changed, 29 insertions(+), 18 deletions(-)

diff --git a/rpcs3/rpcs3qt/main_window.cpp b/rpcs3/rpcs3qt/main_window.cpp
index a951574d27..6c22ae9deb 100644
--- a/rpcs3/rpcs3qt/main_window.cpp
+++ b/rpcs3/rpcs3qt/main_window.cpp
@@ -1109,7 +1109,7 @@ void main_window::CreateConnects()
 
 	auto openSettings = [=](int tabIndex)
 	{
-		settings_dialog dlg(guiSettings, emuSettings,  tabIndex, this);
+		settings_dialog dlg(guiSettings, emuSettings, tabIndex, this);
 		connect(&dlg, &settings_dialog::GuiSettingsSaveRequest, this, &main_window::SaveWindowState);
 		connect(&dlg, &settings_dialog::GuiSettingsSyncRequest, this, &main_window::ConfigureGuiFromSettings);
 		connect(&dlg, &settings_dialog::GuiStylesheetRequest, this, &main_window::RequestGlobalStylesheetChange);
@@ -1270,21 +1270,22 @@ void main_window::CreateConnects()
 	{
 		int index;
 
-		if (act == ui->setIconSizeTinyAct) index = 0;
-		else if (act == ui->setIconSizeSmallAct) index = gui::get_Index(gui::gl_icon_size_small);
-		else if (act == ui->setIconSizeMediumAct) index = gui::get_Index(gui::gl_icon_size_medium);
-		else index = gui::gl_max_slider_pos;
+		if (act == ui->setIconSizeTinyAct)
+			index = 0;
+		else if (act == ui->setIconSizeSmallAct)
+			index = gui::get_Index(gui::gl_icon_size_small);
+		else if (act == ui->setIconSizeMediumAct)
+			index = gui::get_Index(gui::gl_icon_size_medium);
+		else
+			index = gui::gl_max_slider_pos;
 
+		m_save_slider_pos = true;
 		resizeIcons(index);
 	});
 
 	connect (m_gameListFrame, &game_list_frame::RequestIconSizeActSet, [=](const int& idx)
 	{
-		if (idx < gui::get_Index((gui::gl_icon_size_small + gui::gl_icon_size_min) / 2)) ui->setIconSizeTinyAct->setChecked(true);
-		else if (idx < gui::get_Index((gui::gl_icon_size_medium + gui::gl_icon_size_small) / 2)) ui->setIconSizeSmallAct->setChecked(true);
-		else if (idx < gui::get_Index((gui::gl_icon_size_max + gui::gl_icon_size_medium) / 2)) ui->setIconSizeMediumAct->setChecked(true);
-		else ui->setIconSizeLargeAct->setChecked(true);
-
+		SetIconSizeActions(idx);
 		resizeIcons(idx);
 	});
 
@@ -1469,16 +1470,13 @@ void main_window::ConfigureGuiFromSettings(bool configure_all)
 	ui->showCatUnknownAct->setChecked(guiSettings->GetCategoryVisibility(Category::Unknown_Cat));
 	ui->showCatOtherAct->setChecked(guiSettings->GetCategoryVisibility(Category::Others));
 
-	int idx = guiSettings->GetValue(gui::gl_iconSize).toInt();
-	int index = gui::gl_max_slider_pos / 4;
-	if (idx < index) ui->setIconSizeTinyAct->setChecked(true);
-	else if (idx < index * 2) ui->setIconSizeSmallAct->setChecked(true);
-	else if (idx < index * 3) ui->setIconSizeMediumAct->setChecked(true);
-	else ui->setIconSizeLargeAct->setChecked(true);
+	SetIconSizeActions(guiSettings->GetValue(gui::gl_iconSize).toInt());
 
 	bool isListMode = guiSettings->GetValue(gui::gl_listMode).toBool();
-	if (isListMode) ui->setlistModeListAct->setChecked(true);
-	else ui->setlistModeGridAct->setChecked(true);
+	if (isListMode)
+		ui->setlistModeListAct->setChecked(true);
+	else
+		ui->setlistModeGridAct->setChecked(true);
 	m_categoryVisibleActGroup->setEnabled(isListMode);
 
 	if (configure_all)
@@ -1491,6 +1489,18 @@ void main_window::ConfigureGuiFromSettings(bool configure_all)
 	}
 }
 
+void main_window::SetIconSizeActions(int idx)
+{
+	if (idx < gui::get_Index((gui::gl_icon_size_small + gui::gl_icon_size_min) / 2))
+		ui->setIconSizeTinyAct->setChecked(true);
+	else if (idx < gui::get_Index((gui::gl_icon_size_medium + gui::gl_icon_size_small) / 2))
+		ui->setIconSizeSmallAct->setChecked(true);
+	else if (idx < gui::get_Index((gui::gl_icon_size_max + gui::gl_icon_size_medium) / 2))
+		ui->setIconSizeMediumAct->setChecked(true);
+	else
+		ui->setIconSizeLargeAct->setChecked(true);
+}
+
 void main_window::keyPressEvent(QKeyEvent *keyEvent)
 {
 	if (((keyEvent->modifiers() & Qt::AltModifier) && keyEvent->key() == Qt::Key_Return) || (isFullScreen() && keyEvent->key() == Qt::Key_Escape))
diff --git a/rpcs3/rpcs3qt/main_window.h b/rpcs3/rpcs3qt/main_window.h
index 8a731142fc..be94545c81 100644
--- a/rpcs3/rpcs3qt/main_window.h
+++ b/rpcs3/rpcs3qt/main_window.h
@@ -91,6 +91,7 @@ private Q_SLOTS:
 
 	void SaveWindowState();
 	void ConfigureGuiFromSettings(bool configure_all = false);
+	void SetIconSizeActions(int idx);
 
 protected:
 	void closeEvent(QCloseEvent *event) override;