fix(website): fix sponsor banner truncation on mobile - #1549
Conversation
Replace truncated "Sustained by sponsors" text with "Sponsors" on mobile to prevent text overflow. Desktop keeps the full string. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
|
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: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
Included review availability: Your plan provides up to 4 included reviews per hour; 2 remain after this review. 📝 WalkthroughWalkthroughThe landing page sponsors banner now displays “Sponsors” on small screens. It keeps the existing desktop message and removes the truncation wrapper. ChangesSponsors banner
Estimated code review effort: 1 (Trivial) | ~2 minutes Merge Risk: ⚪ Minimal · up to This localized website change prevents the sponsor banner from truncating on mobile while preserving the desktop wording; no actionable merge-blocking risk remains beyond normal checks and review. 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
Full details: Description checkExplanation The description accurately summarizes the code changes, but it does not follow the repository template. It omits the required challenge submission checklist, sponsorship notice, and related information. Resolution Update the description to include the required challenge submission checklist and sponsorship notice from the repository template. Also ensure the title starts with Full details: Docstring CoverageExplanation No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)
✨ Finishing Touches 💡 1🛠️ Fix failing CI checks 💡
🧪 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 |
Summary
min-w-0andtruncateclasses that caused the ellipsisSummary by CodeRabbit