Change Zoom tool to a selection box

josiahseaman / skittle

DNASkittle is a web based genome browser that provides an interactive visualization of the raw sequence. Every nucleotide of 10Kbp can fit on one screen at a time. It makes heavy use of signal processing techniques to extract tandem repeats, and other genome elements.

GNU General Public License v2.0

7 stars 4 forks source link

Change Zoom tool to a selection box #14

Closed josiahseaman closed 12 years ago

josiahseaman commented 12 years ago

Zoom tool is hard to use right now. Make it a click and drag box that zoom into the selected region.

josiahseaman commented 12 years ago

After finishing this Issue, proceed to use the same selection code for Issue #71.

josiahseaman commented 12 years ago

add pair<int,int> AbstractGraph::getIndicesFromZoomEvent(point2D pt)
- this is the zoom equivalent of string AbstractGraph::SELECT_MouseClick(point2D pt)
- internally this uses int AbstractGraph::getRelativeIndexFromMouseClick twice
glWidget calls getIndicesFromZoomEvent(point2D pt) on each graph similar to GlWidget::783
- use the returned indices to call GlWidget::zoomRange(start, stop)

josiahseaman commented 12 years ago

The only part remaining is the xOffset for drawSelectionBox is not properly shifted for glWidget layout. (Test this with an AnnotationDisplay up.) You'll probably want to not display a selectionBox until one of the graphs returns a positive hit on rangeOverlap. Then use the xOffset of that graph. This will be calculated cumulatively as it is done several places in the code: i.e. GlWidget:616