chore(tui): update footer, remove unused code

Author: maniac-en

internal/tui/app/context.go

@@ -27,4 +27,3 @@ func NewContext(
 		History:     history,
 	}
 }
-

internal/tui/app/model.go

@@ -33,7 +33,6 @@ func NewModel(ctx *Context) Model {
 		mode:              CollectionsViewMode,
 		collectionsView:   views.NewCollectionsView(ctx.Collections),
 		addCollectionView: views.NewAddCollectionView(ctx.Collections),
-		// editCollectionView will be created on demand
 	}
 }
 
@@ -46,7 +45,6 @@ func (m Model) Update(msg tea.Msg) (tea.Model, tea.Cmd) {
 
 	switch msg := msg.(type) {
 	case tea.KeyMsg:
-		// Handle global keybinds only when not in filtering mode
 		isFiltering := m.mode == CollectionsViewMode && m.collectionsView.IsFiltering()
 
 		if !isFiltering {
@@ -55,7 +53,6 @@ func (m Model) Update(msg tea.Msg) (tea.Model, tea.Cmd) {
 				if m.mode == CollectionsViewMode {
 					return m, tea.Quit
 				}
-				// For other views, 'q' goes back to collections
 				m.mode = CollectionsViewMode
 				return m, nil
 			case "a":
@@ -66,7 +63,6 @@ func (m Model) Update(msg tea.Msg) (tea.Model, tea.Cmd) {
 				}
 			case "e":
 				if m.mode == CollectionsViewMode {
-					// Get selected collection and switch to edit mode
 					if selectedItem := m.collectionsView.GetSelectedItem(); selectedItem != nil {
 						m.selectedIndex = m.collectionsView.GetSelectedIndex()
 						m.mode = EditCollectionViewMode
@@ -78,7 +74,6 @@ func (m Model) Update(msg tea.Msg) (tea.Model, tea.Cmd) {
 				}
 			case "x":
 				if m.mode == CollectionsViewMode {
-					// Delete selected collection
 					if selectedItem := m.collectionsView.GetSelectedItem(); selectedItem != nil {
 						return m, func() tea.Msg {
 							err := m.ctx.Collections.Delete(context.Background(), selectedItem.ID)
@@ -96,29 +91,24 @@ func (m Model) Update(msg tea.Msg) (tea.Model, tea.Cmd) {
 		m.height = msg.Height
 	case views.BackToCollectionsMsg:
 		m.mode = CollectionsViewMode
-		// Reload collections to show any changes
 		m.collectionsView.SetSelectedIndex(m.selectedIndex)
 		return m, m.collectionsView.Init()
 	case views.EditCollectionMsg:
 		m.mode = EditCollectionViewMode
 		m.editCollectionView = views.NewEditCollectionView(m.ctx.Collections, msg.Collection)
 		return m, nil
 	case views.CollectionDeletedMsg:
-		// Collection deleted, reload collections view
 		return m, m.collectionsView.Init()
 	case views.CollectionDeleteErrorMsg:
-		// Delete failed, just continue
 		return m, nil
 	case views.CollectionCreatedMsg:
-		// Collection created successfully, clear form and go to first page with first item selected
 		m.addCollectionView.ClearForm()
 		m.mode = CollectionsViewMode
-		m.selectedIndex = 0 // Reset to first item
+		m.selectedIndex = 0
 		m.collectionsView.SetSelectedIndex(m.selectedIndex)
 		return m, m.collectionsView.Init()
 	}
 
-	// Forward messages to the appropriate view
 	switch m.mode {
 	case CollectionsViewMode:
 		m.collectionsView, cmd = m.collectionsView.Update(msg)

internal/tui/components/collection_item.go

@@ -33,4 +33,4 @@ func (i CollectionItem) GetDescription() string {
 
 func (i CollectionItem) GetCollection() collections.CollectionEntity {
 	return i.collection
-}
+}

internal/tui/components/form.go

@@ -7,21 +7,20 @@ import (
 )
 
 type Form struct {
-	inputs      []TextInput
-	focusIndex  int
-	width       int
-	height      int
-	title       string
-	submitText  string
-	cancelText  string
+	inputs     []TextInput
+	focusIndex int
+	width      int
+	height     int
+	title      string
+	submitText string
+	cancelText string
 }
 
 func NewForm(title string, inputs []TextInput) Form {
-	// Focus the first input by default
 	if len(inputs) > 0 {
 		inputs[0].Focus()
 	}
-	
+
 	return Form{
 		inputs:     inputs,
 		focusIndex: 0,
@@ -34,10 +33,9 @@ func NewForm(title string, inputs []TextInput) Form {
 func (f *Form) SetSize(width, height int) {
 	f.width = width
 	f.height = height
-	
-	// Set width for all inputs
+
 	for i := range f.inputs {
-		f.inputs[i].SetWidth(width - 4) // Account for padding
+		f.inputs[i].SetWidth(width - 4)
 	}
 }
 
@@ -99,7 +97,7 @@ func (f *Form) nextInput() {
 	if len(f.inputs) == 0 {
 		return
 	}
-	
+
 	f.inputs[f.focusIndex].Blur()
 	f.focusIndex = (f.focusIndex + 1) % len(f.inputs)
 	f.inputs[f.focusIndex].Focus()
@@ -109,7 +107,7 @@ func (f *Form) prevInput() {
 	if len(f.inputs) == 0 {
 		return
 	}
-	
+
 	f.inputs[f.focusIndex].Blur()
 	f.focusIndex--
 	if f.focusIndex < 0 {
@@ -120,22 +118,22 @@ func (f *Form) prevInput() {
 
 func (f Form) View() string {
 	var content []string
-	
+
 	// Add form inputs
 	for _, input := range f.inputs {
 		content = append(content, input.View())
 	}
-	
+
 	// Add spacing
 	content = append(content, "")
-	
+
 	// Add action buttons
 	buttonStyle := styles.ListItemStyle.Copy().
 		Padding(0, 2).
 		Background(styles.Primary).
 		Foreground(styles.TextPrimary).
 		Bold(true)
-	
+
 	buttons := lipgloss.JoinHorizontal(
 		lipgloss.Top,
 		buttonStyle.Render(f.submitText+" (enter)"),
@@ -145,6 +143,6 @@ func (f Form) View() string {
 			Render(f.cancelText+" (esc)"),
 	)
 	content = append(content, buttons)
-	
+
 	return lipgloss.JoinVertical(lipgloss.Left, content...)
-}
+}

internal/tui/components/layout.go

@@ -59,4 +59,3 @@ func (l Layout) FullView(title, content, instructions string) string {
 		footer,
 	)
 }
-

internal/tui/components/paginated_list.go

@@ -35,7 +35,7 @@ func NewPaginatedList(items []ListItem, title string) PaginatedList {
 	l.Title = title
 	l.SetShowStatusBar(false)
 	l.SetFilteringEnabled(true) // Enable filtering
-	l.SetShowHelp(false) // Disable built-in help text
+	l.SetShowHelp(false)        // Disable built-in help text
 	l.Styles.Title = styles.TitleStyle
 
 	return PaginatedList{
@@ -46,7 +46,7 @@ func NewPaginatedList(items []ListItem, title string) PaginatedList {
 func (pl *PaginatedList) SetSize(width, height int) {
 	pl.width = width
 	pl.height = height
-	
+
 	// Safety check to prevent nil pointer dereference
 	if width > 0 && height > 0 {
 		pl.list.SetWidth(width)
@@ -107,4 +107,4 @@ func (d paginatedItemDelegate) Render(w io.Writer, m list.Model, index int, item
 
 		fmt.Fprint(w, fn(str))
 	}
-}
+}

internal/tui/components/text_input.go

@@ -66,11 +66,11 @@ func (t TextInput) View() string {
 	labelStyle := styles.TitleStyle.Copy().
 		Width(12).
 		Align(lipgloss.Right)
-	
+
 	return lipgloss.JoinHorizontal(
 		lipgloss.Top,
 		labelStyle.Render(t.label+":"),
 		" ",
 		t.textInput.View(),
 	)
-}
+}

internal/tui/styles/colors.go

@@ -4,18 +4,18 @@ import "github.com/charmbracelet/lipgloss"
 
 var (
 	// Primary colors
-	Primary   = lipgloss.Color("62")   // Blue
-	Secondary = lipgloss.Color("170")  // Purple
-	Success   = lipgloss.Color("40")   // Green
-	Warning   = lipgloss.Color("220")  // Yellow
-	Error     = lipgloss.Color("196")  // Red
-	
+	Primary   = lipgloss.Color("62")  // Blue
+	Secondary = lipgloss.Color("170") // Purple
+	Success   = lipgloss.Color("40")  // Green
+	Warning   = lipgloss.Color("220") // Yellow
+	Error     = lipgloss.Color("196") // Red
+
 	// Text colors
 	TextPrimary   = lipgloss.Color("230") // Light
 	TextSecondary = lipgloss.Color("241") // Dim
 	TextMuted     = lipgloss.Color("238") // Very dim
-	
+
 	// Background colors
 	BackgroundPrimary   = lipgloss.Color("0")   // Black
 	BackgroundSecondary = lipgloss.Color("234") // Dark gray
-)
+)

internal/tui/styles/layout.go

@@ -4,30 +4,30 @@ import "github.com/charmbracelet/lipgloss"
 
 var (
 	HeaderStyle = lipgloss.NewStyle().
-		Padding(1, 2).
-		Background(Primary).
-		Foreground(TextPrimary).
-		Bold(true).
-		Align(lipgloss.Center)
+			Padding(1, 2).
+			Background(Primary).
+			Foreground(TextPrimary).
+			Bold(true).
+			Align(lipgloss.Center)
 
 	FooterStyle = lipgloss.NewStyle().
-		Padding(0, 2).
-		Foreground(TextSecondary).
-		Align(lipgloss.Center)
+			Padding(0, 2).
+			Foreground(TextSecondary).
+			Align(lipgloss.Center)
 
 	ContentStyle = lipgloss.NewStyle().
-		Padding(1, 2)
+			Padding(1, 2)
 
 	ListItemStyle = lipgloss.NewStyle().
-		PaddingLeft(4)
+			PaddingLeft(4)
 
 	SelectedListItemStyle = lipgloss.NewStyle().
-		PaddingLeft(2).
-		Foreground(Secondary)
+				PaddingLeft(2).
+				Foreground(Secondary)
 
 	TitleStyle = lipgloss.NewStyle().
-		MarginLeft(2).
-		MarginBottom(1).
-		Foreground(Primary).
-		Bold(true)
-)
+			MarginLeft(2).
+			MarginBottom(1).
+			Foreground(Primary).
+			Bold(true)
+)

internal/tui/views/add_collection.go

@@ -22,10 +22,10 @@ func NewAddCollectionView(collectionsManager *collections.CollectionsManager) Ad
 	inputs := []components.TextInput{
 		components.NewTextInput("Name", "Enter collection name"),
 	}
-	
+
 	form := components.NewForm("Add Collection", inputs)
 	form.SetSubmitText("Create")
-	
+
 	return AddCollectionView{
 		layout:             components.NewLayout(),
 		form:               form,
@@ -45,39 +45,36 @@ func (v AddCollectionView) Update(msg tea.Msg) (AddCollectionView, tea.Cmd) {
 		v.width = msg.Width
 		v.height = msg.Height
 		v.layout.SetSize(v.width, v.height)
-		v.form.SetSize(v.width-4, v.height-8) // Account for layout padding
-		
+		v.form.SetSize(v.width-4, v.height-8)
+
 	case tea.KeyMsg:
 		if v.submitting {
-			// Don't handle keys while submitting
 			return v, nil
 		}
-		
+
 		switch msg.String() {
 		case "enter":
 			return v, func() tea.Msg { return v.submitForm() }
 		case "esc":
 			return v, func() tea.Msg { return BackToCollectionsMsg{} }
 		}
-		
+
 	case CollectionCreateErrorMsg:
-		// Handle error - for now just stop submitting
 		v.submitting = false
 	}
-	
-	// Update form
+
 	v.form, cmd = v.form.Update(msg)
 	return v, cmd
 }
 
 func (v *AddCollectionView) submitForm() tea.Msg {
 	v.submitting = true
 	values := v.form.GetValues()
-	
+
 	if len(values) == 0 || values[0] == "" {
 		return CollectionCreateErrorMsg{err: crud.ErrInvalidInput}
 	}
-	
+
 	return v.createCollection(values[0])
 }
 
@@ -104,15 +101,14 @@ func (v AddCollectionView) View() string {
 
 	content := v.form.View()
 	instructions := "tab/↑↓: navigate • enter: create • esc: cancel"
-	
+
 	return v.layout.FullView(
 		"Add Collection",
 		content,
 		instructions,
 	)
 }
 
-// Messages for collection operations
 type CollectionCreatedMsg struct {
 	collection collections.CollectionEntity
 }
@@ -141,4 +137,4 @@ type BackToCollectionsMsg struct{}
 
 type EditCollectionMsg struct {
 	Collection collections.CollectionEntity
-}
+}