chore(core): remove unused function in `trezor.ui.layouts.eckhart`
What the developer wrote
Authored by Roman Zeyde
chore(core): remove unused function in `trezor.ui.layouts.eckhart`
[no changelog]
What changed, and why it matters
This commit simply deletes an unused Python helper function from a Trezor hardware wallet UI layout file. The function was not being called anywhere, and removing it does not change any runtime behavior or introduce any security risk.
No action required. This is a routine code-cleanup commit with no security relevance.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The diff removes confirm_single() from core/src/trezor/ui/layouts/eckhart/__init__.py. The function built an emphasized confirmation dialog using trezorui_api.confirm_emphasized(). The dead code also contained an unreachable call to show_continue_in_app(TR.address__confirmed) after the return statement. No callers, exports, or security-sensitive logic are affected.
Changed components
core/src/trezor/ui/layouts/eckhart/__init__.pyInspect captured patch +0 / −27
diff --git a/core/src/trezor/ui/layouts/eckhart/__init__.py b/core/src/trezor/ui/layouts/eckhart/__init__.py
index 7b6286f1a..5b6d6410b 100644
--- a/core/src/trezor/ui/layouts/eckhart/__init__.py
+++ b/core/src/trezor/ui/layouts/eckhart/__init__.py
@@ -76,33 +76,6 @@ def confirm_action(
)
-def confirm_single(
- br_name: str,
- title: str,
- description: str,
- description_param: str | None = None,
- verb: str | None = None,
-) -> Awaitable[None]:
- description_param = description_param or ""
-
- # Placeholders are coming from translations in form of {0}
- template_str = "{0}"
- assert template_str in description
-
- begin, _separator, end = description.partition(template_str)
- return raise_if_cancelled(
- trezorui_api.confirm_emphasized(
- title=title,
- items=(begin, (True, description_param), end),
- verb=None,
- ),
- br_name,
- ButtonRequestType.ProtectCall,
- )
-
- show_continue_in_app(TR.address__confirmed)
-
-
def confirm_reset_device(recovery: bool = False) -> Awaitable[None]:
return raise_if_cancelled(
trezorui_api.confirm_reset_device(recovery=recovery), None
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.