[frameworks/kirigami] src/forms: FormEntry: make spacing more consistent

Noah Davis <[email protected]>
Newsgroups gmane.comp.kde.cvs
Message-ID <[email protected]>
Git commit a80ba1a9cffcb3af6bb4cb4aebf0300cac43daf5 by Noah Davis.
Committed on 17/08/2026 at 14:49.
Pushed by ndavis into branch 'master'.

FormEntry: make spacing more consistent

There were parts where the default 5px spacing was used.

M  +5    -4    src/forms/cards/FormEntry.qml
M  +4    -2    src/forms/flat/FormEntry.qml

https://invent.kde.org/frameworks/kirigami/-/commit/a80ba1a9cffcb3af6bb4cb4aebf0300cac43daf5

diff --git a/src/forms/cards/FormEntry.qml b/src/forms/cards/FormEntry.qml
index 96d717e56..9000ac550 100644
--- a/src/forms/cards/FormEntry.qml
+++ b/src/forms/cards/FormEntry.qml
@@ -119,8 +119,9 @@ FT.FormEntry {
 
         contentItem: GridLayout {
             id: mainLayout
-            columnSpacing: Platform.Units.smallSpacing
-            rowSpacing: Platform.Units.smallSpacing
+            readonly property real spacing: Platform.Units.smallSpacing
+            columnSpacing: spacing
+            rowSpacing: spacing
             columns: 1 + leadingItems.visible + trailingItems.visible
             QQC.Label {
                 id: inlineLabel
@@ -135,7 +136,7 @@ FT.FormEntry {
                 id: leadingItems
                 Layout.rowSpan: subtitleLabel.visible ? 2 : 1
                 visible: children.length > 0 && width > 0
-                spacing: Platform.Units.smallSpacing
+                spacing: parent.spacing
                 children: root.leadingItems
             }
             QQC.Control {
@@ -167,7 +168,7 @@ FT.FormEntry {
                 Layout.minimumWidth: visible ? implicitWidth : 0
                 Layout.alignment: Qt.AlignVCenter | Qt.AlignRight
                 visible: children.length > 0 && width > 0
-                spacing: Platform.Units.smallSpacing
+                spacing: parent.spacing
                 children: root.trailingItems
             }
 
diff --git a/src/forms/flat/FormEntry.qml b/src/forms/flat/FormEntry.qml
index 0a163eefc..4ca71cf92 100644
--- a/src/forms/flat/FormEntry.qml
+++ b/src/forms/flat/FormEntry.qml
@@ -133,11 +133,12 @@ FT.FormEntry {
         RowLayout {
             id: leadingItems
             visible: children.length > 0
-            spacing: Platform.Units.smallSpacing
+            spacing: parent.spacing
             children: root.leadingItems
         }
         ColumnLayout {
             id: layout
+            spacing: parent.spacing
             Layout.fillWidth: true
             Layout.minimumWidth: contentItemWrapper.Layout.minimumWidth
             Layout.preferredWidth: contentItemWrapper.implicitWidth
@@ -160,6 +161,7 @@ FT.FormEntry {
             RowLayout {
                 id: contentItemAndTrailingLayout
                 Layout.fillWidth: true
+                spacing: parent.spacing
                 QQC.Control {
                     id: contentItemWrapper
                     padding: 0
@@ -177,7 +179,7 @@ FT.FormEntry {
 
                     Layout.minimumWidth: implicitWidth
                     visible: children.length > 0
-                    spacing: Platform.Units.smallSpacing
+                    spacing: parent.spacing
                     children: root.trailingItems
                 }
             }
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.