From 23e4cf3eef32f21ee829e6b0fb599a99152b63c6 Mon Sep 17 00:00:00 2001 From: Bruce Zu Date: Tue, 23 Oct 2012 14:51:10 +0800 Subject: [PATCH] Fix: Right border of the expanded lines is missing. Expand the lines of context in SideBySide view, the expanded lines' right border is missing. This commit fixed it. Change-Id: I7d2402631fab46af66e5ef498ea1f238f00aed4e --- .../com/google/gerrit/client/patches/SideBySideTable.java | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/gerrit-gwtui/src/main/java/com/google/gerrit/client/patches/SideBySideTable.java b/gerrit-gwtui/src/main/java/com/google/gerrit/client/patches/SideBySideTable.java index ec63a83121..57e1532620 100644 --- a/gerrit-gwtui/src/main/java/com/google/gerrit/client/patches/SideBySideTable.java +++ b/gerrit-gwtui/src/main/java/com/google/gerrit/client/patches/SideBySideTable.java @@ -242,6 +242,7 @@ public class SideBySideTable extends AbstractPatchContentTable { fmt.addStyleName(row, A, Gerrit.RESOURCES.css().diffText()); fmt.addStyleName(row, B, Gerrit.RESOURCES.css().diffText()); fmt.addStyleName(row, B + 1, Gerrit.RESOURCES.css().lineNumber()); + fmt.addStyleName(row, B + 1, Gerrit.RESOURCES.css().rightmost()); } private int finish(final Iterator i, int row, final int col, boolean expandComment) { @@ -310,9 +311,7 @@ public class SideBySideTable extends AbstractPatchContentTable { CellFormatter fmt = table.getCellFormatter(); for (int i = 0 + offset; i < loopTo + offset; i++) { - // The overridden version of insertRow adds some css classes we don't - // want. - super.insertRow(row + i); + insertRow(row + i); table.getRowFormatter().setVerticalAlign(row + i, HasVerticalAlignment.ALIGN_TOP); int lineA = line.getStartA() + i;