Skip to content

feat(website): link comparison guides in footer#5428

Open
NicholasKissel wants to merge 1 commit into
mainfrom
NicholasKissel/compare-guides-footer
Open

feat(website): link comparison guides in footer#5428
NicholasKissel wants to merge 1 commit into
mainfrom
NicholasKissel/compare-guides-footer

Conversation

@NicholasKissel

Copy link
Copy Markdown
Member
  • Add a "Compare" column to the marketing site footer
  • Link the Rivet vs Cloudflare Durable Objects and Rivet vs Temporal comparison guides

Note: the /compare pages stay noindex and out of the sitemap; this only makes them reachable from the footer, it does not change their indexing.

@claude

claude Bot commented Jul 19, 2026

Copy link
Copy Markdown
Contributor

Review

Small, self-contained change: adds a "Compare" footer column linking to the existing /compare/rivet-vs-cloudflare-durable-objects and /compare/rivet-vs-temporal pages.

Verified correct:

  • Both hrefs resolve to real, existing slugs in website/src/data/compare/{cloudflare-durable-objects,temporal}.tsx.
  • website/astro.config.mjs already excludes /compare from the sitemap, and [slug].astro sets robots="noindex, nofollow" — consistent with the PR description's claim that indexing is unaffected.
  • lg:grid-cols-6 is the right column count: 1 brand column + 5 category columns (Product, Developers, Resources, Compare, Legal).
  • Follows the existing column pattern exactly (heading classes, <ul>/<li> structure, link styling), so no new patterns introduced.

Minor (non-blocking) nit — worth a visual check:
The category count went from 4 to 5, but md:grid-cols-4 and min-[440px]:grid-cols-2 weren't adjusted. At those breakpoints the 5 category columns no longer divide evenly, so the last column ("Legal") will land alone on its own row instead of pairing up like before. Purely cosmetic, but worth a quick look at tablet (768–1023px) and small (440–767px) widths to confirm it still reads fine.

Other:

  • No security, performance, or test-coverage concerns — this is static internal marketing content with no logic branches.

@railway-app

railway-app Bot commented Jul 19, 2026

Copy link
Copy Markdown

🚅 Deployed to the rivet-pr-5428 environment in rivet-frontend

Service Status Web Updated (UTC)
website 😴 Sleeping (View Logs) Web Jul 21, 2026 at 10:00 pm
frontend-cloud 😴 Sleeping (View Logs) Web Jul 21, 2026 at 7:30 pm
kitchen-sink 😴 Sleeping (View Logs) Web Jul 21, 2026 at 10:02 am
frontend-inspector 😴 Sleeping (View Logs) Web Jul 21, 2026 at 9:50 am
mcp-hub ✅ Success (View Logs) Web Jul 19, 2026 at 5:51 am
ladle ✅ Success (View Logs) Web Jul 19, 2026 at 5:51 am

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant