[miguel-heygen]

A menu row's hover and keyboard highlight now has a larger corner radius than the panel around it, so the highlight reads as sitting inside the menu instead of cutting its corners.

The row radius was rounded-sm (4px) inside a rounded-lg panel. It is now the panel radius plus 6px (calc(var(--radius-lg) + 6px)): 14px in the default theme, 18px where a host sets --radius-lg to 12px. It is set once on the shared row class in Menu.tsx, so every MenuItem (and the radio items built on it) changes together. Padding, height and gap are untouched, so no row moves.

What I measured

• Row and panel rects in a browser, before vs after, 3 menus in both themes: identical x, y, width, height for the panel and every row. Computed row radius 4px -> 14px, panel 8px both.
• Menu.test.tsx, menuStyle.test.ts and src/styles: 8 files, 68 tests pass on miga.
• The new test passes with this change and fails with the old rounded-sm class: run both ways on miga.

fails without the fix: packages/studio/src/components/ui/Menu.test.tsx > "open motion" > "rounds a row's highlight more than the panel that holds it" (itemBase put back to rounded-sm: 1 failed)

What I did NOT exercise

• Desktop's own menus (brand kit, preview, project): the captures below render the real Studio Menu in a harness, three shapes (plain rows, icon and shortcut rows, a separator with a danger row), not Desktop's menus.
• Not run in Desktop. Desktop picks the change up with its next pin of the Studio package.

Size

A lone one-class fix at the single owner of the row shape; nothing else carries it.

Before

After

[heygengenesis]

Requesting changes at 6aa7a03c. There is one blocker, and it is in the PR body, not the code.

Blocker: this is a fix PR (fix(studio): …), but the body has no fails without the fix: line naming the test. The prose under "What I measured" describes the test, but the review gate needs that exact line. Please add, for example:

fails without the fix: packages/studio/src/components/ui/Menu.test.tsx › "open motion" › "rounds a row's highlight more than the panel that holds it"

What I verified, so a re-review can be quick once the line is added:

• Menu.test.tsx passes at this head: 16/16.
• With itemBase put back to rounded-sm, exactly that test fails: 1 failed, 15 passed. So the test does guard the change.
• The diff is one class on the shared itemBase, so every MenuItem and the radio items built on it change together. Padding, gap and height classes are unchanged.

Non-blocking notes:

• calc(var(--radius-lg)+6px) is 14px by default. A row is py-1.5 plus a text-step-11 line, about 28px tall, so the highlight is effectively a pill. At --radius-lg: 12px the 18px value is capped at half the row height, so it stays a pill. That reads as the intended design choice, but the radius is no longer concentric with the panel (panel radius minus p-1 would be 4px). Worth confirming the pill look in the before/after captures.
• The body says visual evidence: pending.

[heygengenesis]

Approving at 6aa7a03c.

The head is the same one I reviewed, and the body now has the fails without the fix: line. It names Menu.test.tsx › "open motion" › "rounds a row's highlight more than the panel that holds it". I had already checked that test both ways at this head:

• It passes with this change: 16/16.
• It is the only failure with itemBase put back to rounded-sm: 1 failed, 15 passed.

That line was the only blocker. My earlier non-blocking notes on the pill radius and the pending visual evidence still stand.

[github-actions]

Edit accuracy: accurate 267 (base branch 267), smooth 100 of those

The gate passes.
Smoothness is reported in the artifact, not gated. A case fails only if it fails 2 of 3 runs.

Quarantined, measured but not gated (8)

• seqplainkeys-none-center-r0-nested-z100 (fixed by studio: undo/redo on a keyframed plain element sometimes leaves the box 230-400px off (edit-accuracy seqplainkeys-none-*) #6989): pass, passes
• seqplainkeys-none-center-r0-root-z100 (fixed by studio: undo/redo on a keyframed plain element sometimes leaves the box 230-400px off (edit-accuracy seqplainkeys-none-*) #6989): pass, passes
• seqplainkeys-none-center-r30-root-z100 (fixed by studio: undo/redo on a keyframed plain element sometimes leaves the box 230-400px off (edit-accuracy seqplainkeys-none-*) #6989): not run
• seqplainkeys-none-pct-r0-root-z100 (fixed by studio: undo/redo on a keyframed plain element sometimes leaves the box 230-400px off (edit-accuracy seqplainkeys-none-*) #6989): pass, passes
• seqplainkeys-none-pct-r30-root-z100 (fixed by studio: undo/redo on a keyframed plain element sometimes leaves the box 230-400px off (edit-accuracy seqplainkeys-none-*) #6989): not run
• seqplainkeys-none-px-r0-nested-z100 (fixed by studio: undo/redo on a keyframed plain element sometimes leaves the box 230-400px off (edit-accuracy seqplainkeys-none-*) #6989): pass, passes
• seqplainkeys-none-px-r30-nested-z100 (fixed by studio: undo/redo on a keyframed plain element sometimes leaves the box 230-400px off (edit-accuracy seqplainkeys-none-*) #6989): not run
• seqplainkeys-none-px-r30-root-z100 (fixed by studio: undo/redo on a keyframed plain element sometimes leaves the box 230-400px off (edit-accuracy seqplainkeys-none-*) #6989): not run

Unstable (2)

• seqplainkeys-none-pct-r0-nested-z100: tracking 0.05, pressJump 0, drop 0, reload 0, render 0.02, renderKey -, undo true, teleport true / tracking 0.05, pressJump 0, drop 0, reload 0, render 0.02, renderKey -, undo true, teleport true / tracking 0.05, pressJump 0, drop 0, reload 0, render 0.02, renderKey -, undo true, teleport true
• seqab-none-pct-r0-root-z100: tracking 0.05, pressJump 0, drop 0, reload 0, render 0.03, renderKey -, undo true, teleport true / tracking 0.05, pressJump 0, drop 0, reload 0, render 0.03, renderKey -, undo true, teleport true / tracking 0.05, pressJump 0, drop 0, reload 0, render 0.03, renderKey -, undo true, teleport true