docs: mode-page redesign comps and implementation queue (#162–#164) - #1521
Conversation
Warning Review limit reachedYou’ve reached a temporary PR review limit under our Fair Usage Limits Policy. Next review available in:35 minutes Your organization has reached its usage spending cap. Adjust your spending cap in the billing tab. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (5)
📝 WalkthroughWalkthroughThe PR adds documentation for July 2026 mode-page redesign comps. It records directions for Tools, Services, and Favourites search pages, adds tracking issues, updates the issue queue, and logs the comps handoff review. ChangesMode-page redesign documentation
Estimated code review effort: 1 (Trivial) | ~3 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Comment |
Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com>
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
BigSimmo
commented
Jul 31, 2026
@cursoragent resolve merge conflict |
Merge conflict on PR #1521 is resolved and pushed. Cause: Resolution:
PR: #1521 |
Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com>
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@docs/branch-review-ledger.md`:
- Line 502: Update the ledger row for mode-page-redesign-comps-handoff to append
the decisive output from check:outstanding-issues, using the actual gate result
such as next-id=163; unique IDs. Preserve the existing review metadata and
record the verification output explicitly in the row.
In `@public/mockups/mode-page-redesign-2026-07/README.md`:
- Around line 37-39: Update the “Current baselines” documentation to instruct
contributors to run npm run ensure before capturing screenshots, use the local
URL printed by that command instead of hardcoding http://localhost:4461, and
document the capture command for reproducible verification.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: 19987354-2c8a-4ae9-b6a2-7d134d418ebf
⛔ Files ignored due to path filters (26)
public/mockups/mode-page-redesign-2026-07/current/current-favourites-home-desktop.pngis excluded by!**/*.pngpublic/mockups/mode-page-redesign-2026-07/current/current-favourites-home-phone.pngis excluded by!**/*.pngpublic/mockups/mode-page-redesign-2026-07/current/current-favourites-search-desktop.pngis excluded by!**/*.pngpublic/mockups/mode-page-redesign-2026-07/current/current-favourites-search-phone.pngis excluded by!**/*.pngpublic/mockups/mode-page-redesign-2026-07/current/current-services-search-desktop.pngis excluded by!**/*.pngpublic/mockups/mode-page-redesign-2026-07/current/current-services-search-phone.pngis excluded by!**/*.pngpublic/mockups/mode-page-redesign-2026-07/current/current-tools-search-desktop.pngis excluded by!**/*.pngpublic/mockups/mode-page-redesign-2026-07/current/current-tools-search-phone.pngis excluded by!**/*.pngpublic/mockups/mode-page-redesign-2026-07/favourites-hybrid/favourites-hybrid-a-unified-library-desk-desktop.pngis excluded by!**/*.pngpublic/mockups/mode-page-redesign-2026-07/favourites-hybrid/favourites-hybrid-a-unified-library-desk-phone.pngis excluded by!**/*.pngpublic/mockups/mode-page-redesign-2026-07/favourites-hybrid/favourites-hybrid-b-search-led-workspace-desktop.pngis excluded by!**/*.pngpublic/mockups/mode-page-redesign-2026-07/favourites-hybrid/favourites-hybrid-b-search-led-workspace-phone.pngis excluded by!**/*.pngpublic/mockups/mode-page-redesign-2026-07/favourites-hybrid/favourites-hybrid-c-operational-command-desktop.pngis excluded by!**/*.pngpublic/mockups/mode-page-redesign-2026-07/favourites-hybrid/favourites-hybrid-c-operational-command-phone.pngis excluded by!**/*.pngpublic/mockups/mode-page-redesign-2026-07/services-search/services-search-redesign-a-query-h1-results-phone.pngis excluded by!**/*.pngpublic/mockups/mode-page-redesign-2026-07/services-search/services-search-redesign-a-query-h1-results.pngis excluded by!**/*.pngpublic/mockups/mode-page-redesign-2026-07/services-search/services-search-redesign-b-progressive-workflow-phone.pngis excluded by!**/*.pngpublic/mockups/mode-page-redesign-2026-07/services-search/services-search-redesign-b-progressive-workflow.pngis excluded by!**/*.pngpublic/mockups/mode-page-redesign-2026-07/services-search/services-search-redesign-c-dense-referral-table-phone.pngis excluded by!**/*.pngpublic/mockups/mode-page-redesign-2026-07/services-search/services-search-redesign-c-dense-referral-table.pngis excluded by!**/*.pngpublic/mockups/mode-page-redesign-2026-07/tools-search/tools-search-redesign-a-compact-results-phone.pngis excluded by!**/*.pngpublic/mockups/mode-page-redesign-2026-07/tools-search/tools-search-redesign-a-compact-results.pngis excluded by!**/*.pngpublic/mockups/mode-page-redesign-2026-07/tools-search/tools-search-redesign-b-dense-launcher-list-phone.pngis excluded by!**/*.pngpublic/mockups/mode-page-redesign-2026-07/tools-search/tools-search-redesign-b-dense-launcher-list.pngis excluded by!**/*.pngpublic/mockups/mode-page-redesign-2026-07/tools-search/tools-search-redesign-c-split-command-phone.pngis excluded by!**/*.pngpublic/mockups/mode-page-redesign-2026-07/tools-search/tools-search-redesign-c-split-command.pngis excluded by!**/*.png
📒 Files selected for processing (4)
docs/branch-review-ledger.mddocs/outstanding-issues.mdmockups/README.mdpublic/mockups/mode-page-redesign-2026-07/README.md
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
This pull request has been ignored for the connected project Preview Branches by Supabase. |
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
BigSimmo
commented
Jul 31, 2026
@cursoragent resolve merge conflict |
Merge conflict on PR #1521 is resolved and pushed. Cause: Resolution:
PR: #1521 |
BigSimmo
commented
Jul 31, 2026
@cursoragent This PR was just reopened. Do not merge, force-push, rebase, or arm auto-merge
|
PR #1521 babysit complete — merge decision left to you. Snapshot
Mergeability
CI
Threads
Residual risks
No merge, force-push, rebase, or auto-merge armed. |
Avoid hardcoding localhost:4461; capture after npm run ensure and use the printed project URL for reproducible comparison. Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com>
BigSimmo
commented
Jul 31, 2026
@cursoragent resolve merge conflict |
No conflict left to fix on PR #1521. The failure note is stale — it referred to old tip
CI on this tip is still in progress ( |
BigSimmo
commented
Jul 31, 2026
@cursoragent This PR was just reopened. Do not merge, force-push, rebase, or arm auto-merge
|
PR #1521 babysit complete — merge left to you. Snapshot
Actions this pass
CI on this tip onlyRequired gates so far: Static PR checks, Safety, Build, Unit coverage, Production UI critical, PR policy, PR mergeability, Semgrep, Gitleaks — all SUCCESS. Still running: Production UI shards (1)/(2)/(3); Residual risks
|
Late sync after tip CI went green; merge-tree was clean (behind-by 1). Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com>
CI triageCI failed on this PR. Automated classification of the 2 failed job(s):
Compared with main CI run #7482 (cancelled). Classification is evidence routing, not permission to ignore a failure. Exact quarantined Playwright identities remain governed by the flake ledger. |
BigSimmo
commented
Jul 31, 2026
@cursoragent resolve CI failing |
CI failure on PR #1521 is fixed and green on the current tip. Cause
Exact Fix
Evidence
Merge left to you (auto-merge is already enabled on your account). |
The #146 header-bottom poll can false-settle one frame before data-scroll-hidden and the result anchor finish recovering, leaving the exact 131px chrome-height jump. Require hidden chrome attributes plus the pre-resize anchor before asserting. Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com>
Behind-but-clean sync so pull_request CI can build refs/pull/1521/merge after the phone-scroll settle harden. Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com>


Summary
#162.#163.#164.#162,#163,#164, with comps underpublic/mockups/mode-page-redesign-2026-07/.origin/mainand resolveddocs/outstanding-issues.mdconflicts by keeping main’s rows (#160archived, open#161hover tokens) and renumbering redesign tasks to#162–#164(issues:next-id=165).npm run ensurerather than a hardcoded localhost port.tests/ui-phone-scroll-page-owned.spec.tsso#146does not false-settle onheader.bottomalone.This PR does not change production UI behaviour beyond the phone-scroll test harden — docs + static mockup PNGs + one Playwright settle poll.
Verification
npm run check:outstanding-issues— passed (next-id=165, unique ids)git merge-tree --write-tree origin/main HEAD— CLEANtests/ui-phone-scroll-page-owned.spec.ts— Services viewport continuous journey (running / pending on tip)npm run verify:pr-local— docs/static assets + single phone-scroll test hardenRisk and rollout
Notes
public/mockups/mode-page-redesign-2026-07/README.mdmockups/README.md#162→ Services#163→ Favourites#164Summary by CodeRabbit