6741dd95c5ee863e7edae4ed20348f2c0cc3dab6 tdreszer Fri Feb 3 16:54:19 2012 -0800 For unknown reasons IE8 fails to find '#pop' occasionally. This fixes 6723 diff --git src/hg/js/hgTracks.js src/hg/js/hgTracks.js index 407c1db..7153642 100644 --- src/hg/js/hgTracks.js +++ src/hg/js/hgTracks.js @@ -2337,31 +2337,31 @@ error: errorHandler, cmd: rightClick.selectedMenuItem, cache: false }); }, hgTrackUi: function (trackName,descriptionOnly) { waitOnFunction( popUp._uiDialigRequest, trackName, descriptionOnly ); // Launches the popup but shields the ajax with a waitOnFunction }, uiDialogOk: function (popObj, trackName) { // When hgTrackUi Cfg popup closes with ok, then update cart and refresh parts of page var rec = hgTracks.trackDb[trackName]; var subtrack = tdbIsSubtrack(rec) ? trackName :undefined; // If subtrack then vis rules differ - var allVars = getAllVars($('#pop'), subtrack ); + var allVars = getAllVars($('#hgTrackUiDialog'), subtrack );// For unknown reasons IE8 fails to find $('#pop'), occasionally var changedVars = varHashChanges(allVars,popUp.saveAllVars); //warn("cfgVars:"+varHashToQueryString(changedVars)); var newVis = changedVars[trackName]; var hide = (newVis != null && (newVis == 'hide' || newVis == '[]')); // subtracks do not have "hide", thus '[]' if($('#imgTbl') == undefined) { // On findTracks or config page setVarsFromHash(changedVars); //if(hide) // TODO: When findTracks or config page has cfg popup, then vis change needs to be handled in page here } else { // On image page if(hide) { setVarsFromHash(changedVars); $(document.getElementById('tr_' + trackName)).remove(); dragReorder.init(); dragSelect.load(false); } else {