Skip to content

Commit

Permalink
Append mouse event nodes before chart elements (#2359)
Browse files Browse the repository at this point in the history
Mouse event nodes were appended last, which covered up the chart elements
so that their events weren't firing correctly.

Fixes #2329 and #2354
  • Loading branch information
kcrwfrd authored and kt3k committed Jul 10, 2018
1 parent 46cf7c6 commit 5ba965a
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions src/core.js
Expand Up @@ -244,6 +244,9 @@ c3_chart_internal_fn.initWithData = function (data) {
// Grid lines
if (config.grid_lines_front) { $$.initGridLines(); }

// Cover whole with rects for events
$$.initEventRect();

// Define g for chart
$$.initChartElements();

Expand All @@ -253,9 +256,6 @@ c3_chart_internal_fn.initWithData = function (data) {
// Set targets
$$.updateTargets($$.data.targets);

// Cover whole with rects for events
$$.initEventRect();

// Set default extent if defined
if (config.axis_x_selection) { $$.brush.selectionAsValue($$.getDefaultSelection()); }

Expand Down

0 comments on commit 5ba965a

Please sign in to comment.