Skip to content

Adopt restyled gestures in Command Universe - #5320

Merged
raineorshine merged 7 commits into
mainfrom
fbmcipher/issue-4186-command-universe
Sep 11, 2026
Merged

Adopt restyled gestures in Command Universe#5320
raineorshine merged 7 commits into
mainfrom
fbmcipher/issue-4186-command-universe

Conversation

@fbmcipher

@fbmcipher fbmcipher commented Sep 3, 2026

Copy link
Copy Markdown
Collaborator

Fixes #4186

Apply the new gesture appearance to the mobile Command Universe using the API from #5315#5319. Based on #5319; this is the final PR in the stack.

CommandUniverseGridItem configures GestureDiagram through props. It adds no geometry or SVG rendering code. Other callers retain their current renderer, including solid binary highlighting in the Gesture Menu.

CleanShot 2026-09-09 at 06 16 13@2x

Appearance changes

  • Replace the fixed 80px/130px diagram with a centered square container with width: 100%, capped at 130px.
  • Use a continuous gradient from #58b5d4 at 45% alpha to white. The new gestureDiagramGradientStart and gestureDiagramGradientEnd tokens are identical in both palettes because the dialog uses fixed dark background art.
  • Set size={150}, strokeWidth={12}, cornerRadius={12}, and arrowhead='outlined-wide'. Disable the glow with glow={false}. Geometry sizing and framing use the shared behavior from Fix inconsistent gesture sizing and stroke thickness #5319. Ordinary block styling lets the SVG fit the parent.
  • Forward command.rounded, so gestures such as newThoughtAbove and toggleSort use circular geometry in the grid, where that flag was previously omitted.
  • Set arrowSize={1}; framing reserves room for the geometry-based wide chevron, while rdld remains arrowhead-free.

Validation and visual review

Four examples of the CU presentation are added to TestGestureDiagram: softened straight geometry, a circular gesture, rdld, and a diagram filling its container. This PR updates the gesture-diagram snapshot for those examples. Earlier fixture definitions and other snapshot files are unchanged.

The stack includes direct geometry and static-rendering tests, plus the existing Gesture Menu and Puppeteer coverage. Review the mobile Command Universe at narrow and wide widths: compare gesture size, gradients, corner joins, circular gestures, chevrons, and clipping. Check the Gesture Menu's partial highlighting separately to confirm its binary treatment remains intact.

Circular gestures inherit #5318's tangent connection to the chevron, keeping the shaft connected and the gradient continuous through the tip. Cancel-gesture styling and conventional marker shapes are outside this PR.

Preview Deployment · Sep 11, 2026, 6:10 PM UTC · 21f94ee

Preview deployment

@fbmcipher
fbmcipher force-pushed the fbmcipher/issue-4186-command-universe branch from 6ef52bd to f76b1bb Compare September 9, 2026 02:40
@fbmcipher
fbmcipher force-pushed the fbmcipher/issue-4186-command-universe branch from f76b1bb to 898e355 Compare September 9, 2026 03:37
@fbmcipher
fbmcipher force-pushed the fbmcipher/issue-4186-command-universe branch from 898e355 to 1dde3cf Compare September 9, 2026 04:59
@fbmcipher
fbmcipher force-pushed the fbmcipher/issue-4186-command-universe branch from 1dde3cf to 4320d67 Compare September 9, 2026 05:28
@fbmcipher
fbmcipher force-pushed the fbmcipher/issue-4186-command-universe branch from 4320d67 to 4a97fb5 Compare September 9, 2026 05:42
@fbmcipher
fbmcipher marked this pull request as ready for review September 9, 2026 05:55
@raineorshine
raineorshine force-pushed the fbmcipher/issue-4186-command-universe branch from 4a97fb5 to abf92d6 Compare September 9, 2026 18:41
@raineorshine
raineorshine force-pushed the fbmcipher/issue-4186-command-universe branch from abf92d6 to f384146 Compare September 9, 2026 19:02
@raineorshine
raineorshine force-pushed the fbmcipher/issue-4186-command-universe branch from f384146 to 7ec99a6 Compare September 9, 2026 19:23
fbmcipher and others added 5 commits September 11, 2026 11:02
Add the Command Universe gradient tokens, select the established GestureDiagram presentation and container options, and cover the three geometry families plus responsive framing in the visual fixture.

Co-Authored-By: Codex GPT-5 <noreply@openai.com>
Opt the grid into normalized curved geometry and centered square framing without changing default diagram callers.

Co-Authored-By: Codex GPT-6 <noreply@openai.com>
Select the production sizing policy in all four new examples and update the reviewed snapshot. Legacy examples retain their original dimensions.

Co-Authored-By: Codex GPT-6 <noreply@openai.com>
Remove the uniform sizing option from the production grid and visual examples now that sizing is consistent for every caller.

Co-Authored-By: Codex GPT-6 <noreply@openai.com>
Record the reviewed shared sizing and framing behavior in the four CU presentation examples.

Co-Authored-By: Codex GPT-6 <noreply@openai.com>
@raineorshine
raineorshine force-pushed the fbmcipher/issue-4186-command-universe branch from 3752610 to a4da49a Compare September 11, 2026 18:02
The yarn 4.18.0 upgrade changed the hash of the builtin compat/resolve
patch, so `yarn install --immutable` in CI wanted to rewrite the lockfile
and every job failed at the install step.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@raineorshine
raineorshine merged commit 7d433c9 into main Sep 11, 2026
13 checks passed
@raineorshine
raineorshine deleted the fbmcipher/issue-4186-command-universe branch September 11, 2026 18:19
raineorshine added a commit that referenced this pull request Sep 11, 2026
A pull request stacked on another's branch contains every commit of its
base, so GET /commits/{sha}/pulls lists both. Preview QR refused to pick
one and left the base pull request's description stuck at "Generating new
QR code…" for a commit it had already force-pushed away (#5319, stacked
under #5320). Select the pull request whose current head is the commit
instead; only two pull requests sharing a head commit are still ambiguous.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
raineorshine added a commit that referenced this pull request Sep 11, 2026
…5526)

A pull request stacked on another's branch contains every commit of its
base, so GET /commits/{sha}/pulls lists both. Preview QR refused to pick
one and left the base pull request's description stuck at "Generating new
QR code…" for a commit it had already force-pushed away (#5319, stacked
under #5320). Select the pull request whose current head is the commit
instead; only two pull requests sharing a head commit are still ambiguous.

Co-authored-by: Claude Fable 5.1 <noreply@anthropic.com>
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.

Restyle the gesture diagram in the Command Universe

2 participants