Skip to content

Commit

Permalink
double click is going to be harder than it first looked
Browse files Browse the repository at this point in the history
  • Loading branch information
danvk committed Oct 27, 2015
1 parent f13f76c commit 9402504
Showing 1 changed file with 30 additions and 2 deletions.
32 changes: 30 additions & 2 deletions src/main/VisualizationWrapper.js
Original file line number Diff line number Diff line change
Expand Up @@ -5,11 +5,14 @@

var React = require('react'),
ReactDOM = require('react-dom'),
types = require('./react-types'),
d3utils = require('./d3utils'),
_ = require('underscore'),
d3 = require('../lib/minid3');

var types = require('./react-types'),
d3utils = require('./d3utils'),
utils = require('./utils'),
Interval = require('./Interval');

class VisualizationWrapper extends React.Component {
hasDragBeenInitialized: boolean;

Expand Down Expand Up @@ -84,14 +87,32 @@ class VisualizationWrapper extends React.Component {
.on('dragend', dragended);

d3.select(div).call(drag).on('click', this.handleClick.bind(this));

// This is saved in a member var for later removal via removeEventListener.
// this.dblClickListener = e => {
// e.preventDefault();
// this.handleDoubleClick();
// };
// div.addEventListener('dblclick', this.dblClickListener);
}

handleClick(): any {
if (d3.event.defaultPrevented) {
d3.event.stopPropagation();
} else {
}
}

handleDoubleClick(): any {
var r = this.props.range;
var iv = utils.scaleRange(new Interval(r.start, r.stop), 0.5);
this.props.onRangeChange({
contig: r.contig,
start: iv.start,
stop: iv.stop
});
}

render(): any {
var range = this.props.range;
var component = this.props.visualization.component;
Expand All @@ -110,6 +131,13 @@ class VisualizationWrapper extends React.Component {

return <div className='drag-wrapper'>{el}</div>;
}

componentWillUnmount() {
if (this.dblClickListener) {
var div = ReactDOM.findDOMNode(this);
div.removeEventListener('dblclick', this.dblClickListener);
}
}
}
VisualizationWrapper.displayName = 'VisualizationWrapper';

Expand Down

0 comments on commit 9402504

Please sign in to comment.