feat(component): add DataZoom support for bar and line charts

[alfredo1996]

Summary

Add scroll-to-zoom (DataZoom) as a shared BaseChart prop for bar and line charts. ECharts DataZoom (type: 'inside') enables exploration of large datasets by scrolling to zoom on both axes.

Changes

• New prop: enableDataZoom on BaseChartProps (types.ts)
• BaseChart: injects dataZoom: [{type: 'inside', xAxisIndex: 0}, {type: 'inside', yAxisIndex: 0}] when enabled
• chart-options-schema: new "Enable Scroll Zoom" option for bar/line charts
• 3 new tests: DataZoom injection, default off, explicit false

Test plan

• cd component && npm test — 67 suites, 1012 tests passing
• cd app && npm test — 72 suites, 1260 tests passing
• npm run build — clean
• npm run lint — clean

Closes #134

🤖 Generated with Claude Code

Summary by CodeRabbit

• New Features

  • Added data zoom capability to bar and line charts, enabling scroll-to-zoom interaction for intuitive data axis exploration.

• Tests

  • Added unit tests for data zoom configuration behavior across multiple scenarios (enabled, disabled, and omitted states).

[coderabbitai]

Warning

Rate limit exceeded

@alfredo1996 has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 23 minutes and 49 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 5c0f4391-1a07-4fd3-ae4e-2206e3e4fc41

📥 Commits

Reviewing files that changed from the base of the PR and between 7dd15c4 and 670bd51.

📒 Files selected for processing (3)

• component/src/charts/__tests__/base-chart.test.tsx
• component/src/charts/base-chart.tsx
• component/vitest.setup.ts

Walkthrough

A new enableDataZoom boolean prop was added to BaseChart, allowing users to enable ECharts inside-type data zooming for bar and line charts. The feature conditionally injects zoom configuration into chart options, includes proper TypeScript types, and is exposed through the chart options schema with unit test coverage.

Changes

Cohort / File(s)	Summary
BaseChart Implementation component/src/charts/base-chart.tsx, component/src/charts/types.ts	Added enableDataZoom prop to BaseChartProps and implemented conditional injection of ECharts dataZoom configuration (type: 'inside') for xAxis and yAxis. Updated effect dependencies to trigger option recomputation when prop changes.
Chart Options Schema component/src/components/composed/chart-options-schema.ts	Registered new dataZoomOptions containing the enableDataZoom option and integrated it into the option registry for bar and line chart types.
Unit Tests component/src/charts/__tests__/base-chart.test.tsx	Added three test cases validating dataZoom configuration: undefined when disabled/omitted, properly structured array when enabled, and undefined when explicitly set to false.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The PR title accurately summarizes the main change: adding DataZoom support for bar and line charts, which directly matches the changeset content.
Linked Issues check	✅ Passed	All acceptance criteria from issue #134 are met: enableDataZoom prop added to BaseChart, dataZoom works for both bar and line charts, exposed in chart-options-schema, and default-off behavior prevents unintended interactions.
Out of Scope Changes check	✅ Passed	All changes directly support the DataZoom feature implementation. No unrelated modifications detected outside the scope of issue #134.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

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

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch feat/issue-134-datazoom

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
93.3% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[alfredo1996]

Superseded by consolidated PR #185 (release/chart-improvements)