crates/tsql/src/ui/sidebar.rs (1)

419-432: Coordinate space is correct; border click filtering is optional.

The use of absolute coordinates Position::new(x, y) is appropriate—TreeState::click_at() expects absolute terminal/frame coordinates, which aligns with your implementation. If you want to improve UX by preventing accidental selection changes when clicking on the schema block border or title row, you can optionally add a check to ignore clicks on or above the schema area's top boundary:

 fn handle_schema_click(
     &mut self,
     x: u16,
     y: u16,
     _schema_area: Rect,
 ) -> (Option<SidebarAction>, Option<SidebarSection>) {
+    // Optional: ignore clicks on the border/title row(s).
+    if let Some(area) = self.schema_area {
+        if y <= area.y {
+            return (None, Some(SidebarSection::Schema));
+        }
+    }
+
     // TreeState's click_at expects absolute screen coordinates, not relative ones.
     // The tree widget stores absolute y positions in last_rendered_identifiers
     // during render, so we pass the mouse coordinates directly.
     use ratatui::layout::Position;
     self.schema_state.click_at(Position::new(x, y));

     (None, Some(SidebarSection::Schema))
 }

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

Disabled knowledge base sources:

• Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.