f9803408404df869b2927caeb12ff463e6cbfed7
tdreszer
  Tue May 10 16:28:31 2011 -0700
Last fix to SNPs Expandable options worked in IE in compatability mode but not IE8.  Now it does.
diff --git src/hg/lib/jsHelper.c src/hg/lib/jsHelper.c
index 48e1784..5dc100a 100644
--- src/hg/lib/jsHelper.c
+++ src/hg/lib/jsHelper.c
@@ -453,36 +453,37 @@
     }
 //else
 //    warn("already looked up advancedJsEnabled");  // got msg 41 times in one page!
 return advancedJsEnabled;
 }
 
 void jsBeginCollapsibleSection(struct cart *cart, char *track, char *section, char *sectionTitle,
 			       boolean isOpenDefault)
 /* Make the hidden input, collapse/expand button and <TR id=...> needed for utils.js's
  * setTableRowVisibility().  Caller needs to have already created a <TABLE> and <FORM>. */
 {
 char collapseGroupVar[512];
 safef(collapseGroupVar, sizeof(collapseGroupVar), "%s.section_%s_close", track, section);
 boolean isOpen = !cartUsualBoolean(cart, collapseGroupVar, !isOpenDefault);
 
-printf("<TR><TD width='20'><input type='hidden' name=\"%s\" id=\"%s\" value=\"%s\">\n",
+// Both plus button and title are now in same <TD>
+// but still colspan=2 because we are lib code and callers own the table.
+printf("<TR><TD colspan=2 style='text-align:left;'>\n");
+printf("<input type='hidden' name='%s' id='%s' value='%s'>\n",
        collapseGroupVar, collapseGroupVar, isOpen ? "0" : "1");
-printf("<A HREF=\"%s?%s&%s=%s#%sGroup\" class='bigBlue'>\n",
-       cgiScriptName(), cartSidUrlString(cart), collapseGroupVar, (isOpen ? "1" : "0"), section);
 char *buttonImage = (isOpen ? "../images/remove_sm.gif" : "../images/add_sm.gif");
 printf("<IMG height='18' width='18' "
        "onclick=\"return setTableRowVisibility(this, '%s', '%s.section', 'section', true);\" "
-       "id=\"%s_button\" src=\"%s\" alt=\"%s\" title='%s this section' class='bigBlue'>"
-       "</A></TD>\n",
+       "id='%s_button' src='%s' alt='%s' title='%s this section' class='bigBlue'"
+       " style='cursor:pointer;'>\n",
        section, track,
        section, buttonImage, (isOpen ? "-" : "+"), (isOpen ? "Collapse": "Expand"));
-printf("<TD class='bigBlue' style='font-size:large; text-align:left; width:100%%'><B>&nbsp;%s</B></TD></TR>\n", sectionTitle);
+printf("<B style='font-size:larger;'>&nbsp;%s</B></TD></TR>\n", sectionTitle);
 printf("<TR %sid='%s-%d'><TD colspan=2>", isOpen ? "" : "style='display: none' ", section, 1);
 }
 
 void jsEndCollapsibleSection()
 /* End the collapsible <TR id=...>. */
 {
 puts("</TD></TR>");
 }