locale: don't translate string "Electrum", "BIP39"
What the developer wrote
Authored by SomberNight
locale: don't translate string "Electrum", "BIP39"
What changed, and why it matters
This commit removes translation markers from two proper names ('Electrum' and 'BIP39') in a wallet setup wizard dropdown. It is a localization/UI consistency fix with no security relevance.
No security action needed; treat as routine UI/localization cleanup.
Security signals we found
No strong security signals were identified.
Evidence from the diff
In electrum/gui/qml/components/wizard/WCHaveSeed.qml, the qsTr() calls around the seed-type labels ‘Electrum’ and ‘BIP39’ were removed. This prevents translators from localizing these proper/standardized terms. The underlying values (‘electrum’, ‘bip39’) are unchanged, and no logic, validation, or cryptographic behavior is modified.
Changed components
electrum/gui/qml/components/wizard/WCHaveSeed.qmlInspect captured patch +2 / −2
diff --git a/electrum/gui/qml/components/wizard/WCHaveSeed.qml b/electrum/gui/qml/components/wizard/WCHaveSeed.qml
index 9974a60..cdb95bb 100644
--- a/electrum/gui/qml/components/wizard/WCHaveSeed.qml
+++ b/electrum/gui/qml/components/wizard/WCHaveSeed.qml
@@ -174,8 +174,8 @@ WizardComponent {
textRole: 'text'
valueRole: 'value'
model: [
- { text: qsTr('Electrum'), value: 'electrum' },
- { text: qsTr('BIP39'), value: 'bip39' }
+ { text: 'Electrum', value: 'electrum' },
+ { text: 'BIP39', value: 'bip39' }
]
onActivated: {
setSeedTypeHelpText()
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.