From f78ee9bb67a10d0dd9f7e6e8d7f02c347e1f07f9 Mon Sep 17 00:00:00 2001
From: waaake <vivek_ve@outlook.com>
Date: Sat, 30 Nov 2024 19:53:46 +0530
Subject: [PATCH] [ui] SearchBar: Search bar can now be toggled to become
 visible with the search button.

SearchBar gets the clear text feature allowing text to be cleared with a single click.

Added accepted signal for the Searchbar with Return and Enter key Press
---
 meshroom/ui/qml/Controls/SearchBar.qml | 60 +++++++++++++++++++++++++-
 1 file changed, 59 insertions(+), 1 deletion(-)

diff --git a/meshroom/ui/qml/Controls/SearchBar.qml b/meshroom/ui/qml/Controls/SearchBar.qml
index 1ba6923e..56b8bb71 100644
--- a/meshroom/ui/qml/Controls/SearchBar.qml
+++ b/meshroom/ui/qml/Controls/SearchBar.qml
@@ -9,9 +9,24 @@ import MaterialIcons 2.2
  */
 
 FocusScope {
+    id: root
     property alias textField: field
     property alias text: field.text
 
+    // Enables hiding and showing of the text field on Search button click
+    property bool toggle: false
+    property bool isVisible: false
+
+    // Size properties
+    property int maxWidth: 150
+    property int minWidth: 30
+
+    // The default width is computed based on whether toggling is enabled and if the visibility is true
+    width: toggle && isVisible ? maxWidth : minWidth
+
+    // Keyboard interaction related signals
+    signal accepted()
+
     implicitHeight: childrenRect.height
     Keys.forwardTo: [field]
 
@@ -24,10 +39,17 @@ FocusScope {
     }
 
     RowLayout {
+        spacing: 0
         width: parent.width
 
-        MaterialLabel {
+        MaterialToolButton {
             text: MaterialIcons.search
+
+            onClicked: {
+                isVisible = !root.isVisible
+                // Set Focus on the Text Field
+                field.focus = field.visible
+            }
         }
 
         TextField {
@@ -36,10 +58,46 @@ FocusScope {
             Layout.fillWidth: true
             selectByMouse: true
 
+            rightPadding: clear.width
+
+            // The text field is visible either when toggle is not activated or the visible property is set
+            visible: root.toggle ? root.isVisible : true
+
             // Ensure the field has focus when the text is modified
             onTextChanged: {
                 forceActiveFocus()
             }
+
+            // Handle enter Key press and forward it to the parent
+            Keys.onPressed: (event)=> {
+                if ((event.key == Qt.Key_Return || event.key == Qt.Key_Enter)) {
+                    event.accepted = true
+                    root.accepted()
+                }
+            }
+
+            MaterialToolButton {
+                id: clear
+
+                // Anchors
+                anchors.right: parent.right
+                anchors.rightMargin: 2  // Leave a tiny bit of space so that its highlight does not overlap with the boundary of the parent
+                anchors.verticalCenter: parent.verticalCenter
+
+                // Style
+                font.pointSize: 8
+                text: MaterialIcons.close
+                ToolTip.text: "Clears text."
+
+                // States
+                visible: field.text
+
+                // Signals -> Slots
+                onClicked: {
+                    field.text = ""
+                    parent.focus = true
+                }
+            }
         }
     }
 }