From f7eb2186537b5145dd81b17a43d675d4bb540fb6 Mon Sep 17 00:00:00 2001 From: Keunhong Park Date: Mon, 4 Jun 2012 13:43:45 -0600 Subject: [PATCH] Change PatchTable pointer when loading patch This patch fixes an issue with the "file list" table displayed by clicking on the "Files" sub-menu when viewing a diff. Originally when navigating between patch screens the highlighted row (pointer) of the file list table would not change when not directly interacting with the table e.g. by clicking on the previous or next file link. This patch updates the file list table whenever a new patch screen is loaded so that the pointer corresponds to the current patch being displayed. Change-Id: I8944062ab7abda0951b30b304aaadfd5781c92a8 --- .../java/com/google/gerrit/client/patches/PatchScreen.java | 3 +++ 1 file changed, 3 insertions(+) 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 ffc896084e..b1a7feccf0 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 @@ -297,6 +297,9 @@ public abstract class PatchScreen extends Screen implements @Override protected void onLoad() { super.onLoad(); + + fileList.movePointerTo(patchKey); + if (patchSetDetail == null) { Util.DETAIL_SVC.patchSetDetail(idSideB, new GerritCallback() {