Skip to content

style(desktop): Calm UI 1/2, one sans face and 'workflow' everywhere (hold: founder review) - #1309

Merged
abrichr merged 1 commit into
mainfrom
monorepo/port-desktop-calm-foundation
Oct 19, 2026
Merged

abrichr merged 1 commit into
mainfrom
monorepo/port-desktop-calm-foundation

Conversation

@abrichr

@abrichr abrichr commented Oct 18, 2026 •

Copy link
Copy Markdown
Member

For the founder's visual review. Don't merge. This is 1 of 2 stacked PRs; the second is #1310.

This ports OpenAdaptAI/openadapt-desktop#210 into packages/desktop with git am -3 --directory=packages/desktop. Authorship and the commit message are unchanged.

  • One face: --font-heading switches to the sans, matching OpenAdapt Cloud and the phone decision card. It is one revertible constant (--font-heading plus --fw-heading) in src/styles/tokens.css. Page titles are 28px, and cards drop their shadow. The vendored canonical tokens don't change.
  • One noun: user-facing copy says "workflow" instead of "task". Only string literals and JSX text change; identifiers, routes, ids and test ids don't. The primary button reads "Record a workflow", and the nav reads Workflows · Needs you · Results.
  • src/lib/approvalMode.ts and src/screens/NeedsYou.tsx are left alone because the Fix atomacos in requirements.txt #206 port changes them; their wording follows after that port lands.

Release: the commit is a style: commit, so the desktop release-intent check doesn't require a bump. If the founder wants these looks to ship, add a bump with scripts/prepare_release.py at merge time.

Tests: in the original repo, vitest passed (344/344) and so did npm run build. Here, the path-filtered desktop CI runs them; I didn't run them locally to save disk. The name guard and gitleaks found nothing.

🤖 Generated with Claude Code

- --font-heading flips to the sans (one revertible constant, with --fw-heading), matching OpenAdapt Cloud and the phone decision portal. Page titles 28px; cards drop their shadow.
- User-facing copy says 'workflow' instead of 'task' (string literals and JSX text only; identifiers, routes, ids and test ids unchanged). The primary button reads 'Record a workflow'.
- Tests updated to the new wording.
@abrichr

abrichr commented Oct 19, 2026

Copy link
Copy Markdown
Member Author

Before / after: Calm UI 1/2 (desktop)

Before is c933773 (this PR's base on main). After is 213f3f7 (this PR's head).

What changed

  • Headings switch from the serif to the sans on every screen, so the app has one typeface.
  • "Task" becomes "workflow" in visible text. The nav reads Workflows · Needs you · Results, and the Home button reads "Record a workflow" instead of "Show OpenAdapt a new task".
  • The welcome headline reads "Hand off the last step of a workflow", and its cards say "workflow".
  • Page titles are 28 px and cards lose their shadow.
  • Needs you changes only through the shared nav and heading font; its own wording waits for the feat(approval): show who decided each save openadapt-desktop#206 port, as the PR says.

Left is before, right is after. Click an image to open it at full size. Shot locally with npm run dev and the synthetic fixtures in packages/desktop/scripts/cockpit-fixtures.mjs, with the same stand-in Tauri host as scripts/capture-cockpit-previews.mjs. No engine, account, recording or real data. The desktop app has one theme (light, data-theme="paper"), so there is no dark set.

Home (Workflows)

Home (Workflows), 1280 px, before and after

Welcome

Welcome, 1280 px, before and after

Check it

Check it, 1280 px, before and after

Watch it run, stopped run

Watch it run, stopped run, 1280 px, before and after

More screens at 1280 px (Watch it run, checked run, Qualification, steps view, Results, Sign in, First run, Run it, Sample run, Settings, Record (Advanced), Needs you)

Watch it run, checked run

Watch it run, checked run, 1280 px

Qualification, steps view

Qualification, steps view, 1280 px

Results

Results, 1280 px

Sign in

Sign in, 1280 px

First run

First run, 1280 px

Run it

Run it, 1280 px

Sample run

Sample run, 1280 px

Settings

Settings, 1280 px

Record (Advanced)

Record (Advanced), 1280 px

Needs you

Needs you, 1280 px

Phone, 375 px

Home (Workflows)

Home (Workflows), 375 px

Welcome

Welcome, 375 px

Check it

Check it, 375 px

Watch it run, stopped run

Watch it run, stopped run, 375 px

Watch it run, checked run

Watch it run, checked run, 375 px

Qualification, steps view

Qualification, steps view, 375 px

Results

Results, 375 px

Sign in

Sign in, 375 px

First run

First run, 375 px

Run it

Run it, 375 px

Sample run

Sample run, 375 px

Settings

Settings, 375 px

Record (Advanced)

Record (Advanced), 375 px

Needs you

Needs you, 375 px

Images live on a separate branch, pr-screenshots/1309, not on this PR's branch. Nothing here changes the PR.

🤖 Generated with Claude Code

@abrichr
abrichr merged commit d061270 into main Oct 19, 2026
41 checks passed
@abrichr
abrichr deleted the monorepo/port-desktop-calm-foundation branch October 19, 2026 13:15
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