Feature request: continuous/stacked mode for reviews

[Litee]

Hi Umputun,

I love the tool, but I wish it'd support continuous/stacked mode when all changes are in the same long scroll with file names as separators. Zed diff view support it (see screenshot), and I saw it in a similar Hunk tool: https://github.com/modem-dev/hunk/ (another screenshot) As an extra improvements it may use sticky separators - for a long file its separator with file name sticks to the top of the viewport. Also, it is nice if the file in the middle of the viewport is marked in sidebar.

Why I'd want it? To optimize reading through large number of different changes. I want to have single action to scroll through all changes - mouse scroll or keyboard shortcut without switching. In current revdiff implementation there is no such way (feel free to correct me if I am wrong).

[umputun]

you can already move through every change without manually switching files, it's just off by default. With --cross-file-hunks on, ] / [ step hunk-to-hunk and keep going into the next/prev file at the boundary, skipping unchanged context. } / { walk annotations the same way across files. So the navigation you're after mostly exists today.

what's missing is the presentation you describe, one unbroken scroll with filename separators (plus the sticky header and the mid-viewport sidebar highlight on top of that). That's a real gap.

I like the continuous view as an idea. The catch is cost. The diff pane is single-file today, cursor, annotations and search all assume one loaded file, so a true unified buffer touches the core of the model instead of being a small add-on. Worth doing if it stays reasonable and doesn't overcomplicate the rest.

if you want to take it on, go for it, open a PR and let's see how it turns out. I can't promise I'll merge it, depends on how complex the implementation ends up, but I'll definitely consider it.

[Litee]

Cheers. I agree with keeping complexity under control. Once I have time (busy week), I'll look into the complexity costs.

[mishamsk]

Sorry for unsolicited chime in. I had --cross-file-hunks on for a long time. But now, with ability to hide reviewed files #257 and only show truly changed one on reload #256 I actually reverted back. Now a typical flow is to scroll through hunks up to the last one on the file, when done Space and this files disappears from view. In a long-running session for multi-stage review - it won't re-appear unless a follow up commit will introduce a genuine change.