Skip to content

fix: fix vega chart sizing#365

Merged
ilyabo merged 2 commits intomainfrom
fix/fix-vega-chart-sizing
Feb 17, 2026
Merged

fix: fix vega chart sizing#365
ilyabo merged 2 commits intomainfrom
fix/fix-vega-chart-sizing

Conversation

@bdjulbic
Copy link
Collaborator

@bdjulbic bdjulbic commented Feb 13, 2026 •
edited by coderabbitai bot
Loading

Limit the boundary of Vega chart tool result

Summary by CodeRabbit

  • Style
    • Chart containers now use full available width with horizontal scrolling enabled, improving layout flexibility and visibility of larger visualizations.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 13, 2026 •
edited
Loading

📝 Walkthrough

Walkthrough

CSS class names updated in the Vega chart component to add width constraints and horizontal scrolling capability. The outer wrapper and inner chart region now enforce maximum width and enable overflow handling without modifying any functional logic or state management.

Changes

Cohort / File(s) Summary
Layout Constraints
packages/vega/src/VegaChartToolResult.tsx		 Added max-w-full to outer wrapper and updated inner chart region with max-w-full overflow-x-auto for responsive layout and horizontal scrolling support.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 With a hop and a bound, our charts now align,
Max-width constraints make the layout divine,
Scroll left, scroll right with overflow ease,
The bunny's small tweaks bring responsive peace! ✨

🚥 Pre-merge checks | ✅ 4
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'fix: fix vega chart sizing' is directly related to the changeset, which modifies layout constraints for Vega chart sizing using max-width and overflow properties.
Docstring Coverage ✅ Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.
Merge Conflict Detection ✅ Passed ✅ No merge conflicts detected when merging into main

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/fix-vega-chart-sizing

No actionable comments were generated in the recent review. 🎉

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@ilyabo ilyabo merged commit c9ed288 into main Feb 17, 2026
4 checks passed
@ilyabo ilyabo deleted the fix/fix-vega-chart-sizing branch February 17, 2026 12:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ilyabo ilyabo ilyabo approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@bdjulbic @ilyabo