Show changes since last review

[Tyriar]

GitHub.com flow

Blue line indicates there's a change:

Blue + indicates where I need to look from, handy link to the diff since I last looked:

Extension

No indication in tree (ie. I need to check notifications/email/github.com before switching back to extension):

No indication which commits/comments are new:

[dmitry-timofeev]

It might be even more useful to be able to select the commits to review (because "changes since last review" sometimes does not work as expected + gets cleared once you open the changes on github).

[alexr00]

I can use this extension the first time I look at a PR, but after there are changes I can't. I don't need to re-review everything. I just need to review the changes since the last time I looked. Just being able to select which commits to use would solve this for me.

[fenollp]

To add on

because "changes since last review" sometimes does not work as expected
I don't need to re-review everything. I just need to review the changes since the last time I looked.

and now that there is the Viewed checkbox, I'd like to revisit the ask:

The files that Changed since last view should not show the full PR diff but only the diff since last Viewed of each changed file.
Right now one has no other choice than to review the changed file completely all over.

This behavior could be toggled by clicking Changed since last view.

[watjurk]

Any update on this? This extension is really amazing but I think this is one of the key features that it's missing.

[mayurdhurpate]

Hi team, would like to bump this up. Would be very useful as re-reviewing a norm while doing a PR Code Review.

[shubham-agarwal-dd]

+1

[berkeleymalagon]

+1 💙

[halouvi]

+1 please :-)

[benibenj]

@daviddossett, we are looking for two icons for switching between the ChangesSinceLastReview Mode and AllChangesOfPullRequest Mode. The icon will be placed in the panel in the top left (as in image) and when clicking it the Icon should be replaced with the second Icon, corresponding to switching into the other mode. Do you recommend using a specific icon pair or should new Icons be created for this case?

[daviddossett]

@benibenj @alexr00 is this intended to act as a kind of filter? I.e. the "changes since last review" mode would be a subset of the full list?

[alexr00]

@daviddossett it's more of a toggle. In effect, it will filter some of the files out, but also the contents of the files (really the diffs) will change.

[daviddossett]

Ok, I'll see if we have any icons that work for this else I'll propose some new ones.

[daviddossett]

One possible idea. Still a bit rough but basically exploring something riffing off GH's "Files changed" icon and then a varation for pending changes that haven't yet been reviewed.

I'm not crazy about how the pending version looks yet but using these as a starting point.

[benibenj]

@daviddossett I think these icons are a good starting point. The arrows (in pending) could possibly just be replaced with some symbol/icon which would indicate NEW changes (However, all changes are 'New' just some of them are since a review).

A totally different option could be sort of a TODO list as changes since a review will probably be in relation to comments/suggestions made in that review.

I'm looking forward to your suggestions.

[daviddossett]

We could reuse checklist (left) and introduce a variant with unchecked items:

[alexr00]

@daviddossett I'm worried that the checklist and parital-checklist look too similar, but I like the idea of the "Files Changed" icon. What about using the dot idea similar to comment and comment-unresolved?

[daviddossett]

This could work. One concern here would be that the dot icon pattern is typically used for a state that could be cleared. E.g. clicking on an icon with a dot would reveal something and then would remove the dot.

Once reviewed, would this view still be visible? Or is it sort of temporary? This could make more sense if that's the case.

Another option would be to use the "Modified" icon as the nested indicator:

[benibenj]

I prefer the first option as many of the other options also use a dot when there is something new and you could argue that the state will be cleared after reviewing the new changes.

[daviddossett]

Conceptually it makes sense, but how would the interaction work? Would the icon with dot appear dynamically so that users notice it and click on it to view the new changes? I'd expect the dot to go away once reviewed. But then I wouldn't know what purpose the regular files-changed icon would have if there aren't any unseen changes anymore.

[benibenj]

The Icon only appears when there is a change since a review, otherwise no icon appears. When there is a change since a review and a user clicks on the icon it will display only the changes since the review and the other icon (normal diff icon) appears to switch back to showing all changes.

[daviddossett]

Ok, here's an idea to work with that:

• New changes are available as indicated by git-pull-request-new-changes
• Once viewing new changes, git-pull-request-go-to-changes appears as an actionable icon to return to the other view.
• If the new changes were reviewed, the git-pull-request-new-changes icon disappears (as demonstrated in the gif). If not reviewed, it stays.

Added new icons via vscode-codicons#124

[alexr00]

These fit very well with how the interaction works. Thanks @daviddossett!