refactor: use map instead of explicit if let

We also use last instead of first, because the last route should be the active one. Signed-off-by: William Casarin <[email protected]>
damus-io · Jun 24, 2024 · a9a07b3 · a9a07b3
1 parent 677c217
commit a9a07b3
Showing 1 changed file with 2 additions and 6 deletions.
diff --git a/src/ui/global_popup.rs b/src/ui/global_popup.rs
@@ -20,12 +20,8 @@ impl DesktopGlobalPopup {
         }
 
         let rect = ui.ctx().screen_rect().shrink(MARGIN);
-        let title = if let Some(first) = routes.first() {
-            // TODO(kernelkind): not a great way of getting the title of the routes 'grouping'
-            Some(first.title())
-        } else {
-            None
-        };
+
+        let title = routes.last().map(|r| r.title());
 
         let app_ctx = Rc::new(RefCell::new(app));