bec2520f003487171561e0f0751fdb2940c03dc9 galt Thu Apr 14 22:28:23 2016 -0700 Fixes missing &r=winStart&l=winEnd which TimD had removed as a step towards his dynamic portal view, but which we are not using, and it messes up hgc clicks for multi-region. refs #16898 diff --git src/hg/hgTracks/expRatioTracks.c src/hg/hgTracks/expRatioTracks.c index e02fd5a..a3802a8 100644 --- src/hg/hgTracks/expRatioTracks.c +++ src/hg/hgTracks/expRatioTracks.c @@ -171,32 +171,33 @@ } void mapBoxHcTwoItems(struct hvGfx *hvg, int start, int end, int x, int y, int width, int height, char *track, char *item1, char *item2, char *statusLine) /* Print out image map rectangle that would invoke the htc (human track click) * program. */ { char *encodedItem1 = cgiEncode(item1); char *encodedItem2 = cgiEncode(item2); x = hvGfxAdjXW(hvg, x, width); if (theImgBox && curImgTrack) { char link[512]; - safef(link,sizeof(link),"%s&c=%s&o=%d&t=%d&g=%s&i=%s&i2=%s", // NOTE: winStart,winEnd removed due to portal - hgcNameAndSettings(), chromName, start, end, track, encodedItem1, encodedItem2); + // NOTE: winStart,winEnd removed due to portal (added back by Galt for multi-region) + safef(link,sizeof(link),"%s&c=%s&l=%d&r=%d&o=%d&t=%d&g=%s&i=%s&i2=%s", + hgcNameAndSettings(), chromName, winStart, winEnd, start, end, track, encodedItem1, encodedItem2); #ifdef IMAGEv2_SHORT_MAPITEMS if (x < insideX && x+width > insideX) warn("mapBoxHcTwoItems(%s) map item spanning slices. LX:%d TY:%d RX:%d BY:%d link:[%s]", track,x, y, x+width, y+height, link); #endif//def IMAGEv2_SHORT_MAPITEMS imgTrackAddMapItem(curImgTrack,link,statusLine,x, y, x+width, y+height, track); } else { hPrintf("\n", statusLine); } @@ -1424,31 +1425,32 @@ probes = marrays->features; nProbes = slCount(probes); nExps = slCount(marrays); totalHeight = nExps * lineHeight; if ((nProbes > MICROARRAY_CLICK_LIMIT) && !(sameString(tg->table, "expRatioUCSFDemo") || sameString(tg->table, "cnvLungBroadv2") || sameString(tg->table, "CGHBreastCancerUCSF") || sameString(tg->table, "expBreastCancerUCSF")) ) { int xOffRc = hvGfxAdjXW(hvg, xOff, insideWidth); if (theImgBox && curImgTrack) { char link[512]; #if defined(IMAGEv2_DRAG_SCROLL_SZ) && (IMAGEv2_DRAG_SCROLL_SZ > 1) // Tim isn't sure what to do here if/when we implement drag scroll. warn("Tim take a look at this link and whether it needs different winStart and winEnd values with dragScroll > 1."); #endif - safef(link,sizeof(link),"%s&c=%s&o=%d&t=%d&g=%s&i=zoomInMore", hgcNameAndSettings(), chromName, seqStart, seqEnd, tg->track); + // Galt added back winStart, winEnd for multi-region. + safef(link,sizeof(link),"%s&c=%s&l=%d&r=%d&o=%d&t=%d&g=%s&i=zoomInMore", hgcNameAndSettings(), chromName, winStart, winEnd, seqStart, seqEnd, tg->track); #ifdef IMAGEv2_SHORT_MAPITEMS if (xOffRc < insideX && xOffRc+insideWidth > insideX) warn("expRatioMapBoxes(%s) map item spanning slices. LX:%d TY:%d RX:%d BY:%d " "link:[%s]",tg->track,xOffRc, y, xOffRc+insideWidth, y+totalHeight, link); #endif//def IMAGEv2_SHORT_MAPITEMS imgTrackAddMapItem(curImgTrack,link,"zoomInMore",xOffRc, y, xOffRc+insideWidth, y+totalHeight, tg->track); } else { hPrintf("track, chromName, winStart, winEnd, database); hPrintf("TITLE=\"zoomInMore\">\n"); }