fix(web): clarify install helper copy on agent pages - #62
Conversation
Replace the sticky CTA helper and add the same ownership line under the mobile install bar so both surfaces explain what the command does. Co-authored-by: Tommaso <tommaso.carnemolla@gmail.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: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
💤 Files with no reviewable changes (1)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. WalkthroughThe sticky install CTA removes the helper text about file location and ownership. ChangesInstall CTA
Estimated code review effort: 1 (Trivial) | ~2 minutes Merge Risk: ⚪ Minimal · up to This change only removes helper text from the sticky install control and does not alter the install command or mobile behavior; no actionable merge-blocking risk remains beyond normal checks and review. Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 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 |
Co-authored-by: Tommaso <tommaso.carnemolla@gmail.com>
Use the PMM-approved helper on StickyInstallCta and revert MobileInstallBar so that surface stays command + Install with no helper line. Co-authored-by: Tommaso <tommaso.carnemolla@gmail.com>
Drop the helper line under the sticky Copy command control per product direction; keep command and button unchanged. Co-authored-by: Tommaso <tommaso.carnemolla@gmail.com>
Summary
apps/web/components/sticky-install-cta.tsxMobileInstallBar, install command, Copy command button, PostHog, homepage hero, file-list heading, and vs-agentcn are untouchedTest plan
sm+): sticky CTA shows Install label + command + Copy command only (no helper<p>under it)npx shadcn@latest add @evex/<slug>mainis limited to removing that helper line fromsticky-install-cta.tsxSummary by CodeRabbit