@godxjp/ui 32.2.0 → 32.2.1

package/agent/START-HERE.md

@@ -3,7 +3,7 @@
 You are about to write code against a design system you did not author. This file is the whole
 contract. Read it before you write JSX.
 
-**This catalog describes `@godxjp/ui` 32.2.0.** If the project you are editing has a different
+**This catalog describes `@godxjp/ui` 32.2.1.** If the project you are editing has a different
 version in its `package.json`, read the pinned catalog for THAT version instead
 (`…/v<their-version>/agent/…`). A catalog newer than the installed package describes props that do
 not exist yet; older, and it hides props that do. Neither failure announces itself.

package/agent/components/DropdownMenu.json

@@ -38,7 +38,7 @@
       "type": "number"
     },
     {
-      "description": "On the ROOT. A page-level key that OPENS the menu — `\"f\"` for a filter menu, `\"mod+shift+v\"` for a view menu (gh#1264). Same combo syntax and guard as SearchInput/CommandPalette: a bare key is ignored while focus is in an editable element or an IME is composing. Off by default; ignored while `disabled`. Show the key in the trigger (e.g. with `DropdownMenuShortcut`) so it is discoverable.",
+      "description": "On the ROOT. A page-level key that OPENS the menu — `\"f\"` for a filter menu, `\"mod+shift+v\"` for a view menu (gh#1264). Same combo syntax and guard as SearchInput/CommandPalette: a bare key is ignored while focus is in an editable element or an IME is composing. Off by default; ignored while `disabled`. The trigger gets `aria-keyshortcuts` automatically (an explicit consumer value wins); show the key visibly too (e.g. `DropdownMenuShortcut`). Never fires from inside a dialog, menu or listbox that does not contain the menu (gh#1267).",
       "name": "shortcut",
       "type": "string"
     },

package/agent/components/Toolbar.json

@@ -90,6 +90,12 @@
       "name": "views",
       "type": "ReactNode"
     },
+    {
+      "defaultValue": "\"above\"",
+      "description": "Where `views` sits (gh#1267). `above` is its own row over the strip; `inline` puts it at the START of the strip's first row so an idle bar stays ONE row (with `above`, an idle bar measured 55 → 95px and one table row fewer above the fold at 1440×900).",
+      "name": "viewsPlacement",
+      "type": "\"above\" | \"inline\""
+    },
     {
       "description": "Extra classes on the root element.",
       "name": "className",

package/agent/components.json

@@ -8301,7 +8301,7 @@
         "type": "number"
       },
       {
-        "description": "On the ROOT. A page-level key that OPENS the menu — `\"f\"` for a filter menu, `\"mod+shift+v\"` for a view menu (gh#1264). Same combo syntax and guard as SearchInput/CommandPalette: a bare key is ignored while focus is in an editable element or an IME is composing. Off by default; ignored while `disabled`. Show the key in the trigger (e.g. with `DropdownMenuShortcut`) so it is discoverable.",
+        "description": "On the ROOT. A page-level key that OPENS the menu — `\"f\"` for a filter menu, `\"mod+shift+v\"` for a view menu (gh#1264). Same combo syntax and guard as SearchInput/CommandPalette: a bare key is ignored while focus is in an editable element or an IME is composing. Off by default; ignored while `disabled`. The trigger gets `aria-keyshortcuts` automatically (an explicit consumer value wins); show the key visibly too (e.g. `DropdownMenuShortcut`). Never fires from inside a dialog, menu or listbox that does not contain the menu (gh#1267).",
         "name": "shortcut",
         "type": "string"
       },
@@ -8560,6 +8560,12 @@
         "name": "views",
         "type": "ReactNode"
       },
