Merge "Label: Make the meaning of the blank cell visible"

gerrit-gwtui/src/main/java/com/google/gerrit/client/GerritConstants.java

@@ -48,6 +48,8 @@ public interface GerritConstants extends Constants {
 
   String inactiveAccountBody();
 
+  String labelNotApplicable();
+
   String menuAll();
   String menuAllOpen();
   String menuAllMerged();

gerrit-gwtui/src/main/java/com/google/gerrit/client/GerritConstants.properties

@@ -31,6 +31,8 @@ noSuchGroupTitle = Code Review - Unknown Group
 
 inactiveAccountBody = This user is currently inactive.
 
+labelNotApplicable = Label not applicable
+
 menuAll = All
 menuAllOpen = Open
 menuAllMerged = Merged

gerrit-gwtui/src/main/java/com/google/gerrit/client/GerritCss.java

@@ -141,6 +141,7 @@ public interface GerritCss extends CssResource {
   String infoTable();
   String inputFieldTypeHint();
   String labelList();
+  String labelNotApplicable();
   String leftMostCell();
   String lineHeader();
   String lineNumber();

gerrit-gwtui/src/main/java/com/google/gerrit/client/changes/ChangeTable2.java

@@ -230,7 +230,8 @@ public class ChangeTable2 extends NavigationTable<ChangeInfo> {
 
       LabelInfo label = c.label(name);
       if (label == null) {
-        table.clearCell(row, col);
+        fmt.getElement(row, col).setTitle(Gerrit.C.labelNotApplicable());
+        fmt.addStyleName(row, col, Gerrit.RESOURCES.css().labelNotApplicable());
         continue;
       }
 

gerrit-gwtui/src/main/java/com/google/gerrit/client/gerrit.css

@@ -595,6 +595,9 @@ a:hover {
 .changeTable .dataCell.singleLine {
   white-space: nowrap;
 }
+.changeTable .dataCell.labelNotApplicable {
+ background: #F5F5F5;
+}
 .changeTable .iconHeader {
   border-top: 1px solid backgroundColor;
   border-bottom: 1px solid backgroundColor;