Merge pull request #831 from benodiwal/fix/todos-layout-sidebar

fix: moved TODOs up in the sidebar

Author: dgageot

pkg/tui/components/sidebar/sidebar.go

@@ -244,18 +244,9 @@ func (m *model) verticalView() string {
 
 	m.todoComp.SetSize(m.width)
 	todoContent := strings.TrimSuffix(m.todoComp.Render(), "\n")
-
-	// Calculate available height for content
-	availableHeight := m.height - 2 // Account for borders
-	topHeight := strings.Count(topContent, "\n") + 1
-	todoHeight := strings.Count(todoContent, "\n") + 1
-
-	// Calculate padding needed to push todos to bottom
-	paddingHeight := max(availableHeight-topHeight-todoHeight, 0)
-	for range paddingHeight {
-		topContent += "\n"
+	if todoContent != "" {
+		topContent += "\n\n" + todoContent
 	}
-	topContent += todoContent
 
 	return styles.BaseStyle.
 		Width(m.width).

pkg/tui/components/tool/todotool/sidebar.go

@@ -63,7 +63,7 @@ func (c *SidebarComponent) Render() string {
 	}
 
 	var content strings.Builder
-	content.WriteString(styles.HighlightStyle.Render("TODOs"))
+	content.WriteString(styles.HighlightStyle.Render("TODOS"))
 	content.WriteString("\n")
 
 	for _, todo := range c.manager.GetTodos() {