Fix sizing of share details page buttons

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

diff --git a/src/gui/filedetails/ShareDetailsPage.qml b/src/gui/filedetails/ShareDetailsPage.qml
index c2b8b95ba413d4cb44b05196beb3d99dda2779f9..4a78ffe47253a777bd8eb9231e5eb5332c249223 100644 (file)
--- a/src/gui/filedetails/ShareDetailsPage.qml
+++ b/src/gui/filedetails/ShareDetailsPage.qml
@@ -716,18 +716,16 @@ Page {
 
             Button {
                 height: Style.standardPrimaryButtonHeight
-                icon.source: hovered
-                    ? "image://svgimage-custom-color/close.svg/" + palette.brightText
-                    : "image://svgimage-custom-color/close.svg/" + Style.errorBoxBackgroundColor
+                icon.source: "image://svgimage-custom-color/close.svg/" + palette.buttonText
+                icon.height: Style.extraSmallIconSize
                 text: qsTr("Unshare")
                 onClicked: root.deleteShare()
             }
 
             Button {
                 height: Style.standardPrimaryButtonHeight
-                icon.source: hovered
-                    ? "image://svgimage-custom-color/add.svg/" + palette.brightText
-                    : "image://svgimage-custom-color/add.svg/" + root.accentColor
+                icon.source: "image://svgimage-custom-color/add.svg/" + palette.buttonText
+                icon.height: Style.extraSmallIconSize
                 text: qsTr("Add another link")
                 visible: root.isLinkShare && root.canCreateLinkShares
                 enabled: visible

diff --git a/theme/Style/Style.qml b/theme/Style/Style.qml
index cda260235dcb1d2967526f3c4ef14b724925d880..d145b16c39c8db1dfe2d1182e65b3e2282b5a753 100644 (file)
--- a/theme/Style/Style.qml
+++ b/theme/Style/Style.qml
@@ -71,6 +71,7 @@ QtObject {
     property int iconButtonWidth: 36\r
     property int standardPrimaryButtonHeight: 40\r
     readonly property int smallIconSize: 16\r
+    readonly property int extraSmallIconSize: 8\r
 \r
     property int minActivityHeight: variableSize(32)\r
 \r