diff options
author | Edwin Kempin <edwin.kempin@sap.com> | 2011-06-03 13:11:46 +0200 |
---|---|---|
committer | Edwin Kempin <edwin.kempin@sap.com> | 2011-06-03 13:35:12 +0200 |
commit | c82e9e98b6d41dbd2267d72b96627a03c94739ab (patch) | |
tree | 41dae96a7088364a4778bdb30c01afafa2fb8d2f | |
parent | e43d3c46693564119ab10b232ac0ccdc24d598f5 (diff) |
Do not reset Patch History selection on navigation to next file diff
When the user views a file diff and then changes in the Patch History
the selection of side A (left side), then this selection is lost when
the user navigates to the previous or next file diff. The selection of
side A is always reset to the patch set that is selected as 'Old
Version History' on the ChangeScreen (the base for comparing patch
sets).
This change ensures that the selection in the Patch History is kept
when the user navigates to the previous or next file diff.
Bug: issue 999
Change-Id: I19cafedd8a11683dca4a24fbebbcacf63425fe62
Signed-off-by: Edwin Kempin <edwin.kempin@sap.com>
-rw-r--r-- | gerrit-gwtui/src/main/java/com/google/gerrit/client/patches/PatchScreen.java | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/gerrit-gwtui/src/main/java/com/google/gerrit/client/patches/PatchScreen.java b/gerrit-gwtui/src/main/java/com/google/gerrit/client/patches/PatchScreen.java index 0cdfcc8fcc..c815097e16 100644 --- a/gerrit-gwtui/src/main/java/com/google/gerrit/client/patches/PatchScreen.java +++ b/gerrit-gwtui/src/main/java/com/google/gerrit/client/patches/PatchScreen.java @@ -479,6 +479,9 @@ public abstract class PatchScreen extends Screen implements public void setSideA(PatchSet.Id patchSetId) { idSideA = patchSetId; diffSideA = patchSetId; + if (fileList != null) { + fileList.setPatchSetIdToCompareWith(patchSetId); + } } public void setSideB(PatchSet.Id patchSetId) { |