547fafff699e3b2f00cd111de91ba560e29b5442
max
  Fri Sep 23 16:07:28 2016 -0700
crispr track: fixing a link problem visible only on firefox, refs #17235

diff --git src/hg/hgc/bigBedClick.c src/hg/hgc/bigBedClick.c
index a376f68..73de5ea 100644
--- src/hg/hgc/bigBedClick.c
+++ src/hg/hgc/bigBedClick.c
@@ -161,31 +161,31 @@
         }
     
     // print link to location
     printf("<td><a href='%s&db=%s&position=%s%%3A%d-%d'>%s:%d (%c)</a></td>\n", 
         hgTracksPathAndSettings(), database,
         chrom, pos+1, pos+23, chrom, pos+1, strand);
 
     printf("</tr>\n");
     }
 hFreeConn(&conn);
 printf("<tr>\n");
 if (coordCount!=0)
     printf("</table>\n");
 if (collapsed)
     {
-    printf("<p><a id='crisprShowAllLink' href='javascript:crisprShowAll()'>"
+    printf("<p><a id='crisprShowAllLink' href='#' onclick='crisprShowAll(); return false;'>"
         "Show all %d off-targets...</a>\n", coordCount);
     // inline .js is bad style but why pollute our global .js files for such a rare
     // case? Maybe we should have a generic "collapsible" class, like bootstrap?
     printf("<script>\n");
     printf("function crisprShowAll() {\n");
     printf("    $('#crisprShowAllLink').hide();\n");
     printf("    $('.crisprLinkHidden').show();\n");
     printf("    return false;\n");
     printf("}\n");
     printf("</script>\n");
     }
 }
 
 static void detailsTabPrintSpecial(char *name, char *val, struct slPair *extraFields)
 /* some extra fields require special printing code, they all start with '_'  */