umputun / revdiff / 24319041964

90%

Build Type

push

github

Committed by web-flow

Commit Message

refactor(ui): extract overlay sub-package from model (#99)

* add plan: overlay-extraction

* feat: bootstrap overlay package with types, Manager, and composition helpers

Create app/ui/overlay/ sub-package with Manager coordinator, Kind/OutcomeKind
enums, Outcome/RenderCtx types, Resolver interface, and shared composition
helpers (overlayCenter, injectBorderTitle). Per-overlay types are stub
implementations to be filled in subsequent tasks.

* feat: implement help overlay rendering and key handling in overlay package

* feat: implement annotation list overlay rendering and key handling in overlay package

* feat: implement theme selector overlay rendering and key handling in overlay package

* feat: validate M1 milestone gate for overlay package extraction

* feat: add consumer-side overlayManager interface and ModelConfig wiring

* feat: migrate View() overlay dispatch to overlay.Compose and remove overlayCenter

* feat: migrate help overlay dispatch and consolidate overlay key routing

Consolidate all overlay key dispatch into a single unified block in
handleModalKey using m.overlay.Active() + m.overlay.HandleKey(), replacing
the separate help/annotList/themeSelect checks. Migrate help open action
from direct showHelp toggle to overlay.OpenHelp(spec) with buildHelpSpec
converting keymap sections to overlay.HelpSpec. Remove helpOverlay,
writeTOCHelpSection, helpColors, handleHelpKey from handlers.go and
help-related tests from handlers_test.go (covered by overlay/help_test.go).
Add buildAnnotListSpec, jumpToAnnotationTarget, previewThemeByName,
confirmThemeByName stubs for Tasks 9/10.

* feat: migrate annotation list overlay dispatch to overlay.Manager

remove old annotListOverlay, formatAnnotListItem, handleAnnotListKey,
injectBorderTitle, annotListBoxStyle, annotListMaxVisible, and
jumpToAnnotation methods from annotlist.go. these are now owned by the
overlay package. keep buildAnnotListItems, buildAnnotListSpec,
jumpToAnnotationTarget, positionOn... (continued)

Coverage Stats

599 of 630 new or added lines in 10 files covered. (95.08%)

3 existing lines in 2 files now uncovered.

6546 of 7380 relevant lines covered (88.7%)

350.6 hits per line