hotfix: re-consent modal polish (button squash, bold Not-now, wrapping label) - #2589
hotfix: re-consent modal polish (button squash, bold Not-now, wrapping label)#25891 commit merged into
Conversation
ActionModal's sm:flex-1 (built for side-by-side CTAs) squashed the stacked h-13 buttons; 'Not now' inherited .btn's font-bold and competed with the primary; the checkbox label wrapped to two lines on mobile.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThe ReConsent modal shortens its consent checkbox text and updates CTA classes and secondary-button styling to maintain stacked responsive presentation and visually de-emphasize “Not now”. ChangesReConsent modal
Estimated code review effort: 1 (Trivial) | ~5 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Warning There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure. 🔧 ESLint
ESLint install failed: dependency version conflict. Check your lock file or package.json. Comment |
🧪 UI test report — ✅ all greenSuites
📊 Coverage (unit)
⏱ 10 slowest test cases
|
Code-analysis diffPainscore total: 6350.19 → 6350.35 (+0.16) 🆕 New findings (2)
✅ Resolved (2)
|
Three visual fixes in ReConsentModal only:
sm:flex-noneon both CTAs (ActionModal'ssm:flex-1is meant for its side-by-side layout and squashed the stacked h-13 buttons),font-normal text-grey-1on Not-now (every.btnis bold by default), and shorter checkbox copy so it fits one line on mobile.🤖 Generated with Claude Code
Summary by CodeRabbit