chore(tui): enable filtering in collections view

Author: maniac-en

internal/tui/app/model.go

@@ -40,18 +40,23 @@ func (m Model) Update(msg tea.Msg) (tea.Model, tea.Cmd) {
 
 	switch msg := msg.(type) {
 	case tea.KeyMsg:
-		switch msg.String() {
-		case "ctrl+c", "q":
-			if m.mode == CollectionsViewMode {
-				return m, tea.Quit
-			}
-			// For other views, 'q' goes back to collections
-			m.mode = CollectionsViewMode
-			return m, nil
-		case "a":
-			if m.mode == CollectionsViewMode {
-				m.mode = AddCollectionViewMode
+		// Handle global keybinds only when not in filtering mode
+		isFiltering := m.mode == CollectionsViewMode && m.collectionsView.IsFiltering()
+		
+		if !isFiltering {
+			switch msg.String() {
+			case "ctrl+c", "q":
+				if m.mode == CollectionsViewMode {
+					return m, tea.Quit
+				}
+				// For other views, 'q' goes back to collections
+				m.mode = CollectionsViewMode
 				return m, nil
+			case "a":
+				if m.mode == CollectionsViewMode {
+					m.mode = AddCollectionViewMode
+					return m, nil
+				}
 			}
 		}
 	case tea.WindowSizeMsg:

internal/tui/components/layout.go

@@ -46,16 +46,17 @@ func (l Layout) Content(content string, headerHeight, footerHeight int) string {
 func (l Layout) FullView(title, content, instructions string) string {
 	header := l.Header(title)
 	footer := l.Footer(instructions)
-	
+
 	headerHeight := lipgloss.Height(header)
 	footerHeight := lipgloss.Height(footer)
-	
+
 	contentArea := l.Content(content, headerHeight, footerHeight)
-	
+
 	return lipgloss.JoinVertical(
 		lipgloss.Left,
 		header,
 		contentArea,
 		footer,
 	)
-}
+}
+

internal/tui/components/paginated_list.go

@@ -34,7 +34,7 @@ func NewPaginatedList(items []ListItem, title string) PaginatedList {
 	l := list.New(listItems, paginatedItemDelegate{}, defaultWidth, defaultHeight)
 	l.Title = title
 	l.SetShowStatusBar(false)
-	l.SetFilteringEnabled(false)
+	l.SetFilteringEnabled(true) // Enable filtering
 	l.SetShowHelp(false) // Disable built-in help text
 	l.Styles.Title = styles.TitleStyle
 
@@ -81,6 +81,10 @@ func (pl PaginatedList) SelectedIndex() int {
 	return pl.list.Index()
 }
 
+func (pl PaginatedList) IsFiltering() bool {
+	return pl.list.FilterState() == list.Filtering
+}
+
 type paginatedItemDelegate struct{}
 
 func (d paginatedItemDelegate) Height() int                             { return 1 }

internal/tui/views/collections.go

@@ -109,29 +109,31 @@ func (v CollectionsView) Update(msg tea.Msg) (CollectionsView, tea.Cmd) {
 			break
 		}
 
-		// Handle pagination keys first, before the list can consume them
-		switch msg.String() {
-		case "n", "right":
-			// Next page
-			if v.currentPage < v.pagination.TotalPages {
-				return v, func() tea.Msg {
-					return v.loadCollectionsPage(v.currentPage+1, v.pageSize)
+		// Handle pagination keys only when not filtering
+		if !v.list.IsFiltering() {
+			switch msg.String() {
+			case "n", "right":
+				// Next page
+				if v.currentPage < v.pagination.TotalPages {
+					return v, func() tea.Msg {
+						return v.loadCollectionsPage(v.currentPage+1, v.pageSize)
+					}
 				}
-			}
-			return v, nil
-		case "p", "left":
-			// Previous page
-			if v.currentPage > 1 {
-				return v, func() tea.Msg {
-					return v.loadCollectionsPage(v.currentPage-1, v.pageSize)
+				return v, nil
+			case "p", "left":
+				// Previous page
+				if v.currentPage > 1 {
+					return v, func() tea.Msg {
+						return v.loadCollectionsPage(v.currentPage-1, v.pageSize)
+					}
 				}
+				return v, nil
 			}
-			return v, nil
-		default:
-			// Forward other keys to the list
-			v.list, cmd = v.list.Update(msg)
 		}
 
+		// Always forward keys to the list (handles filtering and navigation)
+		v.list, cmd = v.list.Update(msg)
+		
 	default:
 		if v.initialized {
 			v.list, cmd = v.list.Update(msg)
@@ -141,6 +143,10 @@ func (v CollectionsView) Update(msg tea.Msg) (CollectionsView, tea.Cmd) {
 	return v, cmd
 }
 
+func (v CollectionsView) IsFiltering() bool {
+	return v.initialized && v.list.IsFiltering()
+}
+
 func (v CollectionsView) View() string {
 	if !v.initialized {
 		return v.layout.FullView(
@@ -152,9 +158,12 @@ func (v CollectionsView) View() string {
 
 	content := v.list.View()
 
-	// Build instructions with pagination info
-	instructions := "↑↓: navigate • a: add • enter: edit • d: delete • q: quit"
-	if v.pagination.TotalPages > 1 {
+	// Build instructions with pagination and filter info
+	instructions := "↑↓: navigate • /: filter • enter: edit • d: delete • q: quit"
+	if !v.list.IsFiltering() {
+		instructions = "↑↓: navigate • a: add • /: filter • enter: edit • d: delete • q: quit"
+	}
+	if v.pagination.TotalPages > 1 && !v.list.IsFiltering() {
 		instructions += fmt.Sprintf(" • p/n: prev/next page (%d/%d)", v.currentPage, v.pagination.TotalPages)
 	}