Skip to content

Fix stale selector fragments left by a renderer regression - #19

Merged
polymorcodeus merged 2 commits into
mainfrom
model-bug
Aug 25, 2026
Merged

Fix stale selector fragments left by a renderer regression#19
polymorcodeus merged 2 commits into
mainfrom
model-bug

Conversation

@polymorcodeus

Copy link
Copy Markdown
Owner

Summary

Adds alternate-screen rendering for the interactive shelf, collection, and mark forms, printing the completion output after the program exits. This works around a regression introduced by the lipgloss/v2 2.0.6 bump, which transitively upgraded the ultraviolet renderer to a build that resizes its cell buffer before clearing a shrinking inline frame, leaving the tall selector visible above the shorter result.

What's new

  • Alternate-screen forms - shelf, collection, and mark selectors render in the alternate screen buffer and are discarded on exit, so the completion output prints cleanly to the main buffer below the banner.
  • Post-exit result rendering - each get and edit model exposes ResultView(), and a shared runProgram helper prints it after tea.NewProgram(...).Run() returns.
  • Responsive status panel - the side status panel shrinks to fit the terminal width instead of a hardcoded 68 columns, and forms render empty until the first WindowSizeMsg arrives.

Behavior changes / guardrails

  • The interactive form now runs in the alternate screen buffer and is discarded on exit; the result prints to the main buffer instead of lingering in the scrollback beside the form.
  • No dependency is pinned or downgraded; the workaround lives in the app layer, so future dependency bumps do not reintroduce the bug.
  • On narrow terminals the side panel shrinks rather than wrapping, and the first frame renders empty until the real terminal width is known.

Testing

  • TUI rendering is verified manually across book mark list and the shelf and collection flows, which now show only the result with no selector fragments.
  • make check runs fmt, vet, lint, and tests; all green.

@polymorcodeus
polymorcodeus merged commit 4ff0840 into mainAug 25, 2026
4 checks passed
@polymorcodeus
polymorcodeus deleted the model-bug branch August 25, 2026 00:14
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

@polymorcodeus