@godxjp/ui 23.4.11 → 23.4.12

package/dist/components/navigation/dropdown-menu.d.ts

@@ -155,9 +155,21 @@ interface DropdownMenuItemPropsOwn {
     textValue?: string;
     /** Mượn thẻ của con thay vì dựng `<div>` riêng — chỗ để nhét một `<Link>` vào một mục menu. */
     asChild?: boolean;
+    /**
+     * `id` và mọi `data-*` ĐI XUỐNG phần tử gốc của item (gh#631).
+     *
+     * Trigger của chính cụm này đã nhận cách xử lý ấy ở 20.0.0, với đúng lý do CHANGELOG ghi:
+     * "trước đó bị nuốt, nên selector e2e của consumer rời ra trong im lặng". Phần item thì chưa,
+     * nên hai nửa của cùng một component hành xử ngược nhau.
+     *
+     * TypeScript không bắt được: JSX luôn cho qua mọi thuộc tính có gạch nối, nên `data-testid`
+     * biên dịch sạch và chỉ hỏng lúc chạy. Với `id` còn tệ hơn — nó là thứ `aria-activedescendant`
+     * trỏ vào, và nó biến mất không một lời.
+     */
+    id?: string;
 }
-type DropdownMenuItemProps = React.PropsWithChildren<DropdownMenuItemPropsOwn>;
-export declare function DropdownMenuItem({ children, className, inset, variant, disabled, onSelect, textValue, asChild, }: DropdownMenuItemProps): React.JSX.Element;
+type DropdownMenuItemProps = React.PropsWithChildren<DropdownMenuItemPropsOwn> & Record<`data-${string}`, unknown>;
+export declare function DropdownMenuItem({ children, className, inset, variant, disabled, onSelect, textValue, asChild, ...forwarded }: DropdownMenuItemProps): React.JSX.Element;
 interface DropdownMenuLabelPropsOwn {
     className?: string;
     inset?: boolean;

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

@@ -267,22 +267,30 @@ function DropdownMenuRadioGroup({
   value,
   onValueChange
 }) {
-  return /* @__PURE__ */ jsx(
-    MenuSection,
-    {
-      "data-slot": "dropdown-menu-radio-group",
-      className,
-      selectionMode: "single",
-      disallowEmptySelection: true,
-      selectedKeys: value == null ? [] : [value],
-      onSelectionChange: (keys) => {
-        if (keys === "all") return;
-        const [first] = [...keys];
-        if (first != null) onValueChange?.(String(first));
-      },
-      children
+  const selection = {
+    className,
+    selectionMode: "single",
+    disallowEmptySelection: true,
+    selectedKeys: value == null ? [] : [value],
+    onSelectionChange: (keys) => {
+      if (keys === "all") return;
+      const [first] = [...keys];
+      if (first != null) onValueChange?.(String(first));
     }
-  );
+  };
+  const runs = [{ label: null, items: [] }];
+  for (const child of React.Children.toArray(children)) {
+    if (React.isValidElement(child) && child.type === DropdownMenuLabel) {
+      runs.push({ label: child, items: [] });
+    } else {
+      runs[runs.length - 1].items.push(child);
+    }
+  }
+  const sections = runs.filter((run) => run.label != null || run.items.length > 0);
+  return /* @__PURE__ */ jsx(Fragment, { children: sections.map((run, index) => /* @__PURE__ */ jsxs(MenuSection, { "data-slot": "dropdown-menu-radio-group", ...selection, children: [
+    run.label,
+    run.items
+  ] }, index)) });
 }
 function DropdownMenuSub({ children }) {
   const parts = React.Children.toArray(children).filter(React.isValidElement);
@@ -372,7 +380,8 @@ function DropdownMenuItem({
   disabled,
   onSelect,
   textValue,
-  asChild
+  asChild,
+  ...forwarded
 }) {
   const tag = borrowedTag(children, asChild);
   return /* @__PURE__ */ jsx(
@@ -394,7 +403,10 @@ function DropdownMenuItem({
             "data-slot": "dropdown-menu-item",
             "data-inset": inset,
             "data-variant": variant,
-            ...radixItemState(state)
+            ...radixItemState(state),
+            // LAST, so a consumer's own `id`/`data-*` wins over ours rather than being
+            // overwritten by the slot bookkeeping above (gh#631).
+            ...forwarded
           },
           content
         );

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": "23.4.11",
+  "version": "23.4.12",
   "targetSize": {
     "standard": "WCAG 2.2 SC 2.5.8 Target Size (Minimum), level AA — 24×24 CSS px",
     "min": 24,

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@godxjp/ui",
-  "version": "23.4.11",
-  "godxUiMcp": "23.4.11",
+  "version": "23.4.12",
+  "godxUiMcp": "23.4.12",
   "license": "Apache-2.0",
   "repository": {
     "type": "git",