fix: Update BridgeForm link to open Terms of Use in a new tab#577
Conversation
- Modified the Terms of Use link in BridgeForm to include target="_blank" and referrerPolicy="no-referrer" for improved user experience and security.
|
Bugbot is not enabled for this team, so this pull request was not reviewed. Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
✅ Files skipped from review due to trivial changes (1)
📝 WalkthroughWalkthroughThe Terms of Use link in ChangesTerms of Use Link Update
Estimated code review effort: 1 (Trivial) | ~2 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@app/components/bridge/BridgeForm.tsx`:
- Line 475: The Link in BridgeForm still opens a new tab without the same opener
protection used elsewhere in the file, leaving a reverse-tabnabbing gap. Update
the Terms of Use Link in BridgeForm to follow the same target="_blank" pattern
as the other external links in this component by adding the appropriate rel
attribute alongside the existing referrerPolicy, keeping the JSX consistent and
safe.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 65479ca4-7446-4e0d-96a2-eb4403338067
📒 Files selected for processing (1)
app/components/bridge/BridgeForm.tsx
- Changed the Terms of Use link in BridgeForm to use rel="noopener noreferrer" for enhanced security when opening in a new tab.
|
Bugbot is not enabled for this team, so this pull request was not reviewed. Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs. |
Description
References
Testing
Checklist
mainBy submitting a PR, I agree to Paycrest's Contributor Code of Conduct and Contribution Guide.
Summary by CodeRabbit
noopener noreferrerbehavior to help prevent referrer sharing.