Repository navigation
Per-section sync 5/5: per-section results UI (confidence badges, apply or skip each section) #26
Description
Effort: L
Part of #8. Depends on sub-issues 3 and 4.
Present per-section sync results without overwhelming the user (the UX question in #8).
Scope
- Show a per-section confidence badge next to each section's result.
- Let the user apply or skip each section individually, plus an "apply all high-confidence" shortcut; review-each remains available.
- Make clear what will change before applying (per-section factor/offset or shift) and that the change is baked in once applied.
- Follow the host's v3 chrome and shortcut conventions (register any keyboard shortcuts through
window.registerShortcut, which is also the subject of the separate editor keybinds issue Hardcoded keybinds in onKeyDown bypass window.registerShortcut #10).
Acceptance criteria
- A song where one section has low confidence can be synced with that section skipped, in a few clicks.
- The default action never applies a low-confidence section.
- Works for a single-section song (collapses to the existing uniform flow).
Out of scope
The alignment engine, the baking, and the data model.
Activity
pullfrog commentedon Oct 2, 2026
pullfrog commentedon Oct 12, 2026
Implemented #26 (per-section sync review UI) and opened PR #71: #71
Per-section confidence badges, apply/skip toggles, apply-all-high-confidence, and a pre-apply factor/shift preview, all committing through the same
applyPerSectionSyncpath as #25. Low-confidence sections default to skip; failed sections aren't selectable; single-section songs collapse to uniform. Areviewfrogself-review caught a focus-trap leak after toggle re-renders, decision/acceptance confusion on conflict-demoted sections, and a summary that miscounted forced sections — all fixed with a new DOM-driven suite.npm test414 pass, lint clean.big-pickle(free) ｜ 𝕏fix(sync): per-section review focus trap, decision tracking and summary
fix(sync): gate the high-confidence apply on the default plan (#26)
feat(sync): per-section sync results UI (#26) (#71)