[desktop] fix random qt annoyances (#226)

Removes that silly QLayout message and improves consistency thereof for
other QLayouts

Still work to be done but this is all that's needed rn

Signed-off-by: crueter <crueter@eden-emu.dev>

Reviewed-on: https://git.eden-emu.dev/eden-emu/eden/pulls/226
Reviewed-by: Shinmegumi <shinmegumi@eden-emu.dev>

src/yuzu/configuration/configure.ui

@@ -17,7 +17,7 @@
    </size>
   </property>
   <property name="windowTitle">
-   <string>eden Configuration</string>
+   <string>Eden Configuration</string>
   </property>
   <layout class="QVBoxLayout" name="verticalLayout">
    <item>

src/yuzu/configuration/configure_input.cpp

@@ -1,3 +1,6 @@
+// SPDX-FileCopyrightText: Copyright 2025 Eden Emulator Project
+// SPDX-License-Identifier: GPL-3.0-or-later
+
 // SPDX-FileCopyrightText: 2016 Citra Emulator Project
 // SPDX-License-Identifier: GPL-2.0-or-later
 
@@ -98,8 +101,8 @@ void ConfigureInput::Initialize(InputCommon::InputSubsystem* input_subsystem,
     };
 
     for (std::size_t i = 0; i < player_tabs.size(); ++i) {
-        player_tabs[i]->setLayout(new QHBoxLayout(player_tabs[i]));
-        player_tabs[i]->layout()->addWidget(player_controllers[i]);
+        QHBoxLayout *tab_layout = new QHBoxLayout(player_tabs[i]);
+        tab_layout->addWidget(player_controllers[i]);
         connect(player_controllers[i], &ConfigureInputPlayer::Connected, [this, i](bool checked) {
             // Ensures that connecting a controller changes the number of players
             if (connected_controller_checkboxes[i]->isChecked() != checked) {
@@ -138,8 +141,9 @@ void ConfigureInput::Initialize(InputCommon::InputSubsystem* input_subsystem,
             [this](bool is_handheld) { UpdateDockedState(is_handheld); });
 
     advanced = new ConfigureInputAdvanced(hid_core, this);
-    ui->tabAdvanced->setLayout(new QHBoxLayout(ui->tabAdvanced));
-    ui->tabAdvanced->layout()->addWidget(advanced);
+
+    QHBoxLayout* advancedLayout = new QHBoxLayout(ui->tabAdvanced);
+    advancedLayout->addWidget(advanced);
 
     connect(advanced, &ConfigureInputAdvanced::CallDebugControllerDialog,
             [this, input_subsystem, &hid_core, is_powered_on] {

src/yuzu/configuration/configure_input_player.cpp

@@ -1,10 +1,12 @@
+// SPDX-FileCopyrightText: Copyright 2025 Eden Emulator Project
+// SPDX-License-Identifier: GPL-3.0-or-later
+
 // SPDX-FileCopyrightText: 2016 Citra Emulator Project
 // SPDX-License-Identifier: GPL-2.0-or-later
 
 #include <algorithm>
 #include <memory>
 #include <utility>
-#include <QGridLayout>
 #include <QInputDialog>
 #include <QMenu>
 #include <QMessageBox>
@@ -13,7 +15,6 @@
 #include "common/assert.h"
 #include "common/param_package.h"
 #include "configuration/qt_config.h"
-#include "frontend_common/config.h"
 #include "hid_core/frontend/emulated_controller.h"
 #include "hid_core/hid_core.h"
 #include "hid_core/hid_types.h"

src/yuzu/configuration/configure_motion_touch.cpp

@@ -1,3 +1,6 @@
+// SPDX-FileCopyrightText: Copyright 2025 Eden Emulator Project
+// SPDX-License-Identifier: GPL-3.0-or-later
+
 // SPDX-FileCopyrightText: 2018 Citra Emulator Project
 // SPDX-License-Identifier: GPL-2.0-or-later
 
@@ -19,7 +22,7 @@
 CalibrationConfigurationDialog::CalibrationConfigurationDialog(QWidget* parent,
                                                                const std::string& host, u16 port)
     : QDialog(parent) {
-    layout = new QVBoxLayout;
+    layout = new QVBoxLayout(this);
     status_label = new QLabel(tr("Communicating with the server..."));
     cancel_button = new QPushButton(tr("Cancel"));
     connect(cancel_button, &QPushButton::clicked, this, [this] {
@@ -30,7 +33,6 @@ CalibrationConfigurationDialog::CalibrationConfigurationDialog(QWidget* parent,
     });
     layout->addWidget(status_label);
     layout->addWidget(cancel_button);
-    setLayout(layout);
 
     using namespace InputCommon::CemuhookUDP;
     job = std::make_unique<CalibrationConfigurationJob>(

src/yuzu/configuration/configure_profile_manager.cpp

@@ -418,7 +418,7 @@ bool ConfigureProfileManager::LoadAvatarData() {
 ConfigureProfileManagerAvatarDialog::ConfigureProfileManagerAvatarDialog(QWidget* parent)
     : QDialog{parent}, avatar_list{new QListWidget(this)}, bg_color_button{new QPushButton(this)} {
     auto* main_layout = new QVBoxLayout(this);
-    auto* button_layout = new QHBoxLayout(this);
+    auto* button_layout = new QHBoxLayout();
     auto* select_button = new QPushButton(tr("Select"), this);
     auto* cancel_button = new QPushButton(tr("Cancel"), this);
     auto* bg_color_label = new QLabel(tr("Background Color"), this);
@@ -442,7 +442,6 @@ ConfigureProfileManagerAvatarDialog::ConfigureProfileManagerAvatarDialog(QWidget
     button_layout->addWidget(select_button);
     button_layout->addWidget(cancel_button);
 
-    this->setLayout(main_layout);
     this->setWindowTitle(tr("Select Firmware Avatar"));
     main_layout->addWidget(avatar_list);
     main_layout->addLayout(button_layout);
@@ -654,4 +653,4 @@ std::vector<uint8_t> ConfigureProfileManager::DecompressYaz0(const FileSys::Virt
     }
 
     return output;
-}
+}

src/yuzu/configuration/configure_profile_manager.h

@@ -12,6 +12,7 @@
 #include <QDialog>
 #include <QList>
 #include <QWidget>
+#include "core/file_sys/vfs/vfs_types.h"
 
 namespace Common {
 struct UUID;