From 1e48654606d053f176d94f896da2ad79f90ec38a Mon Sep 17 00:00:00 2001
From: Peter Albrecht <albrecpe@gmail.com>
Date: Mon, 29 Jan 2024 11:43:21 +0100
Subject: [PATCH] Add icon

---
 .../skills_gui/skill_details/ProfileMenuWidget.cpp     | 10 ++++++++++
 .../skills_gui/skill_details/ProfileMenuWidget.h       |  4 ++--
 .../skills_gui/skill_details/SkillDetailsGroupBox.cpp  |  6 ++++++
 .../skills_gui/skill_details/SkillDetailsGroupBox.h    |  2 +-
 4 files changed, 19 insertions(+), 3 deletions(-)

diff --git a/source/RobotAPI/libraries/skills_gui/skill_details/ProfileMenuWidget.cpp b/source/RobotAPI/libraries/skills_gui/skill_details/ProfileMenuWidget.cpp
index 20295978a..ac56b71de 100644
--- a/source/RobotAPI/libraries/skills_gui/skill_details/ProfileMenuWidget.cpp
+++ b/source/RobotAPI/libraries/skills_gui/skill_details/ProfileMenuWidget.cpp
@@ -7,6 +7,14 @@
 namespace armarx::skills::gui
 {
 
+    QIcon
+    getIcon(const std::string name)
+    {
+        QPixmap pix(QString::fromStdString(":icons/" + name));
+        QIcon icon(pix);
+        return icon;
+    }
+
     void
     ProfileMenuWidget::setupUi()
     {
@@ -33,7 +41,9 @@ namespace armarx::skills::gui
         // Text
         setArgsFromClipboard->setText(QString::fromStdString(SET_ARGS_BUTTON_TEXT));
         copyArgsToClipboard->setText(QString::fromStdString(COPY_ARGS_BUTTON_TEXT));
+        copyArgsToClipboard->setIcon(getIcon("edit-copy-4.svg"));
         resetArgsToProfile->setText(QString::fromStdString(RESET_ARGS_BUTTON_TEXT));
+        resetArgsToProfile->setIcon(getIcon("refresh-black.svg"));
         profileSelector->addItem(QString::fromStdString(DEFAULT_PROFILE_TEXT));
         profileSelector->setDisabled(true);
         profileSelector->setToolTip(QString::fromStdString(PROFILE_NOT_IMPLEMENTED));
diff --git a/source/RobotAPI/libraries/skills_gui/skill_details/ProfileMenuWidget.h b/source/RobotAPI/libraries/skills_gui/skill_details/ProfileMenuWidget.h
index 32dc1f278..3046b18b8 100644
--- a/source/RobotAPI/libraries/skills_gui/skill_details/ProfileMenuWidget.h
+++ b/source/RobotAPI/libraries/skills_gui/skill_details/ProfileMenuWidget.h
@@ -14,8 +14,8 @@ namespace armarx::skills::gui
         Q_OBJECT
     public:
         static const constexpr char* SET_ARGS_BUTTON_TEXT = "Set Args from Clipboard";
-        static const constexpr char* COPY_ARGS_BUTTON_TEXT = "Copy Args to Clipboard";
-        static const constexpr char* RESET_ARGS_BUTTON_TEXT = "Reset Args to Profile";
+        static const constexpr char* COPY_ARGS_BUTTON_TEXT = " Copy Args to Clipboard";
+        static const constexpr char* RESET_ARGS_BUTTON_TEXT = " Reset Args to Profile";
         static const constexpr char* DEFAULT_PROFILE_TEXT = "<No Profile selected. Using root>";
         static const constexpr char* PROFILE_NOT_IMPLEMENTED =
             "Profiles other than the root profile are currently not supported.";
diff --git a/source/RobotAPI/libraries/skills_gui/skill_details/SkillDetailsGroupBox.cpp b/source/RobotAPI/libraries/skills_gui/skill_details/SkillDetailsGroupBox.cpp
index b54d55c8a..137140d25 100644
--- a/source/RobotAPI/libraries/skills_gui/skill_details/SkillDetailsGroupBox.cpp
+++ b/source/RobotAPI/libraries/skills_gui/skill_details/SkillDetailsGroupBox.cpp
@@ -113,6 +113,12 @@ namespace armarx::skills::gui
         // Text
         this->executeSkillButton->setText(
             QString::fromStdString(SkillDetailGroupBox::EXECUTE_SKILL_BUTTON_TEXT));
+        {
+            QPixmap pix(QString::fromStdString(":icons/run.svg"));
+            QIcon icon(pix);
+            this->executeSkillButton->setIcon(icon);
+        }
+
 
         connectSignals();
     }
diff --git a/source/RobotAPI/libraries/skills_gui/skill_details/SkillDetailsGroupBox.h b/source/RobotAPI/libraries/skills_gui/skill_details/SkillDetailsGroupBox.h
index 918fa2ec3..f777490cf 100644
--- a/source/RobotAPI/libraries/skills_gui/skill_details/SkillDetailsGroupBox.h
+++ b/source/RobotAPI/libraries/skills_gui/skill_details/SkillDetailsGroupBox.h
@@ -19,7 +19,7 @@ namespace armarx::skills::gui
     {
         Q_OBJECT
     public:
-        static const constexpr char* EXECUTE_SKILL_BUTTON_TEXT = "Request Execution";
+        static const constexpr char* EXECUTE_SKILL_BUTTON_TEXT = " Request Execution";
         static const constexpr char* GROUP_BOX_TITLE = "Skill Description";
 
         SkillDetailGroupBox(std::shared_ptr<SkillManagerWrapper> _memory,
-- 
GitLab