Merge pull request #118 from markusressel/bugfix/#114-Table-PgUp-sometimes-selects-the-table-header-even-though-the-list-is-not-fully-scrolled-to-the-top-yet-

fix: prevent table header selection on PgUp when not fully scrolled t…

Author: markusressel

internal/ui/table/table_component.go

@@ -52,6 +52,7 @@ type RowSelectionTable[T RowSelectionTableEntry] struct {
 	entries      []*T
 	entriesMutex sync.Mutex
 
+	lastSelectedEntry      *T
 	multiSelectEnabled     bool
 	multiSelectionEntryMap map[string]*T
 
@@ -119,7 +120,15 @@ func (c *RowSelectionTable[T]) createLayout() {
 	table.SetSelectable(true, false)
 	table.SetSelectionChangedFunc(func(row, column int) {
 		selectedEntry := c.GetSelectedEntry()
+
+		if c.lastSelectedEntry != nil && selectedEntry == nil {
+			// workaround to the table not scrolling to the top when PgUp is pressed and a page jump will
+			// select the table header row.
+			table.ScrollToBeginning()
+		}
 		c.selectionChangedCallback(selectedEntry)
+
+		c.lastSelectedEntry = selectedEntry
 	})
 
 	table.SetInputCapture(func(event *tcell.EventKey) *tcell.EventKey {
@@ -477,3 +486,11 @@ func (c *RowSelectionTable[T]) Down() {
 		c.layout.Select(row+1, col)
 	}
 }
+
+func (c *RowSelectionTable[T]) PageUp() {
+
+}
+
+func (c *RowSelectionTable[T]) PageDown() {
+
+}