Skip to content

Phone Choose mode: dense glance list (no Find/Diagnose/Care) - #1644

Closed
BigSimmo wants to merge 4 commits into
mainfrom
cursor/phone-mode-dense-glance-05c0
Closed

Phone Choose mode: dense glance list (no Find/Diagnose/Care)#1644
BigSimmo wants to merge 4 commits into
mainfrom
cursor/phone-mode-dense-glance-05c0

Conversation

@BigSimmo

@BigSimmoBigSimmo commented Aug 6, 2026•
edited by cursor Bot
Loading

Copy link
Copy Markdown
Owner

Summary

  • Feedback on the merged sectioned YES 01 (Phone Choose mode sheet: review + two large YES mockups #1618): Find / Diagnose / Care organisation and roomy spacing made mode choice harder at a glance.
  • Redesign YES 01 as a dense flat glance list: no clinical lanes, sticky section labels, hints, or count badges.
  • Title-only min-h-12 rows (descriptions stay in aria-label) so the full production catalogue is scannable in one open.
  • Keep Currently · {mode} in the sheet header, interactive selection that lifts into the top-bar pill, Arrow/Home/End roving focus, and contained proof-frame scroll.
  • Organised icon-deck with Find / Diagnose / Care is shown as a rejected contrast, not a shipping YES.

Route: /mockups/phone-mode-sheet-yes (design-scratch; 404 in production).

YES 01 Rest — title-only flat list, no Find/Diagnose/Care
YES 01 three states

Clinical Governance Preflight

  • Not a clinical-risk product change — design-scratch mockups only (/mockups/* 404 in production)
  • No answer generation, retrieval/ranking, ingestion, source rendering, document access, or privacy behaviour changed
  • Rollback: revert this PR; no production surface is affected

Test plan

  • npm run ensure → /mockups/phone-mode-sheet-yes loads
  • Browser proof: Rest frame shows ~12 modes with no Find/Diagnose/Care section chrome
  • npx eslint on changed TSX — clean

RAG impact

RAG impact: no retrieval behaviour change — mockups-only design scratch; no RAG surfaces touched.

To show artifacts inline, enable in settings.

Open in WebOpen in Cursor

@cursoragent
Make YES 01 a dense flat glance list
7ed556b
Drop Find/Diagnose/Care section chrome and roomy rows so nearly the full
catalogue is visible at once. Keep Currently header, keyboard roving,
contained preview scroll, and lifted top-bar selection.
@cursoragent
Tighten YES 01 to title-only glance rows
40b92b6
Drop subtitle chrome so the full mode catalogue scans in one open.
Mark Find/Diagnose/Care icon-deck as rejected, not an alternate YES.
@coderabbitai

coderabbitaiBot commented Aug 6, 2026•
edited
Loading

Copy link
Copy Markdown
Contributor

Warning

Review limit reached

You’ve reached a temporary PR review limit under our Fair Usage Limits Policy.

Your recent review volume is higher than typical usage, so adaptive limits are currently applied.

Next review available in:57 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 @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

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 configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 2256b189-bd26-41a1-969d-040210c58d3f

📥 Commits

Reviewing files that changed from the base of the PR and between 6e04add and 3a0eaf4.

📒 Files selected for processing (3)
  • mockups/README.md
  • src/app/mockups/phone-mode-sheet-yes/page.tsx
  • src/components/phone-mode-sheet-yes-mockups.tsx

Comment @coderabbitai help to get the list of available commands.

@supabase

supabaseBot commented Aug 6, 2026

Copy link
Copy Markdown

This pull request has been ignored for the connected project sjrfecxgysukkwxsowpy because there are no changes detected in supabase directory. You can change this behaviour in Project Integrations Settings ↗︎.

Preview Branches by Supabase.
Learn more about Supabase Branching ↗︎.

@BigSimmo
BigSimmo marked this pull request as ready for review August 6, 2026 13:03
devin-ai-integration[bot]
devin-ai-integrationBot reviewed Aug 6, 2026
View reviewed changes

@devin-ai-integrationdevin-ai-integrationBot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 1 additional finding.

Open in Devin Review

@BigSimmo
BigSimmo marked this pull request as draft August 6, 2026 14:00
@BigSimmo

BigSimmo commented Aug 6, 2026

Copy link
Copy Markdown
OwnerAuthor

Closing: user asked not to create mockups — will apply dense glance list to production Choose mode sheet instead.

@BigSimmoBigSimmo closed this Aug 6, 2026
@BigSimmoBigSimmo mentioned this pull request Aug 6, 2026
Merged
11 tasks
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@devin-ai-integrationdevin-ai-integration[bot]devin-ai-integration[bot] left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@BigSimmo@cursoragent