@estiva-app/ui 0.57.3 → 0.57.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@estiva-app/ui",
-  "version": "0.57.3",
+  "version": "0.57.4",
   "description": "Estiva's design tokens (the contract) and a small set of primitives (a convenience) for every Estiva app.",
   "type": "module",
   "license": "MIT",

package/registry.json

@@ -4,7 +4,7 @@
     "kind": "package",
     "repo": "estiva-ui",
     "package": "@estiva-app/ui",
-    "packageVersion": "0.57.3",
+    "packageVersion": "0.57.4",
     "exports": 130,
     "typeExports": 134,
     "files": null
@@ -4428,7 +4428,7 @@
         },
         {
           "name": "finalFocus",
-          "takes": "RefObject<HTMLElement | null>",
+          "takes": "RefObject<HTMLElement | null> | false",
           "required": false,
           "note": "Where focus goes when the panel closes. With a `trigger` it goes back to the trigger and this is not needed."
         },

package/src/ComposerTriggers.test.tsx

@@ -367,6 +367,9 @@ describe('leaving the box', () => {
     expect(isSuggestionOpen()).toBe(true)
     await act(async () => other().focus())
     expect(screen.queryByRole('listbox')).toBeNull()
+    // The closing list does not pull focus back into the box that was left.
+    await act(async () => new Promise((resolve) => setTimeout(resolve, 50)))
+    expect(document.activeElement).toBe(other())
     // So the other box's own Escape and Enter are its own — no grace either.
     expect(isSuggestionOpen()).toBe(false)
     expect(isSuggestionActive()).toBe(false)

package/src/Popover.mdx

@@ -99,7 +99,9 @@ so it is the one case the caller drives:
 - **It does not take focus**, on purpose: the person is still in the text, and
   a toolbar that moved the caret out of it would end the edit it exists to
   serve. `finalFocus` is therefore only for a panel something inside it *did*
-  take focus into — a field the person clicked.
+  take focus into — a field the person clicked. `finalFocus={false}` moves
+  nothing on close: a panel that closes because focus left for somewhere else
+  (a type-ahead list whose box was clicked out of) must not pull it back.
 - **Render it always and toggle `open`.** Not `{rect && <Popover…>}`: mounting
   the panel only while it is open takes Base UI's state machine with it, and
   its exits go too — measured, Escape and re-opening both became intermittent.

package/src/Popover.tsx

@@ -82,9 +82,10 @@ export interface PopoverProps {
    * the trigger and this is not needed. An anchored panel never took focus, so
    * this only matters when something inside it did — a field the person tabbed
    * or clicked into: point it at what they came from, or focus is left on the
-   * document body.
+   * document body. `false` moves nothing: a panel that closes because focus
+   * left for somewhere else must not pull it back (`SuggestionMenu`).
    */
-  finalFocus?: RefObject<HTMLElement | null>
+  finalFocus?: RefObject<HTMLElement | null> | false
   /** Base UI's imperative handle. `actions.current?.close()` shuts the panel —
    *  for the Cancel and Save buttons a form panel ends with. */
   actionsRef?: RefObject<{ close: () => void; unmount: () => void } | null>

package/src/SuggestionMenu.tsx

@@ -186,7 +186,8 @@ export function SuggestionMenu<T>({ rect, editorElement, ariaLabel, width, maxHe
   }
 
   return (
-    <Popover anchor={rect} open={rect !== null} side="top" align="left" className={width} maxHeight={maxHeight} ariaLabel={ariaLabel}>
+    // finalFocus={false}: focus never left the text, and a list that closes because focus moved elsewhere must not pull it back (71a4577b).
+    <Popover anchor={rect} open={rect !== null} side="top" align="left" className={width} maxHeight={maxHeight} ariaLabel={ariaLabel} finalFocus={false}>
       {/* The list the editor drives from its caret: a listbox, because Base UI's lists need a text field of their own.
           `gap-0.5`: 2px between rows, as a menu's — a section with a heading
           had them from `MenuSection`, and rows without one touched. */}