diff options
| author | Paul Buetow <paul@buetow.org> | 2026-04-07 14:59:23 +0300 |
|---|---|---|
| committer | Paul Buetow <paul@buetow.org> | 2026-04-07 14:59:23 +0300 |
| commit | d5fcc8f9293fc02cfc60fa7027c3f829138907a3 (patch) | |
| tree | 4f10b980dc6b7be83e023f35ab283a313f41b0e9 /internal | |
| parent | da8c1e4b05306ae8336ee6583252e83050ca51dd (diff) | |
ui: remove empty lines from selected (highlighted) ultra cards
Use ultraJoinSections instead of ultraJoinSectionsWithBlank so no blank
separator row is inserted between card sections regardless of selection
state. The outer ultraCardStyle already paints the full card background,
so the per-section bg-coloured blank rows were redundant.
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Diffstat (limited to 'internal')
| -rw-r--r-- | internal/ui/ultra.go | 14 |
1 files changed, 5 insertions, 9 deletions
diff --git a/internal/ui/ultra.go b/internal/ui/ultra.go index 314e57e..fc28bc5 100644 --- a/internal/ui/ultra.go +++ b/internal/ui/ultra.go @@ -631,15 +631,11 @@ func (m *Model) renderUltraCard(t task.Task, width int, selected bool, re *regex bg = m.theme.SelectedBG } - // Blank line between sections must also carry bg; otherwise the terminal - // default (black) shows through on the empty separator line. - blankLine := lipgloss.NewStyle().Width(width).Background(lipgloss.Color(bg)).Render("") - if bg == "" { - blankLine = "" - } - - card := ultraJoinSectionsWithBlank( - blankLine, + // No blank lines between sections — ultraJoinSections passes "" so + // ultraJoinSectionsWithBlank skips the separator entirely. The outer + // ultraCardStyle already fills the selected background across all lines, + // so per-section bg-coloured blank rows are not needed. + card := ultraJoinSections( m.renderUltraHeaderWithRegex(t, width, re, bg), m.renderUltraMetaWithRegex(t, width, re, bg), m.renderUltraDescriptionWithRegex(t, width, re, bg), |