Add deterministic live chart motion and viewports - #58
Conversation
|
Important Review skippedToo many files! This PR contains 105 files, which is 5 over the limit of 100. To get a review, reduce the PR to 100 files or fewer by splitting it into smaller PRs or changing its base branch. Upgrade to a paid plan to raise the limit. This review couldn't start because sufficient usage credits or metered capacity aren't available. Add credits or update usage-based reviews in the billing tab, then retry. ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (105)
You can disable this status message by setting the 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. Comment |
|
View your CI Pipeline Execution ↗ for commit 559b751
☁️ Nx Cloud last updated this comment at |
What changed
Public contracts
ChartInteractionControllerexposesclientToScene,resolvePointer, andsetControlledFocus. Marks can declare viewport ownership per axis, while guides remain fixed by default.Verification
pnpm validatepnpm benchmark:checkpnpm --filter @charts-poc/react-example build:livepnpm --filter @charts-poc/react-example build:pagedThe full CI graph passes, including DOM/framework suites, SSR suites, typecheck, docs, bundle policy, packed exports, React Native bare/Expo Metro boundaries, catalog checks, and seven framework adapter gates. Universal and comparison bundle baselines record the reviewed default-contract cost.