opendev/gerrit
Code Issues Proposed changes

Fix bad usage of setting display to none

Browse Source 
Change-Id: Ib74b3aec326f9970f1e022aa197be5886a19368d
This commit is contained in:
Shawn O. Pearce
2012-06-12 16:34:04 -07:00
parent b032a529f8
commit 3cbd191dd3
1 changed files with 2 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
gerrit-gwtui/src/main/java/com/google/gerrit/client/changes/CommitMessageBlock.java
View File

@@ -19,6 +19,7 @@ import com.google.gwt.user.client.ui.Composite;
import com.google.gwt.core.client.GWT;
import com.google.gwt.dom.client.PreElement;
import com.google.gwt.dom.client.Style.Display;
import com.google.gwt.uibinder.client.UiBinder;
import com.google.gwt.uibinder.client.UiField;
import com.google.gwt.user.client.ui.HTMLPanel;
@@ -50,7 +51,7 @@ public class CommitMessageBlock extends Composite {
// Hide commit body if there is no body
if (commitBody.trim().isEmpty()) {
commitBodyPre.setAttribute("style", "display: none;");
commitBodyPre.getStyle().setDisplay(Display.NONE);
}
// Linkify commit summary