From ecf9b32070a3861ece287a8df2ba95d84cfc4a9e Mon Sep 17 00:00:00 2001
From: rainmakerv2 <30595646+jpau02@users.noreply.github.com>
Date: Wed, 5 Feb 2025 19:18:34 +0800
Subject: [PATCH] Set focus, refresh GUI after closing text editor, prevent no
 comma crash

---
 src/qt_gui/control_settings.cpp | 24 ++++++++++++++++++------
 1 file changed, 18 insertions(+), 6 deletions(-)

diff --git a/src/qt_gui/control_settings.cpp b/src/qt_gui/control_settings.cpp
index 0374d2049..e24a0fb96 100644
--- a/src/qt_gui/control_settings.cpp
+++ b/src/qt_gui/control_settings.cpp
@@ -16,6 +16,7 @@ ControlSettings::ControlSettings(std::shared_ptr<GameInfoClass> game_info_get, Q
 
     AddBoxItems();
     SetUIValuestoMappings();
+    ui->KBMButton->setFocus();
 
     connect(ui->buttonBox, &QDialogButtonBox::clicked, this, [this](QAbstractButton* button) {
         if (button == ui->buttonBox->button(QDialogButtonBox::Save)) {
@@ -31,6 +32,7 @@ ControlSettings::ControlSettings(std::shared_ptr<GameInfoClass> game_info_get, Q
     connect(ui->KBMButton, &QPushButton::clicked, this, [this] {
         auto KBMWindow = new EditorDialog(this);
         KBMWindow->exec();
+        SetUIValuestoMappings();
     });
     connect(ui->ProfileComboBox, &QComboBox::currentTextChanged, this, [this] {
         GetGameTitle();
@@ -416,14 +418,24 @@ void ControlSettings::SetUIValuestoMappings() {
                 RStickYExists = true;
             } else if (input_string.contains("leftjoystick")) {
                 std::size_t comma_pos = line.find(',');
-                int deadzonevalue = std::stoi(line.substr(comma_pos + 1));
-                ui->LeftDeadzoneSlider->setValue(deadzonevalue);
-                ui->LeftDeadzoneValue->setText(QString::number(deadzonevalue));
+                if (comma_pos != std::string::npos) {
+                    int deadzonevalue = std::stoi(line.substr(comma_pos + 1));
+                    ui->LeftDeadzoneSlider->setValue(deadzonevalue);
+                    ui->LeftDeadzoneValue->setText(QString::number(deadzonevalue));
+                } else {
+                    ui->LeftDeadzoneSlider->setValue(2);
+                    ui->LeftDeadzoneValue->setText("2");
+                }
             } else if (input_string.contains("rightjoystick")) {
                 std::size_t comma_pos = line.find(',');
-                int deadzonevalue = std::stoi(line.substr(comma_pos + 1));
-                ui->RightDeadzoneSlider->setValue(deadzonevalue);
-                ui->RightDeadzoneValue->setText(QString::number(deadzonevalue));
+                if (comma_pos != std::string::npos) {
+                    int deadzonevalue = std::stoi(line.substr(comma_pos + 1));
+                    ui->RightDeadzoneSlider->setValue(deadzonevalue);
+                    ui->RightDeadzoneValue->setText(QString::number(deadzonevalue));
+                } else {
+                    ui->RightDeadzoneSlider->setValue(2);
+                    ui->RightDeadzoneValue->setText("2");
+                }
             }
         }
     }