Skip to content
Snippets Groups Projects
Commit 5f5bb142 authored by Tom Robinson's avatar Tom Robinson
Browse files

Don't set 'hovered.event' automatically since we do it within the charts that...

Don't set 'hovered.event' automatically since we do it within the charts that want the tooltip to follow the mouse now. Resolves #2145
parent 1fa6cf92
No related branches found
No related tags found
No related merge requests found
......@@ -4,7 +4,6 @@ import visualizations from "metabase/visualizations";
import i from "icepick";
import _ from "underscore";
import d3 from "d3";
export default class Visualization extends Component {
constructor(props, context) {
......@@ -63,9 +62,6 @@ export default class Visualization extends Component {
onHoverChange(hovered) {
const { renderInfo } = this.state;
if (hovered) {
if (!hovered.event) {
hovered = i.assoc(hovered, "event", d3.event);
}
// if we have Y axis split info then find the Y axis index (0 = left, 1 = right)
if (renderInfo && renderInfo.yAxisSplit) {
const axisIndex = _.findIndex(renderInfo.yAxisSplit, (indexes) => _.contains(indexes, hovered.index));
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment