[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
}
}