Skip to content

fix(dashboards): Bump echarts-for-react to fix legend event listeners - #106678

Merged
gggritso merged 1 commit into
masterfrom
georgegritsouk/dain-1144-widget-legend-repeatedly-updates-the-url
Jan 21, 2026
Merged

fix(dashboards): Bump echarts-for-react to fix legend event listeners#106678
gggritso merged 1 commit into
masterfrom
georgegritsouk/dain-1144-widget-legend-repeatedly-updates-the-url

Conversation

@gggritso

Copy link
Copy Markdown
Member

Fixes DAIN-1144

Upgrades echarts-for-react from 3.0.5 to 3.0.6 to fix an issue where legend event listeners were accumulating on component re-renders. The library was attempting to unbind listeners using the original function references, but the actual bound handlers were wrapped functions, causing old listeners to remain attached. This led to dashboard widget legends repeatedly updating the URL because multiple stale handlers were firing on each interaction.

Version 3.0.6 fixes this by properly tracking the wrapped handler references and unbinding them before attaching new ones. See hustcc/echarts-for-react#618 for details.

Upgrades echarts-for-react from 3.0.5 to 3.0.6 to fix an issue where
legend event listeners were not properly cleaned up, causing the URL
to be repeatedly updated on dashboard widgets.
@gggritso
gggritso requested a review from a team as a code owner January 21, 2026 17:09
@linear

linear Bot commented Jan 21, 2026

Copy link
Copy Markdown

@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Jan 21, 2026
@gggritso
gggritso merged commit 3f2aeed into master Jan 21, 2026
51 checks passed
@gggritso
gggritso deleted the georgegritsouk/dain-1144-widget-legend-repeatedly-updates-the-url branch January 21, 2026 18:15
@github-actions github-actions Bot locked and limited conversation to collaborators Feb 6, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants