Skip to content

fix(dashboards): Glitching bar charts on url changes#82457

Merged
nikkikapadia merged 1 commit into
masterfrom
nikki/fix/glitchy-bar-charts
Dec 20, 2024
Merged

fix(dashboards): Glitching bar charts on url changes#82457
nikkikapadia merged 1 commit into
masterfrom
nikki/fix/glitchy-bar-charts

Conversation

@nikkikapadia

Copy link
Copy Markdown
Member

The glitching bar charts every time the url changes was annoying and it was happening due to the bar chart animations (where it rises from 0). I turned off the animations for dashboard bar charts.

I've also snuck in a fix for the widget preview height on big number widgets.

@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Dec 20, 2024
@nikkikapadia nikkikapadia marked this pull request as ready for review December 20, 2024 15:22
@nikkikapadia nikkikapadia requested review from a team and narsaynorath December 20, 2024 15:22
@narsaynorath

Copy link
Copy Markdown
Member

Just wanted to add that I'm still seeing a flicker when I do stuff like type in the name field, we'll try to look at getting rid of that but this feels fine for us to do now to prevent the impact of the re-rendering

@nikkikapadia nikkikapadia merged commit 0949dd8 into master Dec 20, 2024
@nikkikapadia nikkikapadia deleted the nikki/fix/glitchy-bar-charts branch December 20, 2024 16:03
andrewshie-sentry pushed a commit that referenced this pull request Jan 2, 2025
The glitching bar charts every time the url changes was annoying and it
was happening due to the bar chart animations (where it rises from 0). I
turned off the animations for dashboard bar charts.

I've also snuck in a fix for the widget preview height on big number
widgets.
@github-actions github-actions Bot locked and limited conversation to collaborators Jan 5, 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.

2 participants