22f809907e0828309f9d807bab3ba197b2a5f519 angie Tue May 29 10:59:36 2012 -0700 Bug #8048: Fix suggested by Tim: add a class to ldUi's inversion checkboxes,and detect that class when marking changes in subCfg.js. Also found an slSort that was missing & for its first arg, resulting in weird sorting of subtracks for inversion checkbox display. diff --git src/hg/js/subCfg.js src/hg/js/subCfg.js index 1341bd2..91c03c2 100644 --- src/hg/js/subCfg.js +++ src/hg/js/subCfg.js @@ -855,30 +855,35 @@ // DEBUG ------------- if (this.id != undefined && this.id.length > 0 && $(this).hasClass('filterBy') == false && $(this).hasClass('filterComp') == false) warn('DEBUG: Unexpected control with name ['+this.name + '], and id #'+this.id); // DEBUG ------------- Helps find unexpected problems. $(this).change(function (e) { subCfg.markChange(e,this); subCfg.propagateSetting(this); }); } }); } + // Bugfix #8048: Tim suggested using a special class to handle subtrack settings + // that appear alongside composite settings, as in ldUi (HapMap LD). + var subsInComp = $('select.subtrackInCompositeUi,input.subtrackInCompositeUi'); + $(subsInComp).each( function(ix, el) { + $(el).change(function(e) { subCfg.markChange(e, el); } ) }); // Because of fauxDisabled subCBs, it is necessary to truly disable them before submitting. $("FORM").submit(function (i) { $('input.subCB.changed.disabled').attr('disabled',true); // shadows will go to cart as they should // Unchanged controls will be disabled to avoid sending to the cart $('select,input').filter("[name]").not(".allOrOnly").not('.changed').each( function (i) { if (this.type != 'hidden' || $(this).hasClass('trPos') || $(this).hasClass('cbShadow') || $(this).hasClass('sortOrder')) { // hiddens except priority and boolshad are all sent to the cart // disable instead of unname because unname fills cart with a lot of garbage (Linux/FF)! this.disabled = true; } }); });