@allxsmith/bestax-bulma 5.15.2 → 5.15.3

package/dist/index.esm.js

@@ -12994,7 +12994,7 @@ const LevelRight = ({ className, children, color, bgColor, textColor, ...props }
  * @param {LevelItemProps} props - Props for the LevelItem component.
  * @returns {JSX.Element} The rendered level item.
  */
-const LevelItem = ({ as = 'div', hasTextCentered, className, children, href, target, rel, color, bgColor, textColor, ...props }) => {
+const LevelItem = ({ as = 'div', hasTextCentered, className, children, color, bgColor, textColor, ...props }) => {
     const { bulmaHelperClasses, rest } = useBulmaClasses({
         color: textColor ?? color,
         backgroundColor: bgColor,
@@ -13005,11 +13005,37 @@ const LevelItem = ({ as = 'div', hasTextCentered, className, children, href, tar
         'has-text-centered': hasTextCentered,
     });
     const levelItemClasses = classNames(mainClass, bulmaHelperClasses, className);
-    // If rendering as "a", only pass anchor-specific props
+    // The anchor's own attributes reach an `<a>` and nothing else. The type
+    // declares them at every `as` — narrowing that is source-breaking (#672) — so
+    // the runtime is what keeps `<Level.Item as="p" download>` from rendering a
+    // dead attribute.
+    //
+    // Keyed off `AnchorOnlyAttributes` rather than repeated as a list, because a
+    // hand list beside a derived type drifts in the one direction that matters:
+    // the day React adds an anchor attribute, the type would accept it at every
+    // `as` while the list let it leak onto a `<div>`. `Record` makes that a
+    // compile error here until this object catches up. #682 is the issue for
+    // sharing the rule with the two components that keep their own copies.
+    const ANCHOR_ONLY = {
+        href: true,
+        target: true,
+        download: true,
+        hrefLang: true,
+        ping: true,
+        referrerPolicy: true,
+        media: true,
+        type: true,
+        // `rel` is NOT in the subtraction: React declares it on `HTMLAttributes`,
+        // for every element. It is here because this component has always withheld
+        // it from a non-anchor, and it is named separately so the key set above
+        // stays honest about where each name comes from.
+        rel: true,
+    };
     if (Tag === 'a') {
-        return (jsx("a", { className: levelItemClasses, href: href, target: target, rel: rel, ...rest, children: children }));
+        return (jsx("a", { className: levelItemClasses, ...rest, children: children }));
     }
-    return (jsx(Tag, { className: levelItemClasses, ...rest, children: children }));
+    const withoutAnchorAttrs = Object.fromEntries(Object.entries(rest).filter(([key]) => !(key in ANCHOR_ONLY)));
+    return (jsx(Tag, { className: levelItemClasses, ...withoutAnchorAttrs, children: children }));
 };
 const Level = withSubComponents(LevelComponent, {
     Left: LevelLeft,