Skip to content

Conversation

@Anush2303
Copy link
Contributor

New Behavior

Fix double padding issue in scatter chart

Related Issue(s)

  • Fixes #

@Anush2303 Anush2303 marked this pull request as ready for review November 18, 2025 10:49
@Anush2303 Anush2303 requested a review from a team as a code owner November 18, 2025 10:49
@github-actions
Copy link

github-actions bot commented Nov 18, 2025

📊 Bundle size report

Package & Exports Baseline (minified/GZIP) PR Change
react-charts
AreaChart
401.451 kB
122.847 kB
401.558 kB
122.889 kB
107 B
42 B
react-charts
DeclarativeChart
719.091 kB
208.156 kB
719.478 kB
208.277 kB
387 B
121 B
react-charts
LineChart
410.721 kB
124.268 kB
410.996 kB
124.362 kB
275 B
94 B
react-charts
ScatterChart
390.503 kB
118.5 kB
390.778 kB
118.591 kB
275 B
91 B
react-charts
VerticalBarChart
425.562 kB
123.604 kB
425.661 kB
123.64 kB
99 B
36 B
react-charts
VerticalStackedBarChart
397.348 kB
119.51 kB
397.447 kB
119.549 kB
99 B
39 B
Unchanged fixtures
Package & Exports Size (minified/GZIP)
react-charts
DonutChart
297.339 kB
88.898 kB
react-charts
FunnelChart
288.594 kB
85.727 kB
react-charts
GanttChart
383.208 kB
116.015 kB
react-charts
GaugeChart
312.216 kB
92.539 kB
react-charts
GroupedVerticalBarChart
390.996 kB
118.557 kB
react-charts
HeatMapChart
385.288 kB
117.724 kB
react-charts
HorizontalBarChart
294.96 kB
86.77 kB
react-charts
HorizontalBarChartWithAxis
63 B
83 B
react-charts
Legends
234.066 kB
69.154 kB
react-charts
SankeyChart
199.518 kB
61.371 kB
react-charts
Sparkline
91.385 kB
28.704 kB
🤖 This report was generated against a82a2d210b1a7b96c53a12dd819d896752439563

@github-actions
Copy link

Pull request demo site: URL

@Anush2303
Copy link
Contributor Author

@AtishayMsft
Copy link
Contributor

There is no change in playwright report. Could you add bunch of examples in test set.

@Anush2303
Copy link
Contributor Author

There is no change in playwright report. Could you add bunch of examples in test set.

I think it will fail for the cases where we have xMin and xMax coming from schema. And we might not have those cases in our test set

@Anush2303
Copy link
Contributor Author

Let me run the report again after taking in the xMin xMax changes.

@AtishayMsft
Copy link
Contributor

Let me run the report again after taking in the xMin xMax changes.

There are no changes in the playwright report with xMin/xMax changes also.

@Anush2303 Anush2303 force-pushed the usr/agupta/v9/fixDoublePaddingIssue branch from 38ef576 to 298df8b Compare November 19, 2025 12:08
@Anush2303
Copy link
Contributor Author

Anush2303 commented Nov 19, 2025

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants