Reorder homepage elements on mobile for better CTA visibility#649
Reorder homepage elements on mobile for better CTA visibility#649
Conversation
|
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Important Review skippedBot user detected. To trigger a single review, invoke the You can disable this status message by setting the WalkthroughThe PR addresses mobile visibility for action buttons by reordering page layout elements at different screen breakpoints using CSS order properties, and adds className prop support to the CompatibilityRails component to enable flexible styling. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Poem
Pre-merge checks and finishing touches❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
Comment |
Co-authored-by: yamcodes <2014360+yamcodes@users.noreply.github.com>
Co-authored-by: yamcodes <2014360+yamcodes@users.noreply.github.com>
|
@coderabbitai review |
✅ Actions performedReview triggered.
|
|
@coderabbitai review |
✅ Actions performedReview triggered.
|
Action buttons (Quickstart, Star us) were appearing below compatibility rails on mobile, reducing their visibility and engagement potential.
Changes
apps/www/app/(home)/page.tsx: Applied flexboxorderutilities to swap visual order on mobile (order-1 md:order-2for buttons,order-2 md:order-1for rails)apps/www/components/page/compatibility-rails.tsx: AddedclassNameprop support usingcn()utility for class mergingBehavior
Screenshots
Mobile (375px) - Buttons now prominent
Desktop (1280px) - Original layout preserved
Warning
Firewall rules blocked me from connecting to one or more addresses (expand for details)
I tried to connect to the following addresses, but was blocked by firewall rules:
fonts.googleapis.com/usr/local/bin/node /usr/local/bin/node /home/REDACTED/work/arkenv/arkenv/node_modules/.pnpm/next@16.1.1_@babel+core@7.28.5_@opentelemetry+api@1.9.0_@playwright+test@1.57.0_react-d_f42438f6717e44ec3de5d0c1806f5fb0/node_modules/next/dist/server/lib/start-server.js(dns block)https://api.github.com/repos/yamcodes/arkenv/usr/local/bin/node /usr/local/bin/node /home/REDACTED/work/arkenv/arkenv/node_modules/.pnpm/next@16.1.1_@babel+core@7.28.5_@opentelemetry+api@1.9.0_@playwright+test@1.57.0_react-d_f42438f6717e44ec3de5d0c1806f5fb0/node_modules/next/dist/server/lib/start-server.js(http block)If you need me to access, download, or install something from one of these locations, you can either:
Original prompt
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.
Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.