Skip to content

Commit

Permalink
fix(tooltip): multi chart when mounting to body
Browse files Browse the repository at this point in the history
  • Loading branch information
pearmini committed Jul 7, 2023
1 parent 9418bdd commit e68a222
Show file tree
Hide file tree
Showing 4 changed files with 91 additions and 6 deletions.
29 changes: 29 additions & 0 deletions __tests__/integration/api-chart-tooltip-multi-chart.spec.ts
@@ -0,0 +1,29 @@
import { chartTooltipMultiChart as render } from '../plots/api/chart-tooltip-multi-chart';
import { dispatchFirstElementEvent } from './utils/event';
import './utils/useSnapshotMatchers';
import { createDOMGCanvas } from './utils/createDOMGCanvas';

describe('chart.interaction.tooltip', () => {
const canvas1 = createDOMGCanvas(640, 480);
const canvas2 = createDOMGCanvas(640, 480);

it('tooltip should not be shared if mount to body.', async () => {
const { finished0, finished1 } = render({
canvas1,
canvas2,
container: document.createElement('div'),
});
await Promise.all([finished0, finished1]);

dispatchFirstElementEvent(canvas1, 'pointerover');
dispatchFirstElementEvent(canvas2, 'pointerover');
expect(
Array.from(document.body.getElementsByClassName('g2-tooltip')).length,
).toBe(2);
});

afterAll(() => {
canvas1?.destroy();
canvas2?.destroy();
});
});
51 changes: 51 additions & 0 deletions __tests__/plots/api/chart-tooltip-multi-chart.ts
@@ -0,0 +1,51 @@
import { Chart } from '../../../src';

export function chartTooltipMultiChart(context) {
const { container, canvas1, canvas2 } = context;

const options = {
type: 'interval',
data: [
{ genre: 'Sports', sold: 275 },
{ genre: 'Strategy', sold: 115 },
{ genre: 'Action', sold: 120 },
{ genre: 'Shooter', sold: 350 },
{ genre: 'Other', sold: 150 },
],
encode: {
x: 'genre',
y: 'sold',
},
interaction: { tooltip: { mount: 'body' } },
};

// View0
const view0Container = document.createElement('div');
container.appendChild(view0Container);

const view0 = new Chart({
theme: 'classic',
container: view0Container,
canvas: canvas1,
});

view0.options(options);

const finished0 = view0.render();

// View1.
const view1Container = document.createElement('div');
container.appendChild(view1Container);

const view1 = new Chart({
theme: 'classic',
container: view1Container,
canvas: canvas2,
});

view1.options(options);

const finished1 = view1.render();

return { finished0, finished1 };
}
1 change: 1 addition & 0 deletions __tests__/plots/api/index.ts
Expand Up @@ -38,3 +38,4 @@ export { chartOptionsCompositeMark } from './chart-options-composite-mark';
export { chartEmitItemTooltipHideContent } from './chart-emit-item-tooltip-hide-content';
export { chartEmitClickTooltip } from './chart-emit-click-tooltip';
export { chartChangeDataLegend } from './chart-change-data-legend';
export { chartTooltipMultiChart } from './chart-tooltip-multi-chart';
16 changes: 10 additions & 6 deletions src/interaction/tooltip.ts
Expand Up @@ -106,9 +106,12 @@ function showTooltip({
mount,
bounding,
}) {
// All the views share the same tooltip.
const canvasContainer = root.getRootNode().defaultView.getConfig().container;
const container = single ? getContainer(root, mount) : root;
const container = getContainer(root, mount);

// All the views share the same tooltip.
const parent = single ? canvasContainer : root;

const b = bounding || getBounding(root);
const containerOffset = getContainerOffset(canvasContainer, container);
const {
Expand All @@ -121,7 +124,7 @@ function showTooltip({
b,
containerOffset,
),
} = container;
} = parent as any;
const { items, title = '' } = data;
tooltipElement.update({
x,
Expand All @@ -134,15 +137,16 @@ function showTooltip({
content: render(event, { items, title }),
}),
});
container.tooltipElement = tooltipElement;
parent.tooltipElement = tooltipElement;
}

function hideTooltip({ root, single, emitter, nativeEvent = true, mount }) {
if (nativeEvent) {
emitter.emit('tooltip:hide', { nativeEvent });
}
const container = single ? getContainer(root, mount) : root;
const { tooltipElement } = container;
const canvasContainer = root.getRootNode().defaultView.getConfig().container;
const parent = single ? canvasContainer : root;
const { tooltipElement } = parent;
if (tooltipElement) {
tooltipElement.hide();
}
Expand Down

0 comments on commit e68a222

Please sign in to comment.