What the developer wrote
Authored by Sander van Grieken
qml: various styling updates
What changed, and why it matters
This commit makes small visual styling changes to three QML user-interface files in Electrum's mobile-style GUI. It adds a navigation bar background color property and sets background colors on two info text areas, plus adjusts spacing in a dialog. There is no indication these changes affect security, data handling, cryptography, or network behavior.
No security action required. Treat as a normal UI styling update.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The diff modifies QML UI components only: AddressDetails.qml and BalanceDetails.qml each gain a navigationBarBackgroundColor property bound to constants.highlightBackground; OpenChannelDialog.qml sets backgroundColor: constants.darkerDialogBackground on two InfoTextArea elements and replaces a Layout.topMargin on a BtcField with a spacer Item. These are purely presentational adjustments within the Qt Quick/QML layer and do not alter application logic, input validation, wallet operations, or any security-relevant code paths.
Changed components
electrum/gui/qml/components/AddressDetails.qmlelectrum/gui/qml/components/BalanceDetails.qmlelectrum/gui/qml/components/OpenChannelDialog.qmlInspect captured patch +6 / −2
diff --git a/electrum/gui/qml/components/AddressDetails.qml b/electrum/gui/qml/components/AddressDetails.qml
index b631e4b..eaf4c61 100644
--- a/electrum/gui/qml/components/AddressDetails.qml
+++ b/electrum/gui/qml/components/AddressDetails.qml
@@ -329,6 +329,7 @@ Pane {
}
}
}
+ property color navigationBarBackgroundColor: constants.highlightBackground
AddressDetails {
id: addressdetails
diff --git a/electrum/gui/qml/components/BalanceDetails.qml b/electrum/gui/qml/components/BalanceDetails.qml
index 49b92a4..03efe73 100644
--- a/electrum/gui/qml/components/BalanceDetails.qml
+++ b/electrum/gui/qml/components/BalanceDetails.qml
@@ -233,8 +233,8 @@ Pane {
}
}
-
}
+ property color navigationBarBackgroundColor: constants.highlightBackground
Component {
id: openChannelDialog
diff --git a/electrum/gui/qml/components/OpenChannelDialog.qml b/electrum/gui/qml/components/OpenChannelDialog.qml
index b18da4d..4315510 100644
--- a/electrum/gui/qml/components/OpenChannelDialog.qml
+++ b/electrum/gui/qml/components/OpenChannelDialog.qml
@@ -55,6 +55,7 @@ ElDialog {
'\n\n',
qsTr('If you want to have recoverable channels, you must create a new wallet with an Electrum seed')
].join('')
+ backgroundColor: constants.darkerDialogBackground
}
InfoTextArea {
@@ -65,6 +66,7 @@ ElDialog {
text: [ qsTr('You currently have recoverable channels setting disabled.'),
qsTr('This means your channels cannot be recovered from seed.')
].join(' ')
+ backgroundColor: constants.darkerDialogBackground
}
// gossip
@@ -158,10 +160,11 @@ ElDialog {
}
}
+ Item { Layout.columnSpan: 3; width: 1; height: constants.paddingLarge }
+
BtcField {
id: amountBtc
fiatfield: amountFiat
- Layout.topMargin: constants.paddingLarge
Layout.preferredWidth: amountFontMetrics.advanceWidth('0') * 14 + leftPadding + rightPadding
onTextAsSatsChanged: {
if (!is_max.checked)
Why this scored 15/100
Community notes
Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.
The AI analysis stands alone for now. Submit a note if you can add evidence or important context.