Skip to content

[UIK-5092][d3-chart] added responsiveness for charts by default#2983

Open
ilyabrower wants to merge 21 commits into
release/v17from
UIK-5092/reposiveness-charts
Open

[UIK-5092][d3-chart] added responsiveness for charts by default#2983
ilyabrower wants to merge 21 commits into
release/v17from
UIK-5092/reposiveness-charts

Conversation

@ilyabrower

Copy link
Copy Markdown
Contributor

Changelog

@semcore/d3-chart

Added

  • Responsive container for all charts by default.

Motivation and Context

How has this been tested?

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue).
  • New feature (non-breaking change which adds functionality).
  • Breaking change (fix or feature that would cause existing functionality to not work as expected).
  • Nice improve.

Checklist:

  • I have updated the documentation accordingly.
  • I have added new tests on added of fixed functionality.

@ilyabrower ilyabrower self-assigned this Jun 4, 2026
@ilyabrower ilyabrower force-pushed the UIK-5092/reposiveness-charts branch from 86b79f3 to 3e2482f Compare June 4, 2026 15:55

@j-mnizhek j-mnizhek left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cool!

  1. Updated chart documentation to clarify that charts are responsive by default
  2. Added a story demonstrating responsive charts by default
  3. Updated story names for responsive charts + links in test files

return (
<Chart.Area
w='100%'
h={200}

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it's better to add a wrapper here. In real apps, users will specify the sizes of parent elements

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