a8be2f912e75d724a5d0d313e8488e82fd28e4ec
kate
  Fri Mar 5 12:32:20 2021 -0800
Retire 'exit multi-region' button idea.  With multi-region button on position line and more informative mouseovers and popups it should be easier now for new multi-region users to exit. refs #26385

diff --git src/hg/hgTracks/hgTracks.c src/hg/hgTracks/hgTracks.c
index af3f100..9c5afee 100644
--- src/hg/hgTracks/hgTracks.c
+++ src/hg/hgTracks/hgTracks.c
@@ -8635,46 +8635,33 @@
 
 
     /* Make line that says position. */
 	{
 	char buf[256];
 	char *survey = cfgOptionEnv("HGDB_SURVEY", "survey");
 	char *surveyLabel = cfgOptionEnv("HGDB_SURVEY_LABEL", "surveyLabel");
         char *javascript = "document.location = '/cgi-bin/hgTracks?db=' + document.TrackForm.db.options[document.TrackForm.db.selectedIndex].value;";
         if (containsStringNoCase(database, "zoo"))
             {
             hPuts("Organism ");
             printAssemblyListHtmlExtra(database, "change", javascript);
             }
 
         /* Multi-region button on position line */
-        if (sameString(virtModeType, "default"))
-            {
         hButtonMaybePressed("hgTracksConfigMultiRegionPage", "multi-region",
                     "Configure view in multi-region display mode",
                     "popUpHgt.hgTracks('multi-region config'); return false;", FALSE);
-            }
-        else
-            {
-            char buf[100];
-            safef(buf, sizeof buf, "exit multi-region (%s) mode and return to normal display",
-                (sameString(virtModeType, "exonMostly") ? "exon-only" :
-                (sameString(virtModeType, "geneMostly") ? "gene-only" :
-                (sameString(virtModeType, "singleAltHaplo") ? "alternative haplotype":
-                                                "custom regions"))));
-            hButtonWithMsg("hgt.exitMultiRegion", "exit multi-region", buf);
-            }
         hPrintf(" ");
 
 	if (virtualSingleChrom()) // DISGUISE VMODE
 	    safef(buf, sizeof buf, "%s", windowsSpanPosition());
 	else
 	    safef(buf, sizeof buf, "%s:%ld-%ld", virtChromName, virtWinStart+1, virtWinEnd);
 	
 	position = cloneString(buf);
         char *pressedClass = "", *showVirtRegions = "";
         if (differentString(virtModeType, "default"))
             {
             pressedClass = "pressed";
             showVirtRegions = "show multi-region position ranges and ";
             }
 	hPrintf("<span class='positionDisplay %s' id='positionDisplay' "
@@ -9520,37 +9507,30 @@
 	positionIsVirt = TRUE;
 	}
     }
 
 if (sameString(position, ""))
     {
     hUserAbort("Please go back and enter a coordinate range or a search term in the \"search term\" field.<br>For example: chr22:20100000-20200000.\n");
     }
 
 if (!positionIsVirt)
     {
     if (! resolvePosition(&position))
         return;
     }
 
-if  (cgiVarExists("hgt.exitMultiRegion"))
-    {
-    cartRemove(cart, "hgt.exitMultiRegion");
-    cartSetString(cart, "virtModeType", "default");
-    cartSetBoolean(cart, "virtMode", FALSE);
-    }
-
 virtMode = cartUsualBoolean(cart, "virtMode", FALSE);
 
 /* Figure out basic dimensions of display.  This
  * needs to be done early for the sake of the
  * zooming and dinking routines. */
 setLayoutGlobals();
 
 virtModeType = cartUsualString(cart, "virtModeType", virtModeType);
 
 if (positionIsVirt && virtualSingleChrom())
     {
     // we need chromName to be set before initRegionList() gets called.
     position = cartUsualString(cart, "nonVirtPosition", "");
     if (!sameString(position,""))
 	parseNonVirtPosition(position);