fix(svelte-form): prevent full array re-renders in array mode#2170
Conversation
|
Caution Review failedPull request was closed or merged during review 📝 WalkthroughWalkthroughThis PR implements selective re-render tracking for Svelte form fields to eliminate unnecessary full array re-renders in array mode. Field state now uses multiple selector-based subscriptions instead of snapshot snapshots, with documentation, examples, and tests demonstrating and verifying the behavior. ChangesArray Mode Optimization
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
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 40c7757
☁️ Nx Cloud last updated this comment at |
🚀 Changeset Version Preview3 package(s) bumped directly, 11 bumped as dependents. 🟨 Minor bumps
🟩 Patch bumps
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #2170 +/- ##
==========================================
- Coverage 90.35% 86.31% -4.04%
==========================================
Files 38 8 -30
Lines 1752 95 -1657
Branches 444 8 -436
==========================================
- Hits 1583 82 -1501
+ Misses 149 12 -137
+ Partials 20 1 -19 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Mirrors #1963 but for the Svelte adapter
Summary by CodeRabbit
Bug Fixes
Documentation
Tests