VanessaGiannoni commented on code in PR #37284:
URL: https://github.com/apache/superset/pull/37284#discussion_r2714640819
##########
superset-frontend/plugins/plugin-chart-echarts/src/Timeseries/EchartsTimeseries.tsx:
##########
@@ -259,6 +259,18 @@ export default function EchartsTimeseries({
? getCrossFilterDataMask(seriesName)
: undefined,
});
+
+ // Restore tooltip visibility after the drill menu closes
+ const restoreTooltip = () => {
+ const instance = echartRef.current?.getEchartInstance?.();
+ if (instance) {
Review Comment:
As we discussed in the standup, I took a look at the `useContextMenu` hook
and gave a few trying fixes. However, after expending a few hours debugging
locally, I spotted the root cause. Probably it's a regression in the
`ChartContextMenu` (?) after adding the validation on line 426, it worked for
all charts.
cc: @msyavuz
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
