fix(core/eckhart): make scrolling smoother
What the developer wrote
Authored by obrusvit
fix(core/eckhart): make scrolling smoother
[no changelog]
What changed, and why it matters
This commit tweaks how quickly a menu scrolls when the user swipes the touchscreen on a Trezor hardware wallet model, and adds a request to redraw the screen during scrolling. It is a user-interface polish change with no apparent security relevance.
No security action needed; treat as normal UI quality improvement.
Security signals we found
No strong security signals were identified.
Evidence from the diff
In the Eckhart layout firmware UI, the constant TOUCH_SENSITIVITY_DIVIDER is reduced from 15 to 12, making the menu scroll a bit further per touch delta. Additionally, ctx.request_paint() is called after updating the menu offset and button states during a touch drag, ensuring the display refreshes while scrolling. The change is purely cosmetic/smoothness-related.
Changed components
core/embed/rust/src/ui/layout_eckhart/firmware/vertical_menu_screen.rsInspect captured patch +2 / −1
diff --git a/core/embed/rust/src/ui/layout_eckhart/firmware/vertical_menu_screen.rs b/core/embed/rust/src/ui/layout_eckhart/firmware/vertical_menu_screen.rs
index 09cd4eb8..13fcb490 100644
--- a/core/embed/rust/src/ui/layout_eckhart/firmware/vertical_menu_screen.rs
+++ b/core/embed/rust/src/ui/layout_eckhart/firmware/vertical_menu_screen.rs
@@ -41,7 +41,7 @@ pub enum VerticalMenuScreenMsg {
}
impl<T: MenuItems> VerticalMenuScreen<T> {
- const TOUCH_SENSITIVITY_DIVIDER: i16 = 15;
+ const TOUCH_SENSITIVITY_DIVIDER: i16 = 12;
const SUBTITLE_STYLE: TextStyle = theme::TEXT_MEDIUM_GREY;
const SUBTITLE_HEIGHT: i16 = 68;
const SUBTITLE_DOUBLE_HEIGHT: i16 = 100;
@@ -155,6 +155,7 @@ impl<T: MenuItems> VerticalMenuScreen<T> {
self.menu.set_offset(offset);
self.menu.update_button_states(ctx);
+ ctx.request_paint();
}
_ => {}
}
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.