feat(core/rust): improve progress screen design

[no changelog]

core/embed/rust/src/ui/model_tr/component/progress.rs

@@ -5,18 +5,22 @@ use crate::{
     strutil::StringType,
     ui::{
         component::{
-            base::ComponentExt,
             paginated::Paginate,
             text::paragraphs::{Paragraph, Paragraphs},
             Child, Component, Event, EventCtx, Label, Never, Pad,
         },
+        constant,
         display::{self, Font},
         geometry::Rect,
         util::animation_disabled,
     },
 };
 
-use super::super::{constant, theme};
+use super::super::theme;
+
+const BOTTOM_DESCRIPTION_MARGIN: i16 = 10;
+const LOADER_Y_OFFSET_TITLE: i16 = -10;
+const LOADER_Y_OFFSET_NO_TITLE: i16 = -20;
 
 pub struct Progress<T>
 where
@@ -44,12 +48,13 @@ where
         update_description: fn(&str) -> Result<T, Error>,
     ) -> Self {
         Self {
-            title: Label::centered(title, theme::TEXT_BOLD).into_child(),
+            title: Child::new(Label::centered(title, theme::TEXT_BOLD)),
             value: 0,
             loader_y_offset: 0,
             indeterminate,
-            description: Paragraphs::new(Paragraph::new(&theme::TEXT_BIG, description).centered())
+            description: Child::new(Paragraphs::new(
-                .into_child(),
+                Paragraph::new(&theme::TEXT_NORMAL, description).centered(),
+            )),
             description_pad: Pad::with_background(theme::BG),
             update_description,
         }
@@ -72,10 +77,19 @@ where
             .chars()
             .filter(|c| *c == '\n')
             .count() as i16;
+
+        let (title, rest, loader_y_offset) = if self.title.inner().text().as_ref().is_empty() {
+            (Rect::zero(), Self::AREA, LOADER_Y_OFFSET_NO_TITLE)
+        } else {
             let (title, rest) = Self::AREA.split_top(self.title.inner().max_size().y);
-        let (loader, description) = rest.split_bottom(Font::BIG.line_height() * description_lines);
+            (title, rest, LOADER_Y_OFFSET_TITLE)
+        };
+
+        let (_loader, description) = rest.split_bottom(
+            BOTTOM_DESCRIPTION_MARGIN + Font::NORMAL.line_height() * description_lines,
+        );
         self.title.place(title);
-        self.loader_y_offset = loader.center().y - constant::screen().center().y;
+        self.loader_y_offset = loader_y_offset;
         self.description.place(description);
         self.description_pad.place(description);
         Self::AREA
@@ -88,7 +102,11 @@ where
                     ctx.request_paint();
                 }
                 self.description.mutate(ctx, |ctx, para| {
-                    if para.inner_mut().content().as_ref() != new_description {
+                    // NOTE: not doing any change for empty new descriptions
+                    // (currently, there is no use-case for deleting the description)
+                    if !new_description.is_empty()
+                        && para.inner_mut().content().as_ref() != new_description
+                    {
                         let new_description = unwrap!((self.update_description)(new_description));
                         para.inner_mut().update(new_description);
                         para.change_page(0); // Recompute bounding box.
@@ -112,7 +130,13 @@ where
                 None,
             );
         } else {
-            display::loader(self.value, self.loader_y_offset, theme::FG, theme::BG, None);
+            display::loader(
+                self.value,
+                self.loader_y_offset,
+                theme::FG,
+                theme::BG,
+                Some((theme::ICON_TICK_FAT, theme::FG)),
+            );
         }
         self.description_pad.paint();
         self.description.paint();

core/src/apps/bitcoin/sign_tx/progress.py

@@ -113,14 +113,10 @@ class Progress:
         from trezor import workflow
         from trezor.ui.layouts.progress import bitcoin_progress, coinjoin_progress
 
-        progress_layout = bitcoin_progress
+        progress_layout = coinjoin_progress if self.is_coinjoin else bitcoin_progress
-        if self.is_coinjoin:
-            progress_layout = coinjoin_progress
         workflow.close_others()
-        if self.signing:
+        text = "Signing transaction..." if self.signing else "Loading transaction..."
-            self.progress_layout = progress_layout("Signing transaction")
+        self.progress_layout = progress_layout(text)
-        else:
-            self.progress_layout = progress_layout("Loading transaction")
 
     def report(self) -> None:
         from trezor import utils

core/src/trezor/ui/layouts/tr/progress.py

@@ -44,29 +44,29 @@ def progress(
     )
 
 
-def bitcoin_progress(message: str) -> ProgressLayout:
+def bitcoin_progress(description: str) -> ProgressLayout:
-    return progress(message)
+    return progress("", description)
 
 
 def coinjoin_progress(message: str) -> ProgressLayout:
     # TODO: create show_progress_coinjoin for TR
-    return progress(message, description="Coinjoin")
+    return progress("Coinjoin", message)
     # return RustProgress(
     #     layout=trezorui2.show_progress_coinjoin(title=message, indeterminate=False)
     # )
 
 
 def pin_progress(message: str, description: str) -> ProgressLayout:
-    return progress(message, description=description)
+    return progress(message, description)
 
 
 def monero_keyimage_sync_progress() -> ProgressLayout:
-    return progress("SYNCING")
+    return progress("", "Syncing...")
 
 
 def monero_live_refresh_progress() -> ProgressLayout:
-    return progress("REFRESHING", indeterminate=True)
+    return progress("", "Refreshing...", indeterminate=True)
 
 
 def monero_transaction_progress_inner() -> ProgressLayout:
-    return progress("SIGNING TRANSACTION")
+    return progress("", "Signing transaction...")