Help
Change theme
Press space for more information.
Refresh (Shortcut: Shift+r)
Go home (Shortcut: u)
Copy issue ID
Show links for this issue (Shortcut: i, l)
Previous Issue (Shortcut: k)
Next Issue (Shortcut: j)
Sign in to use full features.
View issue level access limits(Press Alt + Right arrow for more information)
Pending code changes (auto-populated)
The version field defines the version of the software the bug was found in. [ID: 1154637]
Notification menu
Vote: I am impacted
Unintended behavior
View staffing
Description
What steps will reproduce the problem?
1. Consider three patchsets base, X, and Y, with base < X < Y.
2. Owner sends CL for review at PS X.
3. Reviewer inline comment C on PS X, diffing against base.
4. Owner makes changes and uploads PS Y.
5. Owner sends reply against PS Y, but inline comment C is tracked against PS X.
6. Reviewer clicks reply to inline comment and is taken to PS X.
What is the expected output?
Ideally there should be one click to get to the diff of PS X <-> PS Y.
What do you see instead?
Instead, the reviewer needs two clicks to diff PS X <-> PS Y to see that the comments were addressed: once to change the 'new' side to PS Y, and once to change the 'old' side to PS X. In addition, if the reviewer tries to change the 'old' side from base -> X first, they'll find that they can only set it to X - 1.
Please provide any additional information below.