From f79c9ce48407073360b55c21c008567d28b56524 Mon Sep 17 00:00:00 2001
From: Ayman Bagabas <ayman.bagabas@gmail.com>
Date: Tue, 9 Dec 2025 09:56:53 -0500
Subject: [PATCH] fix(ui): fix scrolling up last item

---
 internal/ui/lazylist/list.go | 30 ++++++++++++++++++++----------
 1 file changed, 20 insertions(+), 10 deletions(-)

diff --git a/internal/ui/lazylist/list.go b/internal/ui/lazylist/list.go
index 4a2e47e02b6d8c677d5fecf9ee3fefb7c47f475a..9d6559b7540a97131c0f9d5c08c037c5279750a7 100644
--- a/internal/ui/lazylist/list.go
+++ b/internal/ui/lazylist/list.go
@@ -170,27 +170,37 @@ func (l *List) ScrollBy(lines int) {
 		}
 	} else if lines < 0 {
 		// Scroll up
-		l.offsetLine += lines
-		for l.offsetLine < 0 {
-			if l.offsetIdx == 0 {
-				// Reached the top of the list
-				l.offsetLine = 0
+		// Calculate from offset how many items needed to fill the viewport
+		// This is needed to know when to stop scrolling up
+		var totalLines int
+		var firstItemIdx int
+		for i := l.offsetIdx; i >= 0; i-- {
+			item := l.getItem(i)
+			totalLines += item.height
+			if l.gap > 0 && i < l.offsetIdx {
+				totalLines += l.gap
+			}
+			if totalLines >= l.height {
+				firstItemIdx = i
 				break
 			}
+		}
 
+		// Now scroll up by lines
+		l.offsetLine += lines // lines is negative
+		for l.offsetIdx > firstItemIdx && l.offsetLine < 0 {
 			// Move to previous item
 			l.offsetIdx--
-			item := l.getItem(l.offsetIdx)
-			totalHeight := item.height
+			prevItem := l.getItem(l.offsetIdx)
+			totalHeight := prevItem.height
 			if l.gap > 0 {
 				totalHeight += l.gap
 			}
 			l.offsetLine += totalHeight
 		}
 
-		item := l.getItem(l.offsetIdx)
-		if l.offsetLine >= item.height {
-			l.offsetLine = item.height - 1
+		if l.offsetLine < 0 {
+			l.offsetLine = 0
 		}
 	}
 }