fix(react-router): keep relative link active when changing inherited param#6689
fix(react-router): keep relative link active when changing inherited param#6689
Conversation
|
View your CI Pipeline Execution ↗ for commit 998fb9b
☁️ Nx Cloud last updated this comment at |
📝 WalkthroughWalkthroughuseLinkProps now subscribes to a composite built-location state (path/search/hash and inherited params) and passes that built location into route preloading. Router core's preloadRoute accepts an optional Changes
Sequence Diagram(s)mermaid LinkHook->>RouterCore: buildLocation(next opts) Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 minutes Possibly related PRs
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches
🧪 Generate unit tests (beta)
Comment |
Fix #5655
Summary
packages/react-router/tests/link.test.tsx/post/$postId/step1and/post/$postId/step2behaviortest.failsLinkcomponentsBonus
buildLocationcall duringpreloadRoutesinceLinkcomponents already have a built location on handSummary by CodeRabbit
Bug Fixes
New Features
Tests