Unify small icon sizes in file details components

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

diff --git a/src/gui/filedetails/ShareDelegate.qml b/src/gui/filedetails/ShareDelegate.qml
index c1b96102642d3cf54f7f1381c222bd53a6e52d99..5338ce18cbeca7eec8846c2cfc801528c251053c 100644 (file)
--- a/src/gui/filedetails/ShareDelegate.qml
+++ b/src/gui/filedetails/ShareDelegate.qml
@@ -165,6 +165,8 @@ GridLayout {
             bgNormalOpacity: 0
 
             icon.source: "image://svgimage-custom-color/add.svg/" + Style.ncTextColor
+            icon.width: Style.smallIconSize
+            icon.height: Style.smallIconSize
 
             visible: (root.isPlaceholderLinkShare || root.isSecureFileDropPlaceholderLinkShare) && root.canCreateLinkShares
             enabled: visible
@@ -201,8 +203,8 @@ GridLayout {
 
             icon.source: shareLinkCopied ? "image://svgimage-custom-color/copy.svg/" + Style.ncHeaderTextColor :
                                            "image://svgimage-custom-color/copy.svg/" + Style.ncTextColor
-            icon.width: 16
-            icon.height: 16
+            icon.width: Style.smallIconSize
+            icon.height: Style.smallIconSize
 
             visible: root.isLinkShare || root.isInternalLinkShare
             enabled: visible
@@ -246,6 +248,8 @@ GridLayout {
             bgNormalOpacity: 0
 
             icon.source: "image://svgimage-custom-color/more.svg/" + Style.ncTextColor
+            icon.width: Style.smallIconSize
+            icon.height: Style.smallIconSize
 
             visible: !root.isPlaceholderLinkShare && !root.isSecureFileDropPlaceholderLinkShare && !root.isInternalLinkShare
             enabled: visible

diff --git a/src/gui/filedetails/ShareDetailsPage.qml b/src/gui/filedetails/ShareDetailsPage.qml
index 8ef021527497d452e58624f1c85be596bcd369a7..ecdaa5c6a16d16c7be8dc95632b2d5204fc2c804 100644 (file)
--- a/src/gui/filedetails/ShareDetailsPage.qml
+++ b/src/gui/filedetails/ShareDetailsPage.qml
@@ -263,10 +263,10 @@ Page {
         ColumnLayout {
             id: moreMenu
 
-            property int rowIconWidth: 16
-            property int indicatorItemWidth: 20
-            property int indicatorSpacing: Style.standardSpacing
-            property int itemPadding: Style.smallSpacing
+            readonly property int rowIconWidth: Style.smallIconSize
+            readonly property int indicatorItemWidth: 20
+            readonly property int indicatorSpacing: Style.standardSpacing
+            readonly property int itemPadding: Style.smallSpacing
 
             width: parent.width
 

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