fix: mouse click selects correct package after list scrolls

Persist table scroll offset from ratatui's TableState after rendering
and add it to the clicked row index in both left-click and right-click
handlers. Previously, clicking a row in a scrolled list selected the
wrong package because the scroll offset was not accounted for.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Author: shanselman

Cargo.lock

@@ -82,6 +82,8 @@ pub struct App {
     pub show_help: bool,
     pub should_quit: bool,
     pub layout: LayoutRegions,
+    /// Scroll offset of the package list table (set during rendering)
+    pub table_scroll_offset: usize,
     /// Tick counter for animations (spinner, etc.)
     pub tick: usize,
     /// Incremented on each view refresh; stale results are discarded
@@ -114,6 +116,7 @@ impl App {
             show_help: false,
             should_quit: false,
             layout: LayoutRegions::default(),
+            table_scroll_offset: 0,
             tick: 0,
             view_generation: 0,
             detail_generation: 0,

src/app.rs

@@ -260,7 +260,7 @@ fn handle_mouse(
 
                 let content_y = app.layout.list_content_y;
                 if row >= content_y {
-                    let clicked_idx = (row - content_y) as usize;
+                    let clicked_idx = (row - content_y) as usize + app.table_scroll_offset;
                     if clicked_idx < app.filtered_packages.len() {
                         app.selected = clicked_idx;
                         load_detail_for_selected(app);
@@ -290,7 +290,7 @@ fn handle_mouse(
             if in_rect(col, row, app.layout.package_list) {
                 let content_y = app.layout.list_content_y;
                 if row >= content_y {
-                    let clicked_idx = (row - content_y) as usize;
+                    let clicked_idx = (row - content_y) as usize + app.table_scroll_offset;
                     if clicked_idx < app.filtered_packages.len() {
                         app.selected = clicked_idx;
                         load_detail_for_selected(app);

src/handler.rs

@@ -130,7 +130,7 @@ fn draw_main_content(f: &mut Frame, app: &mut App, area: Rect) {
     draw_detail_panel(f, app, chunks[1]);
 }
 
-fn draw_package_list(f: &mut Frame, app: &App, area: Rect) {
+fn draw_package_list(f: &mut Frame, app: &mut App, area: Rect) {
     let (icon, title) = match app.mode {
         AppMode::Search => ("🔍", "Search Results"),
         AppMode::Installed => ("📦", "Installed"),
@@ -254,6 +254,8 @@ fn draw_package_list(f: &mut Frame, app: &App, area: Rect) {
     let mut state = TableState::default();
     state.select(Some(app.selected));
     f.render_stateful_widget(table, area, &mut state);
+    // Capture scroll offset for mouse click hit-testing
+    app.table_scroll_offset = state.offset();
 
     // Scrollbar
     if app.filtered_packages.len() > (area.height as usize).saturating_sub(3) {