+      {
+        "defaultValue": "\"above\"",
+        "description": "Where `views` sits (gh#1267). `above` is its own row over the strip; `inline` puts it at the START of the strip's first row so an idle bar stays ONE row (with `above`, an idle bar measured 55 → 95px and one table row fewer above the fold at 1440×900).",
+        "name": "viewsPlacement",
+        "type": "\"above\" | \"inline\""
+      },
       {
         "description": "Extra classes on the root element.",
         "name": "className",

package/agent/index.json

@@ -48,7 +48,7 @@
   "note": "Pin to the tag that matches the @godxjp/ui version you installed. A catalog newer than your package describes props you do not have; older, and it hides props you do.",
   "read": {
     "live": "https://raw.githubusercontent.com/godx-jp/godxjp-ui/main/agent/index.json",
-    "pinned": "https://raw.githubusercontent.com/godx-jp/godxjp-ui/v32.2.0/agent/index.json"
+    "pinned": "https://raw.githubusercontent.com/godx-jp/godxjp-ui/v32.2.1/agent/index.json"
   },
   "source": "mcp/src/data — the same data @godxjp/ui-mcp serves — plus the foundation and semantic token tiers, read from src/tokens/*.css",
   "start": "https://raw.githubusercontent.com/godx-jp/godxjp-ui/main/agent/START-HERE.md",
@@ -62,5 +62,5 @@
     "foundation": "the seeds a consumer is invited to set — --primary, --background, --radius",
     "semantic": "named roles that follow the seeds — --ring, --text-link, --overlay-background"
   },
-  "version": "32.2.0"
+  "version": "32.2.1"
 }

package/agent/llms.txt

@@ -1,10 +1,10 @@
 # @godxjp/ui
 
 > A Japanese-enterprise React design system: 161 components, 2193 design tokens,
-> 43 cardinal rules. This file is the entry point for AI agents. Catalog version 32.2.0.
+> 43 cardinal rules. This file is the entry point for AI agents. Catalog version 32.2.1.
 
 If your client can run a process, do not read these files — run the MCP server instead
-(`npx @godxjp/ui-mcp@32.2.0`). It is searchable and version-locked. These files exist for agents
+(`npx @godxjp/ui-mcp@32.2.1`). It is searchable and version-locked. These files exist for agents
 that can only fetch URLs.
 
 ## Start
@@ -26,7 +26,7 @@ that can only fetch URLs.
 ## Pinning
 
 Every URL above tracks `main`. To pin to the release a project actually installed, swap `main` for
-the tag: `.../godx-jp/godxjp-ui/v32.2.0/agent/...`. A catalog that does not match the installed
+the tag: `.../godx-jp/godxjp-ui/v32.2.1/agent/...`. A catalog that does not match the installed
 package describes props that are absent, or hides props that are present, and says nothing either way.
 
 Pinned catalogs only exist for releases whose tag actually contains `agent/`. If `…/v<version>/agent/index.json` returns 404, that release predates this catalog: read `…/main/…` instead and compare `index.json` → `version` against the package you have, so you at least know which way it drifted.

package/dist/components/data-display/data-table.js

@@ -916,7 +916,7 @@ DataTable.Content = function DataTableContent() {
     const el = scrollRef.current;
     if (!el) return;
     for (const cell of el.querySelectorAll("tbody td.ui-data-table-ellipsis")) {
-      if (cell === document.activeElement) continue;
+      if (cell.contains(document.activeElement)) continue;
       const ownTitle = cell.dataset.truncatedTitle !== void 0;
       if (cell.scrollWidth - cell.clientWidth > 1) {
         cell.dataset.truncated = "";

package/dist/components/data-entry/search-input.js

@@ -8,7 +8,12 @@ import { Label } from "../data-entry/label.js";
 import { cn } from "../../lib/utils.js";
 import { resolveFieldA11y } from "../../lib/field-a11y.js";
 import { useDebouncedValue } from "../../lib/hooks.js";
-import { parseShortcut, shortcutHint, shouldHandleShortcut } from "../../lib/shortcut.js";
+import {
+  ariaKeyShortcuts,
+  parseShortcut,
+  shortcutHint,
+  shouldHandleShortcut
+} from "../../lib/shortcut.js";
 function SearchInput({
   value: controlledValue,
   defaultValue = "",
@@ -55,7 +60,7 @@ function SearchInput({
   React.useEffect(() => {
     if (!shortcutSpec || disabled) return;
     const handleKeyDown = (event) => {
-      if (!shouldHandleShortcut(shortcutSpec, event)) return;
+      if (!shouldHandleShortcut(shortcutSpec, event, inputRef.current)) return;
       event.preventDefault();
       inputRef.current?.focus();
     };
@@ -78,6 +83,7 @@ function SearchInput({
           id: inputId,
           type: "text",
           role: "searchbox",
+          "aria-keyshortcuts": shortcutSpec ? ariaKeyShortcuts(shortcutSpec) : void 0,
           value,
           onChange: (e) => {
             setValue(e.target.value);

package/dist/components/navigation/dropdown-menu.js

@@ -20,7 +20,7 @@ import {
   useLocale
 } from "react-aria-components";
 import { Check, ChevronRight } from "lucide-react";
-import { parseShortcut, shouldHandleShortcut } from "../../lib/shortcut.js";
+import { ariaKeyShortcuts, parseShortcut, shouldHandleShortcut } from "../../lib/shortcut.js";
 import { cn } from "../../lib/utils.js";
 import { Slot } from "../../lib/slot.js";
 const DROPDOWN_MENU_PLACEMENT = {
@@ -189,9 +189,10 @@ function DropdownMenu({
       disabled,
       hover,
       racOwnsContextMenu,
-      openedByHover
+      openedByHover,
+      keyShortcuts: shortcutSpec && !disabled ? ariaKeyShortcuts(shortcutSpec) : void 0
     }),
-    [modal, wantsContextMenu, disabled, hover, racOwnsContextMenu, wantsHover]
+    [modal, wantsContextMenu, disabled, hover, racOwnsContextMenu, wantsHover, shortcutSpec]
   );
   return /* @__PURE__ */ jsx(DropdownMenuModalContext.Provider, { value: options, children: /* @__PURE__ */ jsx(
     MenuTrigger,
@@ -208,7 +209,7 @@ function DropdownMenuTrigger({ asChild, children, ...props }) {
   const menuState = React.useContext(
     RootMenuTriggerStateContext
   );
-  const { contextMenuGesture, disabled, hover, racOwnsContextMenu } = React.useContext(DropdownMenuModalContext);
+  const { contextMenuGesture, disabled, hover, racOwnsContextMenu, keyShortcuts } = React.useContext(DropdownMenuModalContext);
   const dataState = state?.isOpen ? "open" : "closed";
   const wantsContextMenu = contextMenuGesture;
   const isDisabled = disabled || props.disabled;
@@ -251,6 +252,8 @@ function DropdownMenuTrigger({ asChild, children, ...props }) {
     return /* @__PURE__ */ jsx(Pressable, { isDisabled, children: React.cloneElement(children, {
       "data-slot": "dropdown-menu-trigger",
       "data-state": dataState,
+      // The root's `shortcut`, announced (gh#1267); a consumer's own value wins.
+      ...keyShortcuts && children.props["aria-keyshortcuts"] === void 0 ? { "aria-keyshortcuts": keyShortcuts } : {},
       // A right click target is usually a region rather than a button, and Shift+F10 needs it
       // focusable — so it is given a tab stop unless the consumer already placed one.
       ...wantsContextMenu && children.props.tabIndex === void 0 ? { tabIndex: 0 } : {},
@@ -270,6 +273,7 @@ function DropdownMenuTrigger({ asChild, children, ...props }) {
       isDisabled,
       "data-slot": "dropdown-menu-trigger",
       "data-state": dataState,
+      "aria-keyshortcuts": props["aria-keyshortcuts"] ?? keyShortcuts,
       children
     }
   );

package/dist/components/navigation/toolbar.d.ts

@@ -1,6 +1,6 @@
 import type { ToolbarGroupProp, ToolbarProp } from "../../props/components/navigation.prop.js";
 export type { FilterBarChipProp, FilterBarChipProp as FilterBarChipProps, FilterBarFilterProp, FilterBarFilterProp as FilterBarFilterProps, FilterBarOverflowProp, FilterBarSearchProp, FilterBarSearchProp as FilterBarSearchProps, ToolbarGroupProp, ToolbarGroupProp as ToolbarGroupProps, ToolbarGroupProp as FilterBarGroupProp, ToolbarGroupProp as FilterBarGroupProps, ToolbarProp, ToolbarProp as ToolbarProps, ToolbarProp as FilterBarProp, ToolbarProp as FilterBarProps, } from "../../props/components/navigation.prop.js";
-export declare function Toolbar({ onClear, hasActiveFilters, sticky, overflow, search, filters, chips, onChipRemove, actions, resultCount, loading, disabled, error, clearDisplay, views, className, children, }: ToolbarProp): import("react").JSX.Element;
+export declare function Toolbar({ onClear, hasActiveFilters, sticky, overflow, search, filters, chips, onChipRemove, actions, resultCount, loading, disabled, error, clearDisplay, views, viewsPlacement, className, children, }: ToolbarProp): import("react").JSX.Element;
 export declare function ToolbarGroup({ label, controlId, className, children }: ToolbarGroupProp): import("react").JSX.Element;
 /** Canonical list-page filter strip. `Toolbar` remains available for backwards compatibility. */
 export declare const FilterBar: typeof Toolbar;

package/dist/components/navigation/toolbar.js

@@ -25,6 +25,7 @@ function Toolbar({
   error,
   clearDisplay = "label",
   views,
+  viewsPlacement = "above",
   className,
   children
 }) {
@@ -90,7 +91,7 @@ function Toolbar({
       className: cn("ui-filter-bar", sticky && "ui-toolbar-sticky", className),
       "data-loading": loading || void 0,
       children: [
-        views != null && /* @__PURE__ */ jsx("div", { className: "ui-filter-bar-views", children: views }),
+        views != null && viewsPlacement === "above" && /* @__PURE__ */ jsx("div", { className: "ui-filter-bar-views", children: views }),
         /* @__PURE__ */ jsxs(
           "div",
           {
@@ -100,6 +101,7 @@ function Toolbar({
             "data-overflow": overflow,
             className: "ui-toolbar",
             children: [
+              views != null && viewsPlacement === "inline" && /* @__PURE__ */ jsx("div", { className: "ui-filter-bar-views", "data-placement": "inline", children: views }),
               search && /* @__PURE__ */ jsx(
                 SearchInput,
                 {

package/dist/contracts/measurement.json

@@ -1,6 +1,6 @@
 {
   "$comment": "AUTO-GENERATED by scripts/gen-measurement-contract.mjs — do not edit. Read this instead of guessing: docs/MEASUREMENT-CONTRACT.md.",
-  "version": "32.2.0",
+  "version": "32.2.1",
   "targetSize": {
     "standard": "WCAG 2.2 SC 2.5.8 Target Size (Minimum), level AA — 24×24 CSS px",
     "min": 24,

package/dist/lib/shortcut.d.ts

@@ -25,9 +25,16 @@ export declare function isEditableTarget(target: EventTarget | null): boolean;
 /**
  * Should this keydown run the shortcut? The ONE gate every global shortcut goes through:
  *  - never during IME composition (the key belongs to the conversion, gh#1054);
+ *  - never from inside a dialog/menu/listbox that does not contain `owner` (gh#1267);
  *  - the combo must match;
  *  - a shortcut with NO command modifier (`"/"`, `"f"`, `"shift+?"`) is ignored while focus is in
  *    an editable element, where that key is text. A modified one (`"mod+k"`) still fires there —
  *    no text field types ⌘K — which is how GitHub's and Linear's palettes behave.
  */
-export declare function shouldHandleShortcut(spec: ShortcutSpec, event: KeyboardEvent): boolean;
+export declare function shouldHandleShortcut(spec: ShortcutSpec, event: KeyboardEvent, owner?: Element | null): boolean;
+/**
+ * The `aria-keyshortcuts` value for a spec (WAI-ARIA 1.2: modifiers then key, `+`-joined, with
+ * `Control`/`Meta`/`Alt`/`Shift` spelled out and a letter upper-cased): `mod+k` → `Meta+K` on
+ * Apple platforms, `Control+K` elsewhere; `"/"` → `/` (gh#1267).
+ */
+export declare function ariaKeyShortcuts(spec: ShortcutSpec): string;

package/dist/lib/shortcut.js

@@ -51,12 +51,29 @@ function isEditableTarget(target) {
   const role = target.getAttribute("role");
   return role === "textbox" || role === "combobox" || role === "searchbox";
 }
-function shouldHandleShortcut(spec, event) {
+function shouldHandleShortcut(spec, event, owner) {
   if (isImeComposing(event) || !shortcutMatches(spec, event)) return false;
+  if (inForeignLayer(event.target, owner)) return false;
   const commandModifier = spec.mod || spec.meta || spec.ctrl || spec.alt;
   return commandModifier || !isEditableTarget(event.target);
 }
+const SHORTCUT_LAYER = '[role="dialog"], [role="alertdialog"], [aria-modal="true"], [role="menu"], [role="listbox"]';
+function inForeignLayer(target, owner) {
+  if (!(target instanceof Element)) return false;
+  const layer = target.closest(SHORTCUT_LAYER);
+  return layer !== null && !(owner && layer.contains(owner));
+}
+function ariaKeyShortcuts(spec) {
+  return [
+    spec.ctrl || spec.mod && !isApplePlatform() ? "Control" : "",
+    spec.meta || spec.mod && isApplePlatform() ? "Meta" : "",
+    spec.alt ? "Alt" : "",
+    spec.shift ? "Shift" : "",
+    spec.key.length === 1 ? spec.key.toUpperCase() : spec.key
+  ].filter(Boolean).join("+");
+}
 export {
+  ariaKeyShortcuts,
   isEditableTarget,
   parseShortcut,
   shortcutHint,

package/dist/props/components/navigation.prop.d.ts

@@ -112,9 +112,17 @@ export type ToolbarProp = {
      * filter set, so it is not one more filter in the strip.
      */
     views?: React.ReactNode;
+    /**
+     * Where `views` sits (gh#1267). `above` (default) is its own row over the strip; `inline` puts it
+     * at the START of the strip's first row, so an idle bar stays ONE row — measured with `above`: an
+     * idle bar 55 → 95px, one table row fewer above the fold at 1440×900.
+     */
+    viewsPlacement?: FilterBarViewsPlacementProp;
     className?: ClassNameProp;
     children?: ChildrenProp;
 };
+/** @see ToolbarProp.viewsPlacement */
+export type FilterBarViewsPlacementProp = "above" | "inline";
 /** @see ToolbarProp.clearDisplay */
 export type FilterBarClearDisplayProp = "label" | "icon";
 /** @see ToolbarGroup */

package/dist/props/registry.d.ts

@@ -2596,6 +2596,11 @@ export declare const COMPONENT_PROP_REGISTRY: {
         readonly file: "components/navigation.prop.ts";
         readonly vocabulary: readonly [];
     };
+    readonly FilterBarViewsPlacementProp: {
+        readonly group: "navigation";
+        readonly file: "components/navigation.prop.ts";
+        readonly vocabulary: readonly [];
+    };
     readonly FilterBarOverflowProp: {
         readonly group: "navigation";
         readonly file: "components/navigation.prop.ts";

package/dist/props/registry.js

@@ -3028,6 +3028,11 @@ const COMPONENT_PROP_REGISTRY = {
     file: "components/navigation.prop.ts",
     vocabulary: []
   },
+  FilterBarViewsPlacementProp: {
+    group: "navigation",
+    file: "components/navigation.prop.ts",
+    vocabulary: []
+  },
   FilterBarOverflowProp: {
     group: "navigation",
     file: "components/navigation.prop.ts",

package/dist/styles/layers.json

@@ -1,5 +1,5 @@
 {
-  "version": "32.2.0",
+  "version": "32.2.1",
   "generatedBy": "scripts/gen-style-layers.mjs — do not edit by hand; run `pnpm gen:style-layers`",
   "base": "base.css",
   "fonts": "fonts.css",

package/dist/styles/table-layout.css

@@ -535,7 +535,8 @@
     white-space: nowrap;
   }
 
-  .ui-data-table-ellipsis[data-truncated]:focus-visible {
+  .ui-data-table-ellipsis[data-truncated]:focus-visible,
+  .ui-data-table-ellipsis[data-truncated]:has(:focus-visible) {
     white-space: normal;
     overflow-wrap: anywhere;
   }