From f06fa656b81ab5039cc0fe9b1e0f6b7cb4ab0476 Mon Sep 17 00:00:00 2001
From: PYRET1C <88980503+PYRET1C@users.noreply.github.com>
Date: Wed, 12 Apr 2023 22:32:17 +0530
Subject: [PATCH] This commit added a browse button the wire for image addition
 functionality.

---
 .../src/gui/TheWire/PulseAddDialog.cpp        |  32 +++-
 .../src/gui/TheWire/PulseAddDialog.h          |   1 +
 .../src/gui/TheWire/PulseAddDialog.ui         | 150 +++++++++++-------
 3 files changed, 123 insertions(+), 60 deletions(-)

diff --git a/retroshare-gui/src/gui/TheWire/PulseAddDialog.cpp b/retroshare-gui/src/gui/TheWire/PulseAddDialog.cpp
index 15fe997a6..84a8e322c 100644
--- a/retroshare-gui/src/gui/TheWire/PulseAddDialog.cpp
+++ b/retroshare-gui/src/gui/TheWire/PulseAddDialog.cpp
@@ -20,6 +20,7 @@
 
 #include <iostream>
 #include <QtGui>
+#include <QFileDialog>
 
 #include "PulseReply.h"
 #include "gui/gxs/GxsIdDetails.h"
@@ -42,9 +43,15 @@ PulseAddDialog::PulseAddDialog(QWidget *parent)
 	connect(ui.textEdit_Pulse, SIGNAL( textChanged( void ) ), this, SLOT( pulseTextChanged( void ) ) );
 	connect(ui.pushButton_picture, SIGNAL(clicked()), this, SLOT( toggle()));
 
+    // this connection is from browse push button to the slot function onBrowseButtonClicked()
+    connect(ui.pushButton_Browse, SIGNAL(clicked()), this, SLOT( onBrowseButtonClicked()));
+
 	ui.pushButton_picture->setIcon(FilesDefs::getIconFromQtResourcePath(QString(":/icons/png/photo.png")));
 	ui.frame_picture->hide();
 
+    // initially hiding the browse button as the attach image button is not pressed
+    ui.frame_PictureBrowse->hide();
+
 	setAcceptDrops(true);
 }
 
@@ -153,8 +160,12 @@ void PulseAddDialog::cleanup()
 	ui.label_image4->clear();
 	ui.label_image4->setText(tr("Drag and Drop Image"));
 
-	// Hide Drag & Drop Frame
-	ui.frame_picture->hide();
+    ui.lineEdit_FilePath->clear();
+
+    // Hide Drag & Drop Frame and the browse frame
+    ui.frame_picture->hide();
+    ui.frame_PictureBrowse->hide();
+
 	ui.pushButton_picture->setChecked(false);
 }
 
@@ -510,12 +521,29 @@ void PulseAddDialog::toggle()
 {
 	if (ui.pushButton_picture->isChecked())
 	{
+        // Show the input methods (drag and drop field and the browse button)
 		ui.frame_picture->show();
+        ui.frame_PictureBrowse->show();
+
 		ui.pushButton_picture->setToolTip(tr("Hide Pictures"));
 	}
 	else
 	{
+        // Hide the input methods (drag and drop field and the browse button)
 		ui.frame_picture->hide();
+        ui.frame_PictureBrowse->hide();
+
 		ui.pushButton_picture->setToolTip(tr("Add Pictures"));
 	}
 }
+
+// Function to get the file dialog for the browse button
+void PulseAddDialog::onBrowseButtonClicked()
+{
+    QString filePath = QFileDialog::getOpenFileName(this, tr("Select image file"), QString(), tr("Image files (*.png *.jpg *.jpeg *.bmp *.gif)"));
+    if (!filePath.isEmpty()) {
+        ui.lineEdit_FilePath->setText(filePath);
+        addImage(filePath);
+    }
+}
+
diff --git a/retroshare-gui/src/gui/TheWire/PulseAddDialog.h b/retroshare-gui/src/gui/TheWire/PulseAddDialog.h
index 258efb736..69007b025 100644
--- a/retroshare-gui/src/gui/TheWire/PulseAddDialog.h
+++ b/retroshare-gui/src/gui/TheWire/PulseAddDialog.h
@@ -53,6 +53,7 @@ private slots:
 	void clearDialog();
 	void pulseTextChanged();
 	void toggle();
+    void onBrowseButtonClicked();
 
 private:
 	// OLD VERSIONs, private now.
diff --git a/retroshare-gui/src/gui/TheWire/PulseAddDialog.ui b/retroshare-gui/src/gui/TheWire/PulseAddDialog.ui
index a731ec3b0..5ef30ef19 100644
--- a/retroshare-gui/src/gui/TheWire/PulseAddDialog.ui
+++ b/retroshare-gui/src/gui/TheWire/PulseAddDialog.ui
@@ -7,20 +7,20 @@
     <x>0</x>
     <y>0</y>
     <width>720</width>
-    <height>449</height>
+    <height>513</height>
    </rect>
   </property>
   <property name="windowTitle">
    <string/>
   </property>
-  <layout class="QGridLayout" name="PulseAddDialog_GL">
-   <property name="topMargin">
-    <number>0</number>
-   </property>
+  <layout class="QVBoxLayout" name="verticalLayout_2">
    <property name="spacing">
     <number>6</number>
    </property>
-   <item row="0" column="0">
+   <property name="topMargin">
+    <number>0</number>
+   </property>
+   <item>
     <widget class="QFrame" name="frame_from">
      <property name="sizePolicy">
       <sizepolicy hsizetype="Expanding" vsizetype="Fixed">
@@ -52,7 +52,6 @@
         <property name="font">
          <font>
           <pointsize>12</pointsize>
-          <weight>75</weight>
           <bold>true</bold>
          </font>
         </property>
@@ -118,7 +117,7 @@
      </layout>
     </widget>
    </item>
-   <item row="1" column="0">
+   <item>
     <widget class="QFrame" name="frame_reply">
      <layout class="QVBoxLayout" name="frame_reply_VL">
       <property name="leftMargin">
@@ -232,18 +231,9 @@
      </layout>
     </widget>
    </item>
-   <item row="2" column="0">
+   <item>
     <widget class="QFrame" name="frame_input">
-     <layout class="QVBoxLayout" name="frame_input_VL">
-      <property name="leftMargin">
-       <number>0</number>
-      </property>
-      <property name="topMargin">
-       <number>0</number>
-      </property>
-      <property name="rightMargin">
-       <number>0</number>
-      </property>
+     <layout class="QVBoxLayout" name="verticalLayout">
       <item>
        <widget class="QTextEdit" name="textEdit_Pulse">
         <property name="font">
@@ -262,7 +252,7 @@
         <property name="minimumSize">
          <size>
           <width>0</width>
-          <height>100</height>
+          <height>80</height>
          </size>
         </property>
         <property name="frameShape">
@@ -271,29 +261,13 @@
         <property name="frameShadow">
          <enum>QFrame::Raised</enum>
         </property>
-        <layout class="QGridLayout" name="frame_picture_GL">
-         <item row="0" column="0">
-          <widget class="QLabel" name="label_image1">
-           <property name="minimumSize">
-            <size>
-             <width>80</width>
-             <height>80</height>
-            </size>
-           </property>
-           <property name="text">
-            <string>Drag and Drop Image</string>
-           </property>
-           <property name="alignment">
-            <set>Qt::AlignCenter</set>
-           </property>
-          </widget>
-         </item>
+        <layout class="QGridLayout" name="gridLayout">
          <item row="0" column="1">
           <widget class="QLabel" name="label_image2">
            <property name="minimumSize">
             <size>
-             <width>80</width>
-             <height>80</height>
+             <width>40</width>
+             <height>40</height>
             </size>
            </property>
            <property name="text">
@@ -308,8 +282,24 @@
           <widget class="QLabel" name="label_image3">
            <property name="minimumSize">
             <size>
-             <width>80</width>
-             <height>80</height>
+             <width>40</width>
+             <height>40</height>
+            </size>
+           </property>
+           <property name="text">
+            <string>Drag and Drop Image</string>
+           </property>
+           <property name="alignment">
+            <set>Qt::AlignCenter</set>
+           </property>
+          </widget>
+         </item>
+         <item row="0" column="0">
+          <widget class="QLabel" name="label_image1">
+           <property name="minimumSize">
+            <size>
+             <width>40</width>
+             <height>40</height>
             </size>
            </property>
            <property name="text">
@@ -324,8 +314,8 @@
           <widget class="QLabel" name="label_image4">
            <property name="minimumSize">
             <size>
-             <width>80</width>
-             <height>80</height>
+             <width>40</width>
+             <height>40</height>
             </size>
            </property>
            <property name="text">
@@ -340,15 +330,49 @@
        </widget>
       </item>
       <item>
-       <widget class="QFrame" name="frame_URL">
-        <layout class="QGridLayout" name="frame_URL_GL">
-         <item row="0" column="0">
-          <widget class="QLabel" name="label_URL">
-           <property name="text">
-            <string>URL</string>
+       <widget class="QFrame" name="frame_PictureBrowse">
+        <property name="minimumSize">
+         <size>
+          <width>0</width>
+          <height>50</height>
+         </size>
+        </property>
+        <property name="frameShape">
+         <enum>QFrame::StyledPanel</enum>
+        </property>
+        <property name="frameShadow">
+         <enum>QFrame::Raised</enum>
+        </property>
+        <layout class="QHBoxLayout" name="horizontalLayout">
+         <item>
+          <widget class="QLineEdit" name="lineEdit_FilePath">
+           <property name="minimumSize">
+            <size>
+             <width>541</width>
+             <height>25</height>
+            </size>
            </property>
           </widget>
          </item>
+         <item>
+          <widget class="QPushButton" name="pushButton_Browse">
+           <property name="minimumSize">
+            <size>
+             <width>101</width>
+             <height>25</height>
+            </size>
+           </property>
+           <property name="text">
+            <string>Browse...</string>
+           </property>
+          </widget>
+         </item>
+        </layout>
+       </widget>
+      </item>
+      <item>
+       <widget class="QFrame" name="frame_URL">
+        <layout class="QGridLayout" name="frame_URL_GL">
          <item row="0" column="1">
           <widget class="QLineEdit" name="lineEdit_URL"/>
          </item>
@@ -359,13 +383,6 @@
            </property>
           </widget>
          </item>
-         <item row="1" column="0">
-          <widget class="QLabel" name="label_DisplayAs">
-           <property name="text">
-            <string>Display As</string>
-           </property>
-          </widget>
-         </item>
          <item row="1" column="1">
           <widget class="QLineEdit" name="lineEdit_DisplayAs"/>
          </item>
@@ -376,13 +393,31 @@
            </property>
           </widget>
          </item>
+         <item row="1" column="0">
+          <widget class="QLabel" name="label_DisplayAs">
+           <property name="text">
+            <string>Display As</string>
+           </property>
+          </widget>
+         </item>
+         <item row="0" column="0">
+          <widget class="QLabel" name="label_URL">
+           <property name="text">
+            <string>URL</string>
+           </property>
+          </widget>
+         </item>
         </layout>
        </widget>
       </item>
      </layout>
+     <zorder>frame_URL</zorder>
+     <zorder>frame_picture</zorder>
+     <zorder>frame_PictureBrowse</zorder>
+     <zorder>textEdit_Pulse</zorder>
     </widget>
    </item>
-   <item row="3" column="0">
+   <item>
     <layout class="QHBoxLayout" name="butons_HL">
      <property name="leftMargin">
       <number>6</number>
@@ -421,7 +456,6 @@
        <property name="font">
         <font>
          <pointsize>12</pointsize>
-         <weight>75</weight>
          <bold>true</bold>
         </font>
        </property>