opendev/gerrit
Code Issues Proposed changes

Don't show the messages panel if there are no messages on this change

Browse Source 
It looks cleaner to not present an empty block.

Signed-off-by: Shawn O. Pearce <sop@google.com>
This commit is contained in:
Shawn O. Pearce
2009-01-15 15:00:13 -08:00
parent f28b922308
commit ecac98f9dc
1 changed files with 1 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/changes/ChangeScreen.java
View File

@@ -303,6 +303,7 @@ public class ChangeScreen extends Screen {
messagesContent.add(messagesMenuBar()); messagesContent.add(messagesMenuBar());
} }
messagesPanel.setOpen(msgList.size() > 0); messagesPanel.setOpen(msgList.size() > 0);
messagesPanel.setVisible(msgList.size() > 0);
} }
private LinkMenuBar messagesMenuBar() { private LinkMenuBar messagesMenuBar() {