diff --git a/gerrit-gwtui/src/main/java/com/google/gerrit/client/patches/AbstractPatchContentTable.java b/gerrit-gwtui/src/main/java/com/google/gerrit/client/patches/AbstractPatchContentTable.java index dc4db73d16..c7a9014191 100644 --- a/gerrit-gwtui/src/main/java/com/google/gerrit/client/patches/AbstractPatchContentTable.java +++ b/gerrit-gwtui/src/main/java/com/google/gerrit/client/patches/AbstractPatchContentTable.java @@ -137,8 +137,8 @@ public abstract class AbstractPatchContentTable extends NavigationTable p = p.getParent(); } - if (Gerrit.RESOURCES.css().commentHolder().equals( - table.getCellFormatter().getStyleName(row - 1, cell))) { + if (table.getCellFormatter().getStyleName(row - 1, cell) + .contains(Gerrit.RESOURCES.css().commentHolder())) { table.getCellFormatter().addStyleName(row - 1, cell, Gerrit.RESOURCES.css().commentPanelLast()); } @@ -560,14 +560,23 @@ public abstract class AbstractPatchContentTable extends NavigationTable } } table.removeRow(row); - } else if (span != 1) { - table.getFlexCellFormatter().setRowSpan(row, col, 1); - for (int r = row + 1; r < row + span; r++) { - table.insertCell(r, col + 1); + } else { + table.getFlexCellFormatter().setStyleName(// + row, col, Gerrit.RESOURCES.css().diffText()); + + if (span != 1) { + table.getFlexCellFormatter().setRowSpan(row, col, 1); + for (int r = row + 1; r < row + span; r++) { + table.insertCell(r, col); + + table.getFlexCellFormatter().setStyleName(// + r, col, Gerrit.RESOURCES.css().diffText()); + } } } } + protected void bindComment(final int row, final int col, final PatchLineComment line, final boolean isLast, boolean expandComment) { if (line.getStatus() == PatchLineComment.Status.DRAFT) {