Merge branch 'render'

raphamorim · Jan 12, 2025 · 970d8ce · 970d8ce
2 parents 9acb347 + a6ca12d
commit 970d8ce
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 6 deletions.
diff --git a/docs/docs/releases.md b/docs/docs/releases.md
@@ -22,6 +22,7 @@ language: 'en'
 	- `COLUMNS`: current columns
 	- `LINES`: current lines
 		- So, for example if you have: `{{COLUMNS}}x{{LINES}}` would show something like `88x66`.
+- Perf improvement on text selection [#898](https://github.com/raphamorim/rio/pull/898) by [@marc2332](https://github.com/marc2332).
 - Window title is now updated regardless the Navigation Mode.
 - Performance: Background and foreground data are only retrieved if is asked (either color automation is enabled or `window.title` contains any request for it).
 - Fixed: Nix build [#853](https://github.com/raphamorim/rio/pull/853).

diff --git a/frontends/rioterm/src/application.rs b/frontends/rioterm/src/application.rs
@@ -668,7 +668,7 @@ impl ApplicationHandler<EventPayload> for Application<'_> {
                     }
 
                     route.window.winit_window.set_cursor(CursorIcon::Pointer);
-                    route.window.screen.context_manager.schedule_render(60);
+                    route.window.screen.context_manager.request_render();
                 }
             }
 
@@ -867,7 +867,7 @@ impl ApplicationHandler<EventPayload> for Application<'_> {
 
                 if route.window.screen.search_nearest_hyperlink_from_pos() {
                     route.window.winit_window.set_cursor(CursorIcon::Pointer);
-                    route.window.screen.context_manager.schedule_render(60);
+                    route.window.screen.context_manager.request_render();
                 } else {
                     let cursor_icon =
                         if !route.window.screen.modifiers.state().shift_key()
@@ -894,7 +894,7 @@ impl ApplicationHandler<EventPayload> for Application<'_> {
                             .context_manager
                             .current_mut()
                             .set_hyperlink_range(None);
-                        route.window.screen.context_manager.schedule_render(60);
+                        route.window.screen.context_manager.request_render();
                     }
                 }
 
@@ -906,7 +906,7 @@ impl ApplicationHandler<EventPayload> for Application<'_> {
                         || !route.window.screen.mouse_mode())
                 {
                     route.window.screen.update_selection(point, square_side);
-                    route.window.screen.context_manager.schedule_render(60);
+                    route.window.screen.context_manager.request_render();
                 } else if square_changed
                     && route.window.screen.has_mouse_motion_and_drag()
                 {

diff --git a/frontends/rioterm/src/context/mod.rs b/frontends/rioterm/src/context/mod.rs
@@ -505,9 +505,9 @@ impl<T: EventListener + Clone + std::marker::Send + 'static> ContextManager<T> {
     }
 
     #[inline]
-    pub fn schedule_render(&mut self, scheduled_time: u64) {
+    pub fn request_render(&mut self) {
         self.event_proxy
-            .send_event(RioEvent::PrepareRender(scheduled_time), self.window_id);
+            .send_event(RioEvent::RenderRoute(self.current_route), self.window_id);
     }
 
     #[inline]