Stacked on #31 (Moving from Arc), which is stacked on #29. Merge in the order #29, #31, then this one. Each base moves to main on its own. It's stacked because both this PR and #31 add a footer link.
Problem. Plan M5 needs factual, sourced comparison pages for the Arc-refugee launch.
Change.
/compare lists /compare/arc, /compare/dia, and /compare/helium.Merge condition: Core. Every AI sentence is inside AI wording needs Core sign-off (G7) markers. Like #28, they describe the launch behavior (the taint rule), so merge only after Core's guard lands and Core has signed off on the wording.
How it was checked (hill-climb round 1).
build/scratch/site-checks.mjs scores 38/38 on main + #28 + #29 + #31 + this PR. It checks every internal link, every external link live, the banned claims, collapsed spaces, 390 px overflow, the download-page detection states, and no remote scripts or cookies.Net-diff hunks: 0 (no patches touched).
🤖 Generated with Claude Code (Claude Opus 5.5, with Sonnet 5.5 and Devin SWE-2)
Note
Low Risk Static Astro marketing pages and UI components only; no changes to auth, data handling, or application runtime.
Overview
Adds a Compare section to the marketing site: a footer link to /compare, an index listing three comparisons, and dedicated pages for Arc, Dia, and Helium.
Each comparison page follows a shared pattern—intro, a rac vs. other table, sections on where rac differs and where the other browser does more, guidance on who should pick which, and a dated Sources list. Claims about competitors use numbered Cite links into that list; table rows can attach citations on the “other” column via new CompareTable, Cite, and Sources components.
CompareTable is a responsive comparison table that keeps ARIA table roles on small screens by stacking each row into labeled cards (under 640px). The three pages cross-link (e.g. Arc → Dia/Helium) and reuse the site’s existing article layout (Base, sticky header, prose-rac). Several AI-related sentences are marked for Core sign-off before merge.
Reviewed by Cursor Bugbot for commit 2276e2293e7534582729ce3697970415d0fc692f. Bugbot is set up for automated code reviews on this repo. Configure here.
Adds three comparison pages for Arc, Dia, and Helium, linked from the footer under /compare, to support the Arc-refugee launch.
AI wording needs Core sign-off (G7) markers written as hidden comments, so it renders nothing until Core signs off. Table headings stay readable to screen readers.Written for commit 2276e2293e7534582729ce3697970415d0fc692f. Summary will update on new commits.
Your OpenCode Zen account is out of credit. Top it up, then re-trigger Pullfrog.
| 1 | provider error: Upstream request failed: Insufficient account funds |
Pullfrog ｜ Rerun failed job ➔ ｜ View workflow run ｜ via Pullfrog ｜ Using kimi-k3
The latest updates on your projects. Learn more about Vercel for GitHub.
| Project | Deployment | Actions | Updated |
|---|---|---|---|
| External image rac | Ready Ready | Preview | Oct 29, 2026 7:53pm UTC |
Important
Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.
e48d6906-1458-4204-9ef4-fe4714ff8a30You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.
Use the checkbox below for a quick retry:
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 @coderabbitai help to get the list of available commands.
Bugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit.
A user or team admin can review and increase usage limits in the Cursor dashboard.
(requestId: 3d176ec8-56d2-49c3-80b6-5dd1029ef4a8)
Devin Review found 2 potential issues.
| … | … | @@ -0,0 +1,111 @@ | |
| 73 | + | <!-- AI wording needs Core sign-off (G7) --> | |
| 74 | + | <li> | |
| 75 | + | Arc import keeps your spaces as spaces in one window, with their pinned tabs, favorites, and themes. Dia turns | |
| 76 | + | them into profiles.<Cite n={6} /> | |
🟡 Imported profiles falsely promised one window
For an Arc user with multiple profiles, Arc import does not keep all imported spaces in one window. Each profile opens its own window, so those spaces are split across windows.
Arc import preserves each Arc profile as a separate rac profile. What comes over says each imported profile has its own window, and Profiles defines spaces within each profile. The Dia comparison promises one window without that qualification, both here and in its Spaces row. An Arc user with two profiles will see two rac windows rather than one containing every imported space.
Example: An Arc setup has a Work profile and a Personal profile, each with two spaces. Import produces a Work window and a Personal window, not one window containing four spaces.
Recommended fix: Qualify both the Spaces row and this import description as 'spaces within each profile' or 'one window per profile,' consistent with the migration page.
Was this helpful? React with 👍 or 👎 to provide feedback.
| … | … | @@ -0,0 +1,134 @@ | |
| 97 | + | ||
| 98 | + | .compare thead { | |
| 99 | + | display: none; | |
| 100 | + | } | |
🟡 Mobile tables lose accessible browser headings
Below 640px, .compare thead removes both browser headings from the accessibility tree. CSS-generated cell labels do not restore table header associations, so screen-reader table navigation loses browser context.
The comparison is a real table with column headers for rac and the other browser. On narrow screens, display: none removes its <thead> from the accessibility tree. The explicit table and cell roles cannot associate cells with headings that no longer exist. The ::before labels are visual CSS content, not column headers for assistive technology.
Example: At a 390px viewport, a screen-reader user navigating the Dia table by cell encounters a value such as 'Chromium' without the Dia column header.
Recommended fix: Visually hide the header row at mobile widths without display: none or visibility: hidden, preserving its cells in the accessibility tree; test the rendered table with screen-reader table navigation.
Was this helpful? React with 👍 or 👎 to provide feedback.
[Low impact] The changes affect the public site’s comparison pages and footer navigation.
The PR appears safe to merge under the author’s stated Core sign-off condition, with a non-blocking mobile accessibility fix recommended.
| 1 | ### Issue 1 |
| 2 | site/src/components/compare/CompareTable.astro:98-100 |
| 3 | Below 640 px, `display: none` removes the browser column headers from screen readers. The explicit roles preserve the table structure, but users navigating by cell lose the header that identifies each browser. The labels from `td::before` do not restore that connection. |
| 4 | |
| 5 | Visually hide `thead` instead of removing it so the mobile cards keep their column headers. |
| 6 | |
| 7 | Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time! |
| 8 | |
| 9 | --- |
| 10 | |
| 11 | For each issue above, determine whether it is valid and should be fixed. If so, fix it directly. |
Adds /compare and comparison pages for Arc, Dia, and Helium.
notkainoa explicitly deferred the AI wording until Core’s guard lands and Core signs off. notkainoa also explicitly rejected changing the second-site launch wording and trimming facts already stated elsewhere on the site. Those acknowledged choices are not reported as findings.
Reviews (1) · Last reviewed commit: "feat(site): add comparison pages for Arc..." · Reviewed by Greptile
| … | … | @@ -0,0 +1,134 @@ | |
| 97 | + | ||
| 98 | + | .compare thead { | |
| 99 | + | display: none; | |
| 100 | + | } | |
P2 Mobile tables lose browser headers
Below 640 px, display: none removes the browser column headers from screen readers. The explicit roles preserve the table structure, but users navigating by cell lose the header that identifies each browser. The labels from td::before do not restore that connection.
Visually hide thead instead of removing it so the mobile cards keep their column headers.
| 1 | This is a comment left during a code review. |
| 2 | Path: site/src/components/compare/CompareTable.astro |
| 3 | Line: 98-100 |
| 4 | |
| 5 | Comment: |
| 6 | **Mobile tables lose browser headers** |
| 7 | |
| 8 | Below 640 px, `display: none` removes the browser column headers from screen readers. The explicit roles preserve the table structure, but users navigating by cell lose the header that identifies each browser. The labels from `td::before` do not restore that connection. |
| 9 | |
| 10 | Visually hide `thead` instead of removing it so the mobile cards keep their column headers. |
| 11 | |
| 12 | --- |
| 13 | |
| 14 | For each issue above, determine whether it is valid and should be fixed. If so, fix it directly. |
Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!
Added 2 commits
Deployment failed for project rac with the following error:
| 1 | Resource is limited - try again in 24 hours (more than 100, code: "api-deployments-free-per-day"). |
Learn More: https://vercel.com/knewton?upgradeToPro=build-rate-limit
Bugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit.
A user or team admin can review and increase usage limits in the Cursor dashboard.
(requestId: 25a0ee25-143f-45c7-a8a2-7b7ae5ce7ba6)
Your OpenCode Zen account is out of credit. Top it up, then re-trigger Pullfrog.
| 1 | provider error: Upstream request failed: Insufficient account funds |
Pullfrog ｜ Rerun failed job ➔ ｜ View workflow run ｜ via Pullfrog ｜ Using kimi-k3
