Skip to content

Add perfected ship-first redesign mockup gallery - #1654

Closed
BigSimmo wants to merge 22 commits into
mainfrom
cursor/ship-first-redesign-mockups-2398
Closed

Add perfected ship-first redesign mockup gallery#1654
BigSimmo wants to merge 22 commits into
mainfrom
cursor/ship-first-redesign-mockups-2398

Conversation

@BigSimmo

@BigSimmo BigSimmo commented Aug 6, 2026

Copy link
Copy Markdown
Owner

Summary

  • Add interactive perfected mockups for the six already-designed ship-first surfaces at /mockups/ship-first-redesign
  • Covers Answer home safety notice (#165/#166), Tools search (#162), Services search (#163), Favourites hybrid (#164), Documents filter cleanup (#170/#171), and Save to Favourites from answer evidence
  • Desktop + phone frames share one gallery; prior static comps remain linked from the header and README
  • Design-scratch only — production routes and clinical behaviour are unchanged

Perfected picks locked in the gallery

# Surface Pick
1 Answer home safety 02 Safety card — obligation on warning-tinted top row; AI-generated verify line
2 Tools search A — Compact Results Instrument (query-as-H1, dense rows, one composer)
3 Services search B — Progressive Referral Workflow (query-as-H1, tiny step dots, shortlist banner only when selected)
4 Favourites B — Search-Led Workspace (sets as chips; Continue on empty; in-place filter)
5 Documents filter One Filter → one panel (phone bottom sheet / desktop dialog)
6 Save from answers Evidence save sheet: citation + page + snippet → set chips

Screenshots

Answer home safety perfected
Tools search perfected
Services search perfected
Favourites perfected
Documents filter perfected
Save to favourites perfected

Verification

  • npm run ensurehttp://localhost:4461 (project identity confirmed via /api/local-project-id)
  • Mockup route HTTP 200: /mockups/ship-first-redesign
  • Prior static comps HTTP 200 under public/mockups/mode-page-redesign-2026-07/perfected-combined/
  • npx eslint on new mockup paths — green after lucide aria fix
  • Playwright screenshots of all six sections
  • npm run test via verify:pr-localTest Files 515 passed (515) / Tests 5446 passed | 1 skipped (5447)
  • ALLOW_BUILD_WITH_DEV_SERVER=1 npm run build — includes /mockups/ship-first-redesign; client bundle secret surface check passed
  • npm run eval:rag:offlineOffline RAG fixture and production-contract checks passed. / 36 golden cases, 23 suites / 574 passed
  • UI verification not run: full verify:ui — mockup routes are design-scratch and 404 in production; section screenshots used instead

Risk and rollout

  • Risk: Low — mockup routes under /mockups/** (404 in production unless opted in)
  • Rollback: Revert this PR / delete the mockup route and components
  • Provider or production effects: None

Notes

  • Static PNG comps for Tools/Services/Favourites remain in public/mockups/mode-page-redesign-2026-07/
  • Answer safety follows the recommended Safety card direction from /mockups/warning-consolidation
  • Services perfected frames use query-as-H1 and progressive shortlist (banner only when selected), tightening the static PNG that still echoed match-count title language

To show artifacts inline, enable in settings.

Open in Web Open in Cursor 

Interactive desktop+phone frames for the six already-designed surfaces:
answer home safety notice, tools/services/favourites search, documents
filter sheet, and save-to-favourites from answer evidence. Design-scratch
only; production routes unchanged.

Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com>
@coderabbitai

coderabbitai Bot commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Draft detected.

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.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 9ec30428-eb0a-42ee-aaa5-04104883e421

You 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:

  • 🔍 Trigger review

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

@supabase

supabase Bot 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 ↗︎.

cursoragent and others added 20 commits August 6, 2026 16:51
Satisfy require-lucide-icon-aria on the decorative Bookmark in the
ship-first services perfected mockup.

Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com>
Register /mockups/ship-first-redesign discovered from the app directory.

Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com>
Empty commit — prior CI run for this tip disappeared during the
GitHub Actions major outage (jobs timing out while queued).

Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com>
Prior CI run cancelled/timed out while queued during the GitHub
Actions major outage. Empty commit to re-fire checks on current tip.

Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com>
Rename src/components/ship-first-redesign to *-mockups so the design-
system contract treats it as design scratch (same as other mockup trees).

Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com>
Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com>
@BigSimmo

BigSimmo commented Aug 7, 2026

Copy link
Copy Markdown
Owner Author

Autopilot + Bugbot pass (2026-08-07)

PR #1654 @ 9d9eb0be47073a7f051a5885359b82f2ff978a85 (draft)

Snapshot

  • Mergeable: MERGEABLE · merge-tree clean vs origin/main
  • Behind/ahead: 1 / 21 — left unsynced (behind-but-clean; required CI already green; anti-churn)
  • Unresolved threads: 0 (no Bugbot/CodeRabbit inline findings; CodeRabbit skipped draft)
  • Required CI: PR required SUCCESS · Production UI (1)/(2)/(3) pass · Build/Static/Safety/Coverage pass

Actions

  • Confirmed GitHub head in isolated worktree Database-wt-pr-1654
  • No conflict unblock; no late sync
  • Bugbot subagent blocked (usage limit) → manual high-confidence review of 13-file mockup gallery delta
  • No P0/P1 (design-scratch only; buttons wired; shell suppressed for /mockups/ship-first-redesign; tokens not hardcoded hex; production routes unchanged)
  • No code push — local-only no-change ledger row recorded; tip unchanged

Blockers

  • Still draft (mergeStateStatus BEHIND while 1 commit behind main — optional sync before mark-ready)
  • Bugbot automated pass not available this session

Never merged / no force-push / no provider gates.

@cursor

cursor Bot commented Aug 8, 2026

Copy link
Copy Markdown
Contributor

Closing as low-yield design-scratch: ship-first redesign mockup gallery only; no production route changes. Production adoption remains tracked as /issues #162#165. Branch left in place for recovery — not deleted.

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.

2 participants