build: prepare v1.1.0 release (#145) - #163
Merged
Merged
Conversation
Spec section 5.2 assigns the bubbles spinner to every busy state that was static text; the card editor's drafting and saving footers were the last two left behind (PR #150 deviation 8). The pattern is PR #150's: the editor holds a spinner.Model, SetStyles adopts the theme's spinner, the tick is batched at the operation start so an idle editor costs no timers, and the tick loop stops as soon as neither drafting nor saving is in flight. Refs #145.
The redesign release: the Strata design language, the full-frame board and proportional overlays, visible semantic-colored buttons, the adopted charm components, and the text-field key fixes. No database migration. Refs #145.
|
This was referenced Aug 20, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.



The release slice of map #136: places the one spinner the restyle cut left
unassigned, then carries the v1.1.0 notes.
Closes #145.
The leftover spinner
Spec section 5.2 assigns the
bubbles/v2 spinnerto every...ing...busystate that was static text. PR #150 wired it into the ADR split and the issue
import and recorded one exception (deviation 8): the card editor's
draftingand
savingstates were #143's surface, already shipped before the patternexisted, and were left for the map to place. This places them.
The pattern is #150's, copied whole:
cardeditor.Modelholds aspinner.Model, constructed from the theme'sSpinnertoken and re-adopted inSetStyles, so the editor never picks itsown frames.
busy()isdrafting || saving;spinTickdrops the tick the momentneither is in flight, so an idle editor costs no timers.
startDraft, both branchesof
startSave), not armed for as long as the overlay is open.busyPrefix()strips the frame to plain text so the footer band renders itin the band's own tier, and returns empty for a zero-value model.
spinner.TickMsgjoinscardeditor.IsMessage, so the root routes it.Tests
TestSpinnerAdvancesOnlyWhileDraftingOrSavingmirrors the existingadrsplitandissueimportspinner tests: idle drops the tick, both busystates take it, both footers carry the frame, a settled editor does not re-arm,
a zero-value model renders no frame, and the root routes the tick.
Three test helpers now walk a batch and skip the spinner tick, which is what
#150's deviation 7 predicted:
commandMsgForEditorin the editor package andresultSkippingSpinnerTickin the root package.No golden changed: the drafting and saving footers are not in any pinned frame.
Release notes
docs/releases/v1.1.0.md, in the v1.0.1 format. v1.1.0 is a minor: the UIchanged materially and nothing broke. The notes cover the Strata design
language, the full-frame board and proportional overlays, the visible
semantic-colored buttons, the adopted charm components, the huh v2 dialog
fields, the Home/End fix including the untypeable
h/limport bug, Ctrl+Aselect-all, and the theme package with zero styling outside it. Upgrade
section records no database migration; install and verify are as before.
.gitignoregains the notes file:/docs/releases/*is ignored with aper-release allowlist.
Verification
go build ./...go vet ./...sh scripts/check-go-format.shsh scripts/check-go-coverage.shinternal/tuicoverageinternal/tui/cardeditorcoveragego test -race ./internal/tui/...go test ./...