Set the initial pileup range correctly

hammerlab · Jun 12, 2015 · 5866d0a · 5866d0a
1 parent 25e6364
commit 5866d0a
Showing 1 changed file with 9 additions and 12 deletions.
diff --git a/cycledash/static/js/examine/components/PileupViewer.js b/cycledash/static/js/examine/components/PileupViewer.js
@@ -132,28 +132,25 @@ var PileupViewer = React.createClass({
     var pileupEl = this.refs.pileupElement.getDOMNode();
 
     this.browser = pileup.create(pileupEl, {
-      range: {
-        contig: '20',  // random position -- it will be changed.
-        start:  2684736 - 50,
-        stop:   2684736 + 50
-      },
+      range: this.rangeForRecord(this.props.selectedRecord),
       tracks: sources
     });
   },
   panToSelection: function() {
-    var rec = this.props.selectedRecord;
-    this.browser.setRange({
-      contig: rec.contig,
-      start: rec.position - 25,
-      stop: rec.position + 25
-    });
+    this.browser.setRange(this.rangeForRecord(this.props.selectedRecord));
   },
   update: function() {
     if (this.props.isOpen && this.props.selectedRecord) {
       this.lazilyCreateDalliance();
-      this.panToSelection();
     }
   },
+  rangeForRecord: function(record) {
+    return {
+      contig: record.contig,
+      start: record.position - 25,
+      stop: record.position + 25
+    };
+  },
   fetchIndexChunks: function() {
     var propBamPathPairs = [['normalBaiChunks', this.props.normalBamPath],
                             ['tumorBaiChunks', this.props.tumorBamPath]];