From fc73559e2b7e3431909a67c1775a85a43548d450 Mon Sep 17 00:00:00 2001
From: l0ng-ai <24760907+l0ng-ai@users.noreply.github.com>
Date: Sat, 8 Aug 2026 22:45:04 +0700
Subject: [PATCH] fix(panes): give the split divider the 8px grab the panel
 edges have
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

The sidebar and right-panel edges hand you an 8px target; a split
divider was 5px, so the one drag you do most often was the fussiest to
start. I had left this alone thinking a wider band would eat clicks at
the edge of the terminal text — it does not: the grid is inset by its
own GRID_PAD_X of 8px, so the extra 1.5px a side lands in padding and
stops 4px short of the nearest cell.

The gutter still measures 5px, so the split looks exactly as it did; the
target is an overlay centred on it, the same shape the panel edges use.

Measured rather than eyeballed: the divider line sits on one pixel
column, and dragging from 1.5px outside the old band now moves it.
---
 src/ui/pane.rs | 51 +++++++++++++++++++++++++++++++++++---------------
 1 file changed, 36 insertions(+), 15 deletions(-)

diff --git a/src/ui/pane.rs b/src/ui/pane.rs
index 6852576e..06956a63 100644
--- a/src/ui/pane.rs
+++ b/src/ui/pane.rs
@@ -551,25 +551,29 @@ impl Pane<PaneSlot> {
                 .size_full();
 
                 let line_color = if dragging.get() { active } else { idle };
-                let divider = div()
-                    .group("split-divider")
-                    .flex_none()
-                    .flex()
-                    .items_center()
-                    .justify_center()
+                // The gutter stays 5px so the split looks the same; the target
+                // is the 8px the sidebar and right-panel edges already hand
+                // you. The extra 1.5px a side reaches into the pane's own 8px
+                // GRID_PAD_X, so it never covers a cell of terminal text.
+                let overhang =
+                    (crate::ui::right_panel::RESIZE_HANDLE_WIDTH - DIVIDER_THICKNESS).max(0.) / 2.;
+                let grab = div()
+                    .occlude()
+                    .absolute()
                     .when(row, |d| {
-                        d.w(px(DIVIDER_THICKNESS)).h_full().cursor_col_resize()
+                        d.top_0()
+                            .h_full()
+                            .left(px(-overhang))
+                            .w(px(DIVIDER_THICKNESS + overhang * 2.))
+                            .cursor_col_resize()
                     })
                     .when(!row, |d| {
-                        d.h(px(DIVIDER_THICKNESS)).w_full().cursor_row_resize()
+                        d.left_0()
+                            .w_full()
+                            .top(px(-overhang))
+                            .h(px(DIVIDER_THICKNESS + overhang * 2.))
+                            .cursor_row_resize()
                     })
-                    .child(
-                        div()
-                            .when(row, |d| d.w(px(1.)).h_full())
-                            .when(!row, |d| d.h(px(1.)).w_full())
-                            .bg(line_color)
-                            .group_hover("split-divider", |s| s.bg(active)),
-                    )
                     .on_mouse_down(MouseButton::Left, {
                         let dragging = dragging.clone();
                         move |_ev, window, _cx| {
@@ -577,6 +581,23 @@ impl Pane<PaneSlot> {
                             window.refresh();
                         }
                     });
+                let divider = div()
+                    .group("split-divider")
+                    .flex_none()
+                    .relative()
+                    .flex()
+                    .items_center()
+                    .justify_center()
+                    .when(row, |d| d.w(px(DIVIDER_THICKNESS)).h_full())
+                    .when(!row, |d| d.h(px(DIVIDER_THICKNESS)).w_full())
+                    .child(
+                        div()
+                            .when(row, |d| d.w(px(1.)).h_full())
+                            .when(!row, |d| d.h(px(1.)).w_full())
+                            .bg(line_color)
+                            .group_hover("split-divider", |s| s.bg(active)),
+                    )
+                    .child(grab);
 
                 div()
                     .size_full()