lerko/nib-v1
1
1
Fork 0
Code Issues Pull Requests 2 Actions Packages Projects Releases Wiki Activity

fix(tui): clean up stream row density — drop ID, fix newline leak, align margins

Browse Source 
ID cluttered rows and caused wrapping on long entries. Body newlines
leaked into stream rendering extra unindented lines. Cursor glyph
shifted selected rows 1 col right of unselected.

Remove ID from all row renderers (detail pane already shows it),
collapse multiline body to first line, cap tags to 2 in stream,
and reserve cursor column on unselected rows for consistent alignment.
This commit is contained in:
Tyler Koenig
2026-05-20 18:12:18 -04:00
parent 98fdae1e3a
commit 3eb778f31b
4 changed files with 22 additions and 18 deletions
Download Patch File Download Diff File Expand all files Collapse all files
internal/tui/absorb.go
+6 -4
View File
@@ -108,12 +108,14 @@ func (a absorbModel) visibleCount() int {
func renderAbsorbSource(e *db.Entity, maxWidth int) string { func renderAbsorbSource(e *db.Entity, maxWidth int) string {
glyph := glyphStyle.Render(display.DisplayGlyph(e.Glyph, e.CardType)) glyph := glyphStyle.Render(display.DisplayGlyph(e.Glyph, e.CardType))
id := idStyle.Render("[" + display.FormatID(e.ID) + "]")
body := e.Body body := e.Body
if e.Title != nil { if e.Title != nil {
body = *e.Title body = *e.Title
} }
if idx := strings.IndexByte(body, '\n'); idx >= 0 {
body = body[:idx]
}
var tags string var tags string
if len(e.Tags) > 0 { if len(e.Tags) > 0 {
@@ -125,11 +127,11 @@ func renderAbsorbSource(e *db.Entity, maxWidth int) string {
tags = " " + strings.Join(tagParts, " ") tags = " " + strings.Join(tagParts, " ")
} }
line := fmt.Sprintf("%s %s%s %s", glyph, body, tags, id) line := fmt.Sprintf("%s %s%s", glyph, body, tags)
if maxWidth > 0 && len(stripAnsi(line)) > maxWidth { if maxWidth > 0 && len(stripAnsi(line)) > maxWidth {
body = truncate(body, maxWidth-20) body = truncate(body, maxWidth-6)
line = fmt.Sprintf("%s %s%s %s", glyph, body, tags, id) line = fmt.Sprintf("%s %s%s", glyph, body, tags)
} }
return line return line
internal/tui/cards.go
+6 -4
View File
@@ -301,12 +301,14 @@ func (c cardsModel) visibleCount() int {
func renderCard(e *db.Entity, maxWidth int) string { func renderCard(e *db.Entity, maxWidth int) string {
glyph := glyphStyle.Render(display.DisplayGlyph(e.Glyph, e.CardType)) glyph := glyphStyle.Render(display.DisplayGlyph(e.Glyph, e.CardType))
id := idStyle.Render("[" + display.FormatID(e.ID) + "]")
body := e.Body body := e.Body
if e.Title != nil { if e.Title != nil {
body = *e.Title body = *e.Title
} }
if idx := strings.IndexByte(body, '\n'); idx >= 0 {
body = body[:idx]
}
affordance := detectAffordance(e) affordance := detectAffordance(e)
affordStr := "" affordStr := ""
@@ -335,11 +337,11 @@ func renderCard(e *db.Entity, maxWidth int) string {
useStr = " " + useCountStyle.Render(fmt.Sprintf("%d×", e.UseCount)) useStr = " " + useCountStyle.Render(fmt.Sprintf("%d×", e.UseCount))
} }
line := fmt.Sprintf("%s %s%s%s%s %s", glyph, body, affordStr, extraStr, useStr, id) line := fmt.Sprintf("%s %s%s%s%s", glyph, body, affordStr, extraStr, useStr)
if maxWidth > 0 && len(stripAnsi(line)) > maxWidth { if maxWidth > 0 && len(stripAnsi(line)) > maxWidth {
body = truncate(body, maxWidth-30) body = truncate(body, maxWidth-8)
line = fmt.Sprintf("%s %s%s%s%s %s", glyph, body, affordStr, extraStr, useStr, id) line = fmt.Sprintf("%s %s%s%s%s", glyph, body, affordStr, extraStr, useStr)
} }
return line return line
internal/tui/list.go
+9 -9
View File
@@ -204,23 +204,23 @@ func renderEntity(e *db.Entity, maxWidth int) string {
} }
glyph := style.Render(glyphStr) glyph := style.Render(glyphStr)
id := idStyle.Render("[" + display.FormatID(e.ID) + "]")
body := e.Body body := e.Body
if e.Title != nil { if e.Title != nil {
body = *e.Title body = *e.Title
} }
if idx := strings.IndexByte(body, '\n'); idx >= 0 {
body = body[:idx]
}
var extras []string var extras []string
if e.Pinned { if e.Pinned {
extras = append(extras, pinnedStyle.Render("•")) extras = append(extras, pinnedStyle.Render("•"))
} }
if len(e.Tags) > 0 { if len(e.Tags) > 0 {
tagParts := make([]string, len(e.Tags)) limit := min(2, len(e.Tags))
for i, t := range e.Tags { for _, t := range e.Tags[:limit] {
tagParts[i] = tagStyle.Render("#" + t) extras = append(extras, tagStyle.Render("#"+t))
} }
extras = append(extras, strings.Join(tagParts, " "))
} }
extraStr := "" extraStr := ""
@@ -228,11 +228,11 @@ func renderEntity(e *db.Entity, maxWidth int) string {
extraStr = " " + strings.Join(extras, " ") extraStr = " " + strings.Join(extras, " ")
} }
line := fmt.Sprintf("%s %s%s %s", glyph, body, extraStr, id) line := fmt.Sprintf("%s %s%s", glyph, body, extraStr)
if maxWidth > 0 && len(stripAnsi(line)) > maxWidth { if maxWidth > 0 && len(stripAnsi(line)) > maxWidth {
body = truncate(body, maxWidth-20) body = truncate(body, maxWidth-6)
line = fmt.Sprintf("%s %s%s %s", glyph, body, extraStr, id) line = fmt.Sprintf("%s %s%s", glyph, body, extraStr)
} }
return line return line
internal/tui/styles.go
+1 -1
View File
@@ -17,7 +17,7 @@ var (
PaddingLeft(1) PaddingLeft(1)
listItemStyle = lipgloss.NewStyle(). listItemStyle = lipgloss.NewStyle().
PaddingLeft(2) PaddingLeft(4)
selectedItemStyle = lipgloss.NewStyle(). selectedItemStyle = lipgloss.NewStyle().
PaddingLeft(1). PaddingLeft(1).