90575b28d8d97732503671e4c5c80f7fbf2d3ac2
tdreszer
  Tue Oct 11 13:57:16 2011 -0700
With subCfg, when only one subtrack, highlevel shouuld show subtrack controls, so that rightClick and subCfg work together.
diff --git src/hg/js/subCfg.js src/hg/js/subCfg.js
index 990a97e..623840c 100644
--- src/hg/js/subCfg.js
+++ src/hg/js/subCfg.js
@@ -23,33 +23,33 @@
 // - SOLVED: OpenChromSynth: subtrack filterby needs to be updated by composite filterBy.
 // - SOLVED: check subtrack to enable/disable fauxVis and wrench
 // - SOLVED: matCB should effect subCb including enable/disable fauxVis and wrench
 // - SOLVED: composite/view vis should effect subVis and enable/disable fauxVis and wrench
 // - SOLVED: inside out: changing subtrack vis should affect subCB and matCB
 // - SOLVED: Loosing checked tracks!!!  Setting vis clears checkboxes?
 // - TESTED: hui.c #ifdef SUBTRACK_CFG should switch full functionality on/off
 // - SOLVED: Make "disabled" subCB clickable!
 // - SOLVED: PROBLEM is fauxDisabled.  SOLUTION is convert fauxDisabled to true disabled on form.submit()
 // - DECIDED: When parent vis makes subs hidden, should they go to unchecked?   No, disabled!
 // - DECIDED: Should user be able to click on disabled vis to check the CB?  No, not important.
 // - DECIDED: Make vis changes "reshape" composite!  NOTE: Do we want to do this???   I am against this as too disruptive.  We may want to end reshaping in CGIs as well!
 // - DECIDED: Decide on a name (scm, subCfg, ? ) and then make a module file like ddcl.js.
 // - SOLVED: subtrack selected count is wrong!
 // - SOLVED: matCb unclick leaves vis/wrench enabled.  Vis/Wrench is usually only enabled for checked and visible.
-//  - Non-configurable will need to show/change vis independently! (separate vis control but no wrench)
+// - SOLVED: subCfg tracks going non-visible should have subCfg div closed.
+//  - Verify all composites work (hg19 hg18 mm9 mm8 panTro3 galGal3 braFlo1 dm3 ce10 homPan20 sacCer3 hg19Patch5 tested so far)
 //  - Speed up massive composites!  HAIB TFBS SYDH TFBS
-//  - Verify all composites work (conservation and SNPs are likely failures)
 //  - Remove debug code when ready
 // NOTE:
 // Current implementation relies upon '.' delimiter in name and no '_-' in name.  Nothing breaks rule yet...
 
 var subCfg = { // subtrack config module.
     //mySelf: null, // There is no need for a "mySelf" unless this object is being instantiated.
 
     // There is one instance and these vars are page wide
     compositeName: undefined,
     visIndependent: false,
     viewTags: [],
 
     markChange: function (eventObj, obj)
     { // Marks a control as having been changed by the user.  Naming will send value to cart.
       // Note this is often called directly as the onchange event function