Skip to content

feat(review): outline focus model, summaries, and progressive gap reveal - #85

Draft
lettertwo wants to merge 23 commits into
review/07-pipelinefrom
review/08-outline-gaps
Draft

feat(review): outline focus model, summaries, and progressive gap reveal#85
lettertwo wants to merge 23 commits into
review/07-pipelinefrom
review/08-outline-gaps

Conversation

@lettertwo

@lettertwo lettertwo commented Sep 4, 2026

Copy link
Copy Markdown
Owner

This PR reworks the outline into a home base with its own focus model (h/l between outline and diff), a scrolloff viewport with g/G, head-first ordering with directories before files, and a summary panel for header and directory rows. File rows carry status letters and opt-in nerd icons. Stage, unstage, and discard work from outline rows, and the diff position survives staging operations.

In the diff, collapsed context gaps expand progressively and can reveal up to the enclosing tree-sitter scope. A gap expansion that reshapes rows cancels any active line selection.

@lettertwo lettertwo changed the title review/08 outline gaps feat(review): outline focus model, summaries, and progressive gap reveal Sep 4, 2026
if_same_then_else in app.rs (merge the n==14 arm into the identical
n==2||n==10 branch) and two useless_vec in summary.rs tests. These
pre-existing lints would red-block the new scoped stop gate once per
code state on every turn that touches git-workon-review.
@lettertwo
lettertwo force-pushed the review/08-outline-gaps branch from d26af82 to 13e4707 Compare September 4, 2026 18:14
Sign up for free to 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