qml: Preferences: disable screenshot protection if !Android
What the developer wrote
Authored by f321x
qml: Preferences: disable screenshot protection if !Android
Disables the "Always allow Screenshots" switch in the preferences if the
App isn't running on Android. QML doesn't have screenshot protection
outside of Android so this toggle is misleading.
What changed, and why it matters
This commit is a small user-interface cleanup, not a security fix. It simply grays out and ignores a 'screenshot protection' setting on non-Android platforms because that feature only exists on Android. There is no vulnerability being patched.
No security action required; treat as a normal UI improvement.
Security signals we found
UI control disabled on platforms where the feature is unsupported
No memory safety, cryptographic, or authorization change
No bug fix for an exploitable behavior
Evidence from the diff
The change disables the ‘Always allow Screenshots’ switch in the QML Preferences page when the app is not running on Android, and also prevents the switch from being checked at startup on non-Android platforms. Screenshot protection is implemented only on Android, so the toggle was misleading elsewhere. The patch is purely UI/UX hardening and does not address an exploitable flaw.
Changed components
electrum/gui/qml/components/Preferences.qmlInspect captured patch +2 / −1
diff --git a/electrum/gui/qml/components/Preferences.qml b/electrum/gui/qml/components/Preferences.qml
index 3bbcff0..c65fee7 100644
--- a/electrum/gui/qml/components/Preferences.qml
+++ b/electrum/gui/qml/components/Preferences.qml
@@ -300,6 +300,7 @@ Pane {
Layout.columnSpan: 2
Layout.fillWidth: true
spacing: 0
+ enabled: AppController.isAndroid()
Switch {
id: disableScreenshots
onCheckedChanged: {
@@ -503,7 +504,7 @@ Pane {
freezeReusedAddressUtxos.checked = Config.freezeReusedAddressUtxos
useTrampolineRouting.checked = !Config.useGossip
enableDebugLogs.checked = Config.enableDebugLogs
- disableScreenshots.checked = !Config.alwaysAllowScreenshots
+ disableScreenshots.checked = !Config.alwaysAllowScreenshots && AppController.isAndroid()
setMaxBrightnessOnQrDisplay.checked = Config.setMaxBrightnessOnQrDisplay
useRecoverableChannels.checked = Config.useRecoverableChannels
syncLabels.checked = AppController.isPluginEnabled('labels')
Why this scored 20/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.