d3cd0121726b3ad2e9461451971a2537fd2d7114
max
  Wed Mar 2 17:11:07 2016 -0800
do not separate extra fields that are not even printed in hgc, refs #14486

diff --git src/hg/hgc/hgc.c src/hg/hgc/hgc.c
index 70bdbaa..37d8229 100644
--- src/hg/hgc/hgc.c
+++ src/hg/hgc/hgc.c
@@ -1573,38 +1573,38 @@
         if (ix == -1 || ix > fieldCount)      // so extraField really just provides a label
             continue;
         }
 
     char *fieldName = col->name;
 
     if (count == 0)
         printf("<br><table class='bedExtraTbl'>");
     
     count++;
 
     // do not print a row if the fieldName from the .as file is in the "skipFields" list
     if (skipIds && slNameInList(skipIds, fieldName))
         continue;
 
-    // split this table to separate current row from the previous one, if the trackDb option is set
-    if (sepFields && slNameInList(sepFields, fieldName))
-        printf("</tr></table>\n<p>\n<table class='bedExtraTbl'>");
-
     // skip this row if it's empty and "skipEmptyFields" option is set
     if (skipEmptyFields && isEmpty(fields[ix]))
         continue;
 
+    // split this table to separate current row from the previous one, if the trackDb option is set
+    if (sepFields && slNameInList(sepFields, fieldName))
+        printf("</tr></table>\n<p>\n<table class='bedExtraTbl'>");
+
     // field description
     printf("<tr><td>%s</td>", col->comment); // bold style now in HGStyle.css
 
     if (col->isList || col->isArray || col->lowType->stringy || asTypesIsInt(col->lowType->type))
         printIdOrLinks(col, fieldToUrl, tdb, fields[ix]);
     else if (asTypesIsFloating(col->lowType->type))
         {
         double valDouble = strtod(fields[ix],NULL);
         if (errno == 0 && valDouble != 0)
             printf("<td>%g</td></tr>\n", valDouble);
         else
             printf("<td>%s</td></tr>\n", fields[ix]); // decided not to print error
         }
     else
         printf("<td>%s</td></tr>\n", fields[ix]);