From 3ae2de52f47145767692e381b6e5ca599466ab0a Mon Sep 17 00:00:00 2001
From: sithlord48 <sithlord48@gmail.com>
Date: Wed, 26 Feb 2025 22:56:46 -0500
Subject: [PATCH] refactor: SettingsDialog set icons in ui where possible

---
 src/apps/deskflow-gui/dialogs/SettingsDialog.cpp | 5 -----
 src/apps/deskflow-gui/dialogs/SettingsDialog.ui  | 9 +++++++++
 2 files changed, 9 insertions(+), 5 deletions(-)

diff --git a/src/apps/deskflow-gui/dialogs/SettingsDialog.cpp b/src/apps/deskflow-gui/dialogs/SettingsDialog.cpp
index 0c1597681..cd29f718d 100644
--- a/src/apps/deskflow-gui/dialogs/SettingsDialog.cpp
+++ b/src/apps/deskflow-gui/dialogs/SettingsDialog.cpp
@@ -40,11 +40,6 @@ SettingsDialog::SettingsDialog(
   ui->comboTlsKeyLength->setItemIcon(1, QIcon::fromTheme(QStringLiteral("security-medium")));
   ui->comboTlsKeyLength->setItemIcon(2, QIcon::fromTheme(QIcon::ThemeIcon::SecurityHigh));
 
-  ui->btnTlsRegenCert->setIcon(QIcon::fromTheme(QIcon::ThemeIcon::ViewRefresh));
-
-  ui->btnTlsCertPath->setIcon(QIcon::fromTheme(QIcon::ThemeIcon::DocumentOpen));
-  ui->btnBrowseLog->setIcon(QIcon::fromTheme(QIcon::ThemeIcon::DocumentOpen));
-
   ui->rbIconMono->setIcon(QIcon::fromTheme(QStringLiteral("deskflow-symbolic")));
   ui->rbIconColorful->setIcon(QIcon::fromTheme(QStringLiteral("deskflow")));
 
diff --git a/src/apps/deskflow-gui/dialogs/SettingsDialog.ui b/src/apps/deskflow-gui/dialogs/SettingsDialog.ui
index 7714c7b74..f4b43beec 100644
--- a/src/apps/deskflow-gui/dialogs/SettingsDialog.ui
+++ b/src/apps/deskflow-gui/dialogs/SettingsDialog.ui
@@ -275,6 +275,9 @@
                <property name="text">
                 <string/>
                </property>
+               <property name="icon">
+                <iconset theme="QIcon::ThemeIcon::DocumentOpen"/>
+               </property>
                <property name="flat">
                 <bool>true</bool>
                </property>
@@ -306,6 +309,9 @@
               <property name="text">
                <string>Regenerate certificate</string>
               </property>
+              <property name="icon">
+               <iconset theme="QIcon::ThemeIcon::ViewRefresh"/>
+              </property>
              </widget>
             </item>
            </layout>
@@ -562,6 +568,9 @@
                <property name="text">
                 <string/>
                </property>
+               <property name="icon">
+                <iconset theme="QIcon::ThemeIcon::DocumentOpen"/>
+               </property>
                <property name="flat">
                 <bool>true</bool>
                </property>