What the developer wrote
Authored by Sander van Grieken
qml: small gridlayout fix in BalanceDetails
What changed, and why it matters
This is a minor user-interface layout fix in Electrum's QML (mobile/desktop GUI) code. It moves a visibility check so that an entire row showing a Lightning color key is hidden when the wallet does not support Lightning, instead of hiding only the color box and label individually. There is no security relevance.
No security action needed; treat as a normal UI fix.
Security signals we found
No strong security signals were identified.
Evidence from the diff
In electrum/gui/qml/components/BalanceDetails.qml, the visible: Daemon.currentWallet.isLightning property was moved from the inner Rectangle and Label to the parent RowLayout. This prevents layout spacing/alignment issues when the Lightning legend should not appear. It is purely a UI/presentation change with no security implications.
Changed components
electrum/gui/qml/components/BalanceDetails.qmlInspect captured patch +1 / −2
diff --git a/electrum/gui/qml/components/BalanceDetails.qml b/electrum/gui/qml/components/BalanceDetails.qml
index 73c2bb1..49b92a4 100644
--- a/electrum/gui/qml/components/BalanceDetails.qml
+++ b/electrum/gui/qml/components/BalanceDetails.qml
@@ -100,14 +100,13 @@ Pane {
}
RowLayout {
+ visible: Daemon.currentWallet.isLightning
Rectangle {
- visible: Daemon.currentWallet.isLightning
Layout.preferredWidth: constants.iconSizeXSmall
Layout.preferredHeight: constants.iconSizeXSmall
color: constants.colorPiechartLightning
}
Label {
- visible: Daemon.currentWallet.isLightning
text: qsTr('Lightning')
}
}
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.