-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
refactor(react-router): Link component self time improvements by removing 'delete' #6456
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
📝 WalkthroughWalkthroughThe Link component in react-router was refactored to simplify prop handling and conditional rendering logic. Key changes include removing innerRef extraction, adjusting the condition for plain anchor element rendering, and modifying how the disabled prop is removed during anchor creation. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes Possibly related PRs
Suggested labels
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
Comment |
|
View your CI Pipeline Execution ↗ for commit 8572e3b
☁️ Nx Cloud last updated this comment at |
On a benchmark w/ 100 concurrent requests for 30s, loading pages w/ 100 links in them, we can observe the
<Link>component with a self time of > 3s. Simply removing thedeletekeyword, we can get this down to < 1s.Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.