de9d43cf6f3e2685b22d9e20cd07d111e2b1e35c
tdreszer
  Mon Aug 16 16:32:44 2010 -0700
Hidden vs. seen controls, id should not equal name
diff --git src/hg/js/hgTracks.js src/hg/js/hgTracks.js
index b4a63ef..7609835 100644
--- src/hg/js/hgTracks.js
+++ src/hg/js/hgTracks.js
@@ -808,6 +808,13 @@
 function imgTblDragHandleMouseOver()
 {
 // Highlights a single row when mouse over a dragHandle column (sideLabel and buttons)
+    if(jQuery.tableDnD == undefined) {
+        //var handle = $("td.dragHandle");
+        //$(handle)
+        //    .unbind('mouseenter')//, jQuery.tableDnD.mousemove);
+        //    .unbind('mouseleave');//, jQuery.tableDnD.mouseup);
+        return;
+    }
     if(jQuery.tableDnD.dragObject == null) {
         $( this ).parents("tr").addClass("trDrag");
     }
@@ -822,7 +829,7 @@
 function imgTblButtonMouseOver()
 {
 // Highlights a composite set of buttons, regarless of whether tracks are adjacent
-    if(jQuery.tableDnD.dragObject == null) {
+    if(jQuery.tableDnD == undefined || jQuery.tableDnD.dragObject == null) {
         var classList = $( this ).attr("class").split(" ");
         var btns = $( "p." + classList[0] );
         $( btns ).removeClass('btnGrey');
@@ -1943,7 +1950,7 @@
 function findTracksChangeVis(seenVis)
 {
     var trackName = $(seenVis).attr('id');
-    hiddenVis = $("input[name='"+trackName+"']");
+    hiddenVis = $("input[name='"+trackName.substring(0,trackName.length - "_id".length)+"']");
     $(hiddenVis).attr('disabled',false);
     $(hiddenVis).val($(seenVis).val());
 }
@@ -1951,14 +1958,15 @@
 function findTracksClickedOne(selCb,justClicked)
 {
     var selName = $(selCb).attr('id');
-    var trackName = selName.substring(0,selName.length - "_sel".length)
-    hiddenSel = $("input[name='"+selName+"']");
-    var seenVis = $('select#' + trackName);
-    hiddenVis = $("input[name='"+trackName+"']");
+    var trackName = selName.substring(0,selName.length - "_sel_id".length)
+    var hiddenSel = $("input[name='"+trackName+"_sel']");
+    var seenVis = $('select#' + trackName + "_id");
+    var hiddenVis = $("input[name='"+trackName+"']");
     var tr = $(selCb).parents('tr.found');
     var subtrack = $(tr).hasClass('subtrack');
     var canPack = $(tr).hasClass('canPack');
     var checked = $(selCb).attr('checked');
+    //warn(trackName +" selName:"+selName +" hiddenSel:"+$(hiddenSel).attr('name') +" seenVis:"+$(seenVis).attr('id') +" hiddenVis:"+$(hiddenVis).attr('name') +" subtrack:"+subtrack +" canPack:"+canPack);
 
     // First deal with seenVis control
     if(checked) {
@@ -1977,7 +1985,7 @@
         if(subtrack) {
             $(hiddenSel).attr('disabled',false);
             if(checked)
-                $(hiddenSel).val('on');
+                $(hiddenSel).val('1');
             else
                 $(hiddenSel).val('[]');
         }