New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.
Starred by 12 users

Issue metadata

Status: New
Owner: ----

Sign in to add a comment

Issue 5946: Simplify comparing a patch set with the latest patch set

Reported by, Apr 5 2017 Project Member

Issue description

Currently in PolyGerrit, when I click the link for a comment, the default view shows the diff between Base and the patch set with the comment. I almost always click the comment links when I want to see the changes the author applied in response to the comment. It makes more sense to me to display the Comment -> Latest Patch Set diff by default. (If the comment is on the latest, then it still should be Base -> Comment of course.)

It would also be helpful to have "diff with Base" and "diff with Latest" links. Fiddling with the patch set drop-down menus to change from Base -> N to N -> Latest is way too painful (especially when I accidentally click the Base drop-down first, because I can't select N until the right-hand side has been moved from N to Latest).

Comment 1 by, Apr 6 2017

Project Member
Labels: Type-Feature Milestone-Chromium-Afterglow Priority-2
Status: Accepted (was: New)

Comment 2 by, Jun 28 2017

Project Member

Comment 3 by, Sep 11 2017

Project Member
 Issue 7172  has been merged into this issue.

Comment 4 by, Oct 31 2017

Project Member
Labels: Hotlist-PatchRange

Comment 5 by, Nov 1 2017

Project Member
 Issue 7613  has been merged into this issue.

Comment 6 by, Nov 14 2017

Project Member
Issue 7746 has been merged into this issue.

Comment 7 by, Nov 27 2017

Project Member
Labels: UX-Requirement

Comment 8 by, Nov 28 2017

Project Member
Labels: -ux-requirement UX-Current

Comment 9 by, Dec 28 2017

Project Member
Labels: Triaged-Yes

Comment 10 by, Apr 18 2018

Project Member
 Issue 8788  has been merged into this issue.

Comment 11 by, Apr 18 2018

Project Member
From 8788:
I often have this workflow:
* CL author uploads patch set 1
* I comment on patch set 1
* Author replies the comment, uploads a new patch set
* I open the comment thread on patch set 1 to read the reply
* Now I want to compare the last patch set to the one I commented on.

So, essentially I want to switch from a base..1 diff to 1..2 (assuming 2 is the latest patch set), but it currently requires two interactions (base..1 -> base..2 -> 1..2), they have to be in the correct order because Gerrit doesn't allow the same patch set on both sides, and I have to remember the old patch set number because the intermediate step loses that information. Having a single button to do this would be very nice :)

Comment 12 by, Nov 22

Project Member
Labels: Triaged-No
Owner: ----
Status: New (was: Accepted)

Sign in to add a comment