From 78097615a389715b770c79dd33bc450adfeeac1f Mon Sep 17 00:00:00 2001
From: Raphael Amorim <rapha850@gmail.com>
Date: Thu, 29 Aug 2024 22:23:45 +0200
Subject: [PATCH] remove grab cursor on macos and place default instead

---
 frontends/rioterm/src/constants.rs  |  3 ---
 frontends/rioterm/src/screen/mod.rs | 11 ++---------
 frontends/rioterm/src/sequencer.rs  | 24 ++++--------------------
 3 files changed, 6 insertions(+), 32 deletions(-)

diff --git a/frontends/rioterm/src/constants.rs b/frontends/rioterm/src/constants.rs
index 898e2f6b68..b8ed6d4e49 100644
--- a/frontends/rioterm/src/constants.rs
+++ b/frontends/rioterm/src/constants.rs
@@ -25,8 +25,5 @@ pub const DEADZONE_START_Y: f64 = 30.;
 #[cfg(target_os = "macos")]
 pub const DEADZONE_END_Y: f64 = -2.0;
 
-#[cfg(target_os = "macos")]
-pub const DEADZONE_START_X: f64 = 80.;
-
 pub const PADDING_X_COLLAPSED_TABS: f32 = 30.;
 pub const PADDING_Y_BOTTOM_TABS: f32 = 22.0;
diff --git a/frontends/rioterm/src/screen/mod.rs b/frontends/rioterm/src/screen/mod.rs
index 1391de73f7..a1d95a709a 100644
--- a/frontends/rioterm/src/screen/mod.rs
+++ b/frontends/rioterm/src/screen/mod.rs
@@ -14,7 +14,7 @@ use crate::bindings::{
     ViAction,
 };
 #[cfg(target_os = "macos")]
-use crate::constants::{DEADZONE_END_Y, DEADZONE_START_X, DEADZONE_START_Y};
+use crate::constants::{DEADZONE_END_Y, DEADZONE_START_Y};
 use crate::context::{self, process_open_url, ContextManager};
 use crate::crosswords::{
     grid::{Dimensions, Scroll},
@@ -280,6 +280,7 @@ impl Screen<'_> {
         )
     }
 
+    #[inline]
     pub fn touch_purpose(&mut self) -> &mut TouchPurpose {
         &mut self.touchpurpose
     }
@@ -292,14 +293,6 @@ impl Screen<'_> {
         pos_y <= DEADZONE_START_Y * scale_f64 && pos_y >= DEADZONE_END_Y * scale_f64
     }
 
-    #[inline]
-    #[cfg(target_os = "macos")]
-    pub fn is_macos_deadzone_draggable(&self, pos_x: f64) -> bool {
-        let layout = self.sugarloaf.layout();
-        let scale_f64 = layout.dimensions.scale as f64;
-        pos_x >= DEADZONE_START_X * scale_f64
-    }
-
     /// update_config is triggered in any configuration file update
     #[inline]
     pub fn update_config(
diff --git a/frontends/rioterm/src/sequencer.rs b/frontends/rioterm/src/sequencer.rs
index c956249e20..65ef496809 100644
--- a/frontends/rioterm/src/sequencer.rs
+++ b/frontends/rioterm/src/sequencer.rs
@@ -870,26 +870,10 @@ impl Sequencer {
                                         .is_native
                                     && route.window.screen.is_macos_deadzone(y)
                                 {
-                                    if route.window.screen.is_macos_deadzone_draggable(x)
-                                    {
-                                        if lmb_pressed || rmb_pressed {
-                                            route.window.screen.clear_selection();
-                                            route
-                                                .window
-                                                .winit_window
-                                                .set_cursor(CursorIcon::Grabbing);
-                                        } else {
-                                            route
-                                                .window
-                                                .winit_window
-                                                .set_cursor(CursorIcon::Grab);
-                                        }
-                                    } else {
-                                        route
-                                            .window
-                                            .winit_window
-                                            .set_cursor(CursorIcon::Default);
-                                    }
+                                    route
+                                        .window
+                                        .winit_window
+                                        .set_cursor(CursorIcon::Default);
 
                                     route.window.is_macos_deadzone = true;
                                     return;