What the developer wrote
Authored by kdmukai
Fix error screen button text
What changed, and why it matters
This commit only changes the text on two on-screen buttons: one shown when an unrecognised QR code is scanned, and one shown after an unexpected system error. It does not change any security logic, input handling, cryptography, or network behaviour. It is a user-interface wording fix with no security relevance.
No security action needed; treat as a routine UI fix.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The patch updates two error views in a Python GUI application. In ScanInvalidQRTypeView the button label changes from “Done” to “Back to Main Menu”; in UnhandledExceptionView a “Back to Main Menu” button is added to the error screen. Both views already returned Destination(MainMenuView, clear_history=True). The change is purely presentational and does not affect code paths, permissions, secrets, or trust boundaries.
Changed components
src/seedsigner/views/scan_views.pysrc/seedsigner/views/view.pyInspect captured patch +2 / −1
diff --git a/src/seedsigner/views/scan_views.py b/src/seedsigner/views/scan_views.py
index af5883d..4f9d9b2 100644
--- a/src/seedsigner/views/scan_views.py
+++ b/src/seedsigner/views/scan_views.py
@@ -223,7 +223,7 @@ class ScanInvalidQRTypeView(View):
title=_("Error"),
status_headline=_("Unknown QR Type"),
text=_("QRCode is invalid or is a data format not yet supported."),
- button_data=[ButtonOption("Done")],
+ button_data=[ButtonOption("Back to Main Menu")],
)
return Destination(MainMenuView, clear_history=True)
diff --git a/src/seedsigner/views/view.py b/src/seedsigner/views/view.py
index 9e1bbea..5197b86 100644
--- a/src/seedsigner/views/view.py
+++ b/src/seedsigner/views/view.py
@@ -368,6 +368,7 @@ class UnhandledExceptionView(View):
title=_("System Error"),
status_headline=self.error[0],
text=self.error[1] + "\n" + self.error[2],
+ button_data=[ButtonOption("Back to Main Menu")],
allow_text_overflow=True, # Fit what we can, let the rest go off the edges
)
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.