opendev/gerrit
Code Issues Proposed changes

Always apply style to last column of branch list

Browse Source 
For the last column of the branch list a style was only applied if a
GitWeb link was configured. The assumption was that this column is
only used to display the GitWeb link, but this is not true anymore,
since also actions are displayed in this column.

Change-Id: Ie796de9ef55665be276f831a057a0f0729ece9d2
Signed-off-by: Edwin Kempin <edwin.kempin@sap.com>
This commit is contained in:
Edwin Kempin
2014-09-10 18:42:13 +02:00
parent 629aa341d0
commit c3f215cfab
1 changed files with 2 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
gerrit-gwtui/src/main/java/com/google/gerrit/client/admin/ProjectBranchesScreen.java
View File

@@ -239,9 +239,7 @@ public class ProjectBranchesScreen extends ProjectScreen {
fmt.addStyleName(0, 1, Gerrit.RESOURCES.css().iconHeader());
fmt.addStyleName(0, 2, Gerrit.RESOURCES.css().dataHeader());
fmt.addStyleName(0, 3, Gerrit.RESOURCES.css().dataHeader());
if (Gerrit.getGitwebLink() != null) {
fmt.addStyleName(0, 4, Gerrit.RESOURCES.css().dataHeader());
}
updateDeleteHandler = new ValueChangeHandler<Boolean>() {
@Override
@@ -416,9 +414,7 @@ public class ProjectBranchesScreen extends ProjectScreen {
fmt.addStyleName(row, 1, iconCellStyle);
fmt.addStyleName(row, 2, dataCellStyle);
fmt.addStyleName(row, 3, dataCellStyle);
if (c != null) {
fmt.addStyleName(row, 4, dataCellStyle);
}
setRowItem(row, k);
}