What the developer wrote
Authored by SomberNight
qml: BalanceDetails: fix typo
What changed, and why it matters
This is a one-character typo fix in a user-interface file for Electrum's QML (mobile-style) GUI. The button that starts a Lightning swap was checking the wrong property name, which likely meant the button stayed disabled even when a Lightning receive balance existed. There is no security issue here—just a UI bug that may have prevented users from clicking a swap button.
No security action needed. Treat as a normal UI bug fix and include in routine release notes if desired.
Security signals we found
No strong security signals were identified.
Evidence from the diff
In electrum/gui/qml/components/BalanceDetails.qml, the enabled expression for the ‘Lightning swap’ button referenced Daemon.currentWallet.lightningCanReceive.satInt (missing the ‘s’ in ‘satsInt’). The patch corrects it to satsInt. This is a straightforward property-name typo fix in QML binding logic. It affects UI state only and has no bearing on cryptographic operations, wallet security, or network behavior.
Changed components
electrum/gui/qml/components/BalanceDetails.qmlInspect captured patch +1 / −1
diff --git a/electrum/gui/qml/components/BalanceDetails.qml b/electrum/gui/qml/components/BalanceDetails.qml
index 03efe73..43b177e 100644
--- a/electrum/gui/qml/components/BalanceDetails.qml
+++ b/electrum/gui/qml/components/BalanceDetails.qml
@@ -214,7 +214,7 @@ Pane {
Layout.preferredWidth: 1
text: qsTr('Lightning swap');
visible: Daemon.currentWallet.isLightning
- enabled: Daemon.currentWallet.lightningCanSend.satsInt > 0 || Daemon.currentWallet.lightningCanReceive.satInt > 0
+ enabled: Daemon.currentWallet.lightningCanSend.satsInt > 0 || Daemon.currentWallet.lightningCanReceive.satsInt > 0
icon.source: Qt.resolvedUrl('../../icons/update.png')
onClicked: app.startSwap()
}
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.