-
-
Notifications
You must be signed in to change notification settings - Fork 10.7k
fix(design-system): chart rendering #40306
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
This pull request has been ignored for the connected project Preview Branches by Supabase. |
|
Studio E2E Results
Artifacts: https://github.com/supabase/supabase/actions/runs/19266020664 Last updated: Tuesday 11, November, 2025 12:59:28 (UTC) |
charislam
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Checked charts in studio preview and looks OK, didn't look at design system
dnywh
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested /design-system/ and /studio/ locally. Works fine after some package wrangling.
I have read the CONTRIBUTING.md file.
YES
What kind of change does this PR introduce?
Our charts weren't rendering in our design system app due to a recharts mismatch from different packages. Monorepo things.
Note
Unifies recharts to 2.15.4 across packages, simplifies design-system component preview, updates LogsBarChart types/docs/demo, and expands ui-patterns exports.
catalog:→^2.15.4acrossapps/*andpackages/*with lockfile updates.dayjstoapps/design-system.components/component-preview.tsx: Simplify preview; remove tabbed view; add collapsible code section (with expand/collapse); streamline suspense fallback.logs-bar-chartdemo markeduse client; data includeswarning_count; doc type updated.ui-patterns/src/LogsBarChart: Type fix for tooltiplabelFormatterparam; usesdayjsformatting; minor styling/stack logic retained.Banners/*(incl.AnnouncementBanner,LW15Banner,index),CommandMenutelemetry hooks, extensiveFilterBar/*modules,McpUrlBuilder/*components/utilities,PopupFrame,Rowutilities/components, andCommandMenu/prepackaged/ai/useAiChat.test.Written by Cursor Bugbot for commit 440b017. This will update automatically on new commits. Configure here.