opendev/gerrit
Code Issues Proposed changes

Automatically set focus to patch tables when showing a patch screen

Browse Source 
This way users can immediately use keyboard navigation within the
patch, especially to insert comments and move the cursor bar.

Signed-off-by: Shawn O. Pearce <sop@google.com>
This commit is contained in:
Shawn O. Pearce
2009-01-15 09:19:00 -08:00
parent 5bb9c0f48f
commit ffe31648db
2 changed files with 2 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
appjar/src/main/java/com/google/gerrit/client/patches/PatchSideBySideScreen.java
View File

@@ -56,5 +56,6 @@ public class PatchSideBySideScreen extends PatchScreen {
private void display(final SideBySidePatchDetail detail) { private void display(final SideBySidePatchDetail detail) {
sbsTable.setAccountInfoCache(detail.getAccounts()); sbsTable.setAccountInfoCache(detail.getAccounts());
sbsTable.display(detail); sbsTable.display(detail);
sbsTable.finishDisplay(true);
} }
} }

1
appjar/src/main/java/com/google/gerrit/client/patches/PatchUnifiedScreen.java
View File

@@ -53,5 +53,6 @@ public class PatchUnifiedScreen extends PatchScreen {
diffTable.setPatchKey(detail.getPatch().getKey()); diffTable.setPatchKey(detail.getPatch().getKey());
diffTable.setAccountInfoCache(detail.getAccounts()); diffTable.setAccountInfoCache(detail.getAccounts());
diffTable.display(detail.getLines()); diffTable.display(detail.getLines());
diffTable.finishDisplay(true);
} }
} }