[Chart] Toggling legends in Stacked Bar Chart v2 does not update total values

[ktmud]

Time-series Bar Chart v2 does not update total values for stacked bar chart when toggling legends.

How to reproduce the bug

1. Create a "Time-series Bar Chart v2"
2. Go to "Customize" and select "Show value", "Stack series" and "Only total"
3. Toggle series in legends
4. The total value should update but it doesn't

The legacy Time-series Bar Chart does not have this issue.

Screenshots

stacked-value-total-update.mp4

Environment

Latest master

Checklist

Make sure to follow these steps before submitting your issue - thank you!

• I have checked the superset logs for python stacktraces and included it here as text if there are any.
• I have reproduced the issue with at least the latest released version of superset.
• I have checked the issue tracker for the same issue and I haven't found one similar.

[RaymondBakerIBM]

Hi I've investigated this bug, from what I can tell there's no way to fix this without waiting for echarts to add the total value bar functionality or making our own legend

Since legend selection is handled by echarts the only way for superset to know what is selected is the legendselectchanged callback. This gets called after the label formatters get called. Causing issues with both useRef and useState solns.

In both the below I was overloading the label formatter for each series.

I tried using a react State in the formatter to hold the selected legend values, this caused the graph to render twice undoing the hidden bar in the process. Once before and once after the value update

I also tried using a react Ref in the formatter this cased the number to not reflect the actual data selected because legendselectchanged gets called after the label formatters.

I think one other option is to give EchartsTimeseries a way to give the newly selected legends to transformers.ts causing a redraw but with the current SuperChartCore.tsx I don't see a way to do that

[TonyBhargav]

@ktmud @RaymondBakerIBM If no one is assigned to this issue or no one is working on this issue, I would like to work on this.

[michael-s-molina]

Fixed by #24477