Skip to content

fix(explore): Fix chart resizing on browser window change - #102876

Merged
gggritso merged 1 commit into
masterfrom
georgegritsouk/dain-971-explore-charts-fail-to-resize-on-browser-window-change
Nov 6, 2025
Merged

fix(explore): Fix chart resizing on browser window change#102876
gggritso merged 1 commit into
masterfrom
georgegritsouk/dain-971-explore-charts-fail-to-resize-on-browser-window-change

Conversation

@gggritso

@gggritso gggritso commented Nov 6, 2025

Copy link
Copy Markdown
Member

There's a bug in Explore charts, where the chart doesn't resize correctly when the browser size changes. This bug is caused by a wacky race condition in echarts-for-react, which was fixed in hustcc/echarts-for-react#613 by yours truly, and made its way into 3.0.5

This has a bug fix that affects our resizing behaviour.
@gggritso
gggritso requested review from a team and billyvg November 6, 2025 17:11
@gggritso
gggritso requested a review from a team as a code owner November 6, 2025 17:11
@linear

linear Bot commented Nov 6, 2025

Copy link
Copy Markdown

@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Nov 6, 2025
@gggritso
gggritso merged commit a9b56d6 into master Nov 6, 2025
45 checks passed
@gggritso
gggritso deleted the georgegritsouk/dain-971-explore-charts-fail-to-resize-on-browser-window-change branch November 6, 2025 17:27
andrewshie-sentry pushed a commit that referenced this pull request Nov 13, 2025
There's a bug in Explore charts, where the chart doesn't resize
correctly when the browser size changes. This bug is caused by a wacky
race condition in `echarts-for-react`, which was fixed in
hustcc/echarts-for-react#613 by yours truly, and
made its way into 3.0.5
@github-actions github-actions Bot locked and limited conversation to collaborators Nov 22, 2025
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.

3 participants