From: Claudio Cambra <claudio.cambra@nextcloud.com>
Date: Thu, 24 Nov 2022 14:18:53 +0000 (+0100)
Subject: Simplify layout and fix alignments, sizings, and layouts of items with native theming... 
X-Git-Tag: archive/raspbian/3.16.7-1_deb13u1+rpi1~1^2~12^2~5^2~28^2~26
X-Git-Url: https://dgit.raspbian.org/?a=commitdiff_plain;h=382c0d24d5583b1e227132d998ac939d8583ce29;p=nextcloud-desktop.git

Simplify layout and fix alignments, sizings, and layouts of items with native theming in unified search input container

Signed-off-by: Claudio Cambra <claudio.cambra@nextcloud.com>
---

diff --git a/src/gui/tray/MainWindow.qml b/src/gui/tray/MainWindow.qml
index 28cec013f..4a421d9ad 100644
--- a/src/gui/tray/MainWindow.qml
+++ b/src/gui/tray/MainWindow.qml
@@ -580,17 +580,13 @@ ApplicationWindow {
 
         UnifiedSearchInputContainer {
             id: trayWindowUnifiedSearchInputContainer
-            height: Math.max(Style.talkReplyTextFieldPreferredHeight, contentHeight)
 
-            anchors {
-                top: trayWindowHeaderBackground.bottom
-                left: trayWindowMainItem.left
-                right: trayWindowMainItem.right
-
-                topMargin: Style.trayHorizontalMargin + controlRoot.padding
-                leftMargin: Style.trayHorizontalMargin + controlRoot.padding
-                rightMargin: Style.trayHorizontalMargin + controlRoot.padding
-            }
+            anchors.top: trayWindowHeaderBackground.bottom
+            anchors.left: trayWindowMainItem.left
+            anchors.right: trayWindowMainItem.right
+            anchors.topMargin: Style.trayHorizontalMargin
+            anchors.leftMargin: Style.trayHorizontalMargin
+            anchors.rightMargin: Style.trayHorizontalMargin
 
             text: UserModel.currentUser.unifiedSearchResultsListModel.searchTerm
             readOnly: !UserModel.currentUser.isConnected || UserModel.currentUser.unifiedSearchResultsListModel.currentFetchMoreInProgressProviderId
diff --git a/src/gui/tray/UnifiedSearchInputContainer.qml b/src/gui/tray/UnifiedSearchInputContainer.qml
index 7a6457c36..186d34481 100644
--- a/src/gui/tray/UnifiedSearchInputContainer.qml
+++ b/src/gui/tray/UnifiedSearchInputContainer.qml
@@ -21,98 +21,85 @@ import Style
 import com.nextcloud.desktopclient
 
 TextField {
-    id: trayWindowUnifiedSearchTextField
+    id: root
+
+    signal clearText()
 
     property bool isSearchInProgress: false
 
     readonly property color textFieldIconsColor: palette.dark
-
     readonly property color placeholderColor: palette.dark
 
-    readonly property int textFieldIconsOffset: Style.trayHorizontalMargin
-
-    readonly property double textFieldIconsScaleFactor: 0.6
-
-    readonly property int textFieldHorizontalPaddingOffset: Style.trayHorizontalMargin
+    readonly property int iconInset: Style.smallSpacing
 
-    signal clearText()
-
-    leftPadding: trayWindowUnifiedSearchTextFieldSearchIcon.width + trayWindowUnifiedSearchTextFieldSearchIcon.anchors.leftMargin + textFieldHorizontalPaddingOffset - 1
-    rightPadding: trayWindowUnifiedSearchTextFieldClearTextButton.width + trayWindowUnifiedSearchTextFieldClearTextButton.anchors.rightMargin + textFieldHorizontalPaddingOffset
+    topPadding: topInset
+    bottomPadding: bottomInset
+    leftPadding: searchIconImage.width + searchIconImage.x + Style.smallSpacing
+    rightPadding: (width - clearTextButton.x) + Style.smallSpacing
+    verticalAlignment: Qt.AlignVCenter
 
     placeholderText: qsTr("Search files, messages, events …")
-    verticalAlignment: Qt.AlignVCenter
 
     selectByMouse: true
 
-    background: Rectangle {
-        border.color: palette.dark
-        radius: Style.trayWindowRadius
-        color: palette.base
-    }
-
     Image {
-        id: trayWindowUnifiedSearchTextFieldSearchIcon
-        width: Style.trayListItemIconSize - anchors.leftMargin
-        fillMode: Image.PreserveAspectFit
-        horizontalAlignment: Image.AlignLeft
+        id: searchIconImage
 
         anchors {
-            left: parent.left
-            leftMargin: parent.textFieldIconsOffset
-            verticalCenter: parent.verticalCenter
+            left: root.left
+            leftMargin: iconInset
+            top: root.top
+            topMargin: Style.extraSmallSpacing
+            bottom: root.bottom
+            bottomMargin: Style.extraSmallSpacing 
         }
 
-        visible: !trayWindowUnifiedSearchTextField.isSearchInProgress
-
-        smooth: true;
+        fillMode: Image.PreserveAspectFit
+        smooth: true
         antialiasing: true
         mipmap: true
-        source: "image://svgimage-custom-color/search.svg" + "/" + trayWindowUnifiedSearchTextField.textFieldIconsColor
-        sourceSize: Qt.size(parent.height * parent.textFieldIconsScaleFactor, parent.height * parent.textFieldIconsScaleFactor)
+        source: "image://svgimage-custom-color/search.svg" + "/" + root.textFieldIconsColor
+        visible: !root.isSearchInProgress
     }
 
     NCBusyIndicator {
         id: busyIndicator
 
         anchors {
-            left: trayWindowUnifiedSearchTextField.left
-            bottom: trayWindowUnifiedSearchTextField.bottom
-            leftMargin: trayWindowUnifiedSearchTextField.textFieldIconsOffset - 4
-            topMargin: Style.smallSpacing
-            bottomMargin: Style.smallSpacing
-            verticalCenter: trayWindowUnifiedSearchTextField.verticalCenter
+            top: root.top
+            topMargin: Style.extraSmallSpacing
+            bottom: root.bottom
+            bottomMargin: Style.extraSmallSpacing
+            left: root.left
+            leftMargin: iconInset
         }
 
         width: height
-        color: trayWindowUnifiedSearchTextField.textFieldIconsColor
-        visible: trayWindowUnifiedSearchTextField.isSearchInProgress
+        color: root.textFieldIconsColor
+        visible: root.isSearchInProgress
         running: visible
     }
 
     Image {
-        id: trayWindowUnifiedSearchTextFieldClearTextButton
+        id: clearTextButton
 
         anchors {
-            right: parent.right
-            rightMargin: parent.textFieldIconsOffset
-            verticalCenter: parent.verticalCenter
+            top: root.top
+            topMargin: Style.extraSmallSpacing
+            bottom: root.bottom
+            bottomMargin: Style.extraSmallSpacing
+            right: root.right
+            rightMargin: iconInset
         }
 
-        smooth: true;
-        antialiasing: true
-        mipmap: true
-
-        visible: parent.text
-        source: "image://svgimage-custom-color/clear.svg" + "/" + trayWindowUnifiedSearchTextField.textFieldIconsColor
-        sourceSize: Qt.size(parent.height * parent.textFieldIconsScaleFactor, parent.height * parent.textFieldIconsScaleFactor)
+        fillMode: Image.PreserveAspectFit
+        visible: root.text
+        source: "image://svgimage-custom-color/clear.svg" + "/" + root.textFieldIconsColor
 
         MouseArea {
-            id: trayWindowUnifiedSearchTextFieldClearTextButtonMouseArea
-
+            id: clearTextButtonMouseArea
             anchors.fill: parent
-
-            onClicked: trayWindowUnifiedSearchTextField.clearText()
+            onClicked: root.clearText()
         }
     }
 }
diff --git a/theme/Style/Style.qml b/theme/Style/Style.qml
index a41247831..cda260235 100644
--- a/theme/Style/Style.qml
+++ b/theme/Style/Style.qml
@@ -151,6 +151,7 @@ QtObject {
     readonly property int unifiedSearchResultSectionItemLeftPadding: 16
     readonly property int unifiedSearchResultSectionItemVerticalPadding: 8
     readonly property int unifiedSearchResultNothingFoundHorizontalMargin: 10
+    readonly property int unifiedSearchInputContainerHeight: 40
 
     readonly property int radioButtonCustomMarginLeftInner: 4
     readonly property int radioButtonCustomMarginLeftOuter: 5