@splunk/react-ui 5.3.0 → 5.4.0

package/Link.js

@@ -83,11 +83,11 @@
     // CONCATENATED MODULE: external "@splunk/themes"
     const f = require("@splunk/themes");
     // CONCATENATED MODULE: ./src/Link/icons/External.tsx
-    var v = d()(p()).withConfig({
+    var b = d()(p()).withConfig({
         displayName: "External",
         componentId: "sc-13bdx45-0"
     })([ "vertical-align:text-bottom;margin-left:", ";" ], f.variables.spacingXSmall);
-    /* harmony default export */    const b = v;
+    /* harmony default export */    const v = b;
     // CONCATENATED MODULE: external "@splunk/react-ui/Clickable"
     const g = require("@splunk/react-ui/Clickable");
     var m =  e.n(g);
@@ -149,12 +149,12 @@
  */    var S =  r().forwardRef((function(e, 
     // SUI-7623 - needs to be specified for react-docgen versions less than 7.1.0
     n) {
-        var t = e.appearance, o = t === void 0 ? "inline" : t, a = e.children, i = e.disabled, s = i === void 0 ? false : i, c = e.elementRef, d = e.openInNewContext, u = d === void 0 ? false : d, p = e.tag, f = p === void 0 ? "a" : p, v = e.to, g = w(e, [ "appearance", "children", "disabled", "elementRef", "openInNewContext", "tag", "to" ]);
+        var t = e.appearance, o = t === void 0 ? "inline" : t, a = e.children, i = e.disabled, s = i === void 0 ? false : i, c = e.elementRef, d = e.openInNewContext, u = d === void 0 ? false : d, p = e.tag, f = p === void 0 ? "a" : p, b = e.to, g = w(e, [ "appearance", "children", "disabled", "elementRef", "openInNewContext", "tag", "to" ]);
         // @docs-props-type LinkPropsBase
                 var m = typeof u === "string" ? u : k;
         // Ensure `Link` always renders as an `a` (unless overriden by `tag`) by passing a value to Clickable,
         // using '' if disabled or the to is not passed.
-                var O = s && "" || v && v || "";
+                var O = f !== "button" ? s && "" || b && b || "" : undefined;
         
         return r().createElement(h, y({
             "data-test": "link",
@@ -166,7 +166,7 @@
             to: O
         }, g, {
             tag: f
-        }), a, u &&  r().createElement(r().Fragment, null,  r().createElement(b, null),  r().createElement(l(), null, m)));
+        }), a, u &&  r().createElement(r().Fragment, null,  r().createElement(v, null),  r().createElement(l(), null, m)));
     }));
     S.propTypes = x;
     /* harmony default export */    const C = S;

package/Markdown.js

@@ -136,6 +136,7 @@
             return e && "function" == typeof Symbol && e.constructor === Symbol && e !== Symbol.prototype ? "symbol" : typeof e;
         }, g(e);
     }
+    // eslint-disable-next-line no-relative-imports-of-published -- because MarkdownHeading and MarkdownAnchorHeading come from the same @splunk/react-ui/Markdown top level export so it would be a circular dependency if using the external/alias (ok here because thin/no context)
     // When this is used in MDX, the node isn't guaranteed to be text,
     // so this method extracts the actual text from the node.
         function h(e) {
@@ -799,15 +800,18 @@
                 tbody: Me
             };
         }), [ r, g, c, u, s, d, v, y ]);
+        var w = (0, n.useMemo)((function() {
+            // prettier-ignore
+            return {
+                pre: (0, f.css)([ ":is(", ")" ], S),
+                blockquote: (0, f.css)([ ":is(", ")" ], x)
+            };
+        }), []);
         
         return t().createElement(p(), Ue({
             "data-test": "markdown",
             elementRef: i,
-            elementSelectors: {
-                /* stylelint-disable */
-                pre: (0, f.css)([ ":is(", ")" ], S),
-                blockquote: (0, f.css)([ ":is(", ")" ], x)
-                /* stylelint-enable */            }
+            elementSelectors: w
         }, O),  t().createElement(a(), Ue({
             remarkPlugins: Qe,
             components: h

package/Menu.js

@@ -436,109 +436,114 @@
     };
     var ye = (0, se._)("(Opens new window)");
     function he(e) {
-        var t = e.active, a = e.children, o = e.description, l = e.descriptionPosition, s = l === void 0 ? "bottom" : l, c = e.disabled, u = e.endAdornment, d = e.elementRef, v = e.hasSubmenu, p = e.itemKey, f = e.matchRanges, m = e.onClick, b = e.onFocus, g = e.openInNewContext, y = e.role, w = e.selectable, A = e.selectableAppearance, D = A === void 0 ? "checkmark" : A, T = e.selected, q = e.startAdornment, $ = e.to, R = e.truncate, H = me(e, [ "active", "children", "description", "descriptionPosition", "disabled", "endAdornment", "elementRef", "hasSubmenu", "itemKey", "matchRanges", "onClick", "onFocus", "openInNewContext", "role", "selectable", "selectableAppearance", "selected", "startAdornment", "to", "truncate" ]);
+        var t;
+        var a = e.active, o = e.children, l = e.description, s = e.descriptionPosition, c = s === void 0 ? "bottom" : s, u = e.disabled, d = e.endAdornment, v = e.elementRef, p = e.hasSubmenu, f = e.itemKey, m = e.matchRanges, b = e.onClick, g = e.onFocus, y = e.openInNewContext, w = e.role, A = e.selectable, D = e.selectableAppearance, T = D === void 0 ? "checkmark" : D, q = e.selected, $ = e.startAdornment, R = e.to, H = e.truncate, F = me(e, [ "active", "children", "description", "descriptionPosition", "disabled", "endAdornment", "elementRef", "hasSubmenu", "itemKey", "matchRanges", "onClick", "onFocus", "openInNewContext", "role", "selectable", "selectableAppearance", "selected", "startAdornment", "to", "truncate" ]);
         // @docs-props-type ItemPropsBase
-                var F = (0, r.useContext)(N), M = F.preventFocus;
+                var M = (0, r.useContext)(N), L = M.preventFocus;
+ // Use arguments[0] (raw props) to check if `selectableAppearance` was explicitly passed by the caller.
+        // This prevents the default value from incorrectly triggering the dev warning.
+        // eslint-disable-next-line prefer-rest-params
+                var K = (t = arguments[0]) === null || t === void 0 ? void 0 : t.selectableAppearance;
         (0, r.useEffect)((function() {
             if (false) {}
-        }), [ R, s, f, a, T, w, D ]);
-        var L = (0, r.useCallback)((function(e) {
-            b === null || b === void 0 ? void 0 : b(e, {
-                itemKey: p
+        }), [ H, c, m, o, q, A, T, K ]);
+        var B = (0, r.useCallback)((function(e) {
+            g === null || g === void 0 ? void 0 : g(e, {
+                itemKey: f
             });
-        }), [ p, b ]);
-        var K = (0, r.useCallback)((function(e) {
-            if (M) {
+        }), [ f, g ]);
+        var W = (0, r.useCallback)((function(e) {
+            if (L) {
                 e.preventDefault();
             }
-        }), [ M ]);
-        var B = (0, r.useMemo)((function() {
-            if (!f || !te()(a)) {
-                return a;
+        }), [ L ]);
+        var z = (0, r.useMemo)((function() {
+            if (!m || !te()(o)) {
+                return o;
             }
             var e = [];
             // before first match. May be empty string.
-                        e.push(a.substring(0, f[0].start));
-            f.forEach((function(t, r) {
+                        e.push(o.substring(0, m[0].start));
+            m.forEach((function(t, r) {
                 e.push(
                 
                 // eslint-disable-next-line react/no-array-index-key
                 n().createElement(P, {
                     key: r,
                     "data-test": "match"
-                }, a.substring(t.start, t.end)));
-                if (r < f.length - 1) {
-                    e.push(a.substring(t.end, f[r + 1].start));
+                }, o.substring(t.start, t.end)));
+                if (r < m.length - 1) {
+                    e.push(o.substring(t.end, m[r + 1].start));
                 } else {
-                    e.push(a.substring(t.end, a.length));
+                    e.push(o.substring(t.end, o.length));
                 }
             }));
             return e;
-        }), [ a, f ]);
-        var W = T === true || T === "some";
-        var z = w || W;
-        var V = y || {
+        }), [ o, m ]);
+        var V = q === true || q === "some";
+        var X = A || V;
+        var G = w || {
             nonselectable: "menuitem",
             checkmark: "menuitemradio",
             checkbox: "menuitemcheckbox"
-        }[z ? D : "nonselectable"];
-        var X = {
-            "aria-haspopup": v ? true : undefined
+        }[X ? T : "nonselectable"];
+        var J = {
+            "aria-haspopup": p ? true : undefined
         };
-        if (V === "menuitemradio" || V === "menuitemcheckbox") {
-            X["aria-checked"] = T === "some" ? "mixed" : W;
-        } else if (V === "option") {
-            X["aria-selected"] = W;
+        if (G === "menuitemradio" || G === "menuitemcheckbox") {
+            J["aria-checked"] = q === "some" ? "mixed" : V;
+        } else if (G === "option") {
+            J["aria-selected"] = V;
         }
-        var G = o && s === "right";
-        var J = o && !G;
-        var Q = T && D === "checkmark" &&  n().createElement(O, null,  n().createElement(ne(), {
+        var Q = l && c === "right";
+        var U = l && !Q;
+        var Y = q && T === "checkmark" &&  n().createElement(O, null,  n().createElement(ne(), {
             "data-test": "selected-checkmark",
             inline: true
         }));
-        var U = false;
-        var Y;
-        if (g) {
-            U = true;
-            Y = typeof g === "string" ? g : ye;
+        var Z = false;
+        var ee;
+        if (y) {
+            Z = true;
+            ee = typeof y === "string" ? y : ye;
         }
         
         return n().createElement(C, fe({
-            $active: t,
-            $isSelectable: z,
-            $preventFocus: M || false,
-            $selectableAppearance: D !== null && D !== void 0 ? D : false,
-            $selected: W,
-            "data-has-icon": !!q || !!u,
+            $active: a,
+            $isSelectable: X,
+            $preventFocus: L || false,
+            $selectableAppearance: T !== null && T !== void 0 ? T : false,
+            $selected: V,
+            "data-has-icon": !!$ || !!d,
             "data-test": "item",
-            "data-test-selected": z ? W : null,
-            disabled: c,
-            elementRef: d,
-            onClick: m,
-            onFocus: L,
-            onMouseDown: K,
-            openInNewContext: U,
-            role: V,
-            tabIndex: M ? -1 : undefined,
-            to: $,
-            title: R && te()(a) ? a : undefined
-        }, X, i()(H, "onFocus")), v &&  n().createElement(j, null,  n().createElement(ie(), null)),  n().createElement(I, null, w && D === "checkbox" &&  n().createElement(x, {
-            checked: T === "some" ? "indeterminate" : T,
-            disabled: !!c,
+            "data-test-selected": X ? V : null,
+            disabled: u,
+            elementRef: v,
+            onClick: b,
+            onFocus: B,
+            onMouseDown: W,
+            openInNewContext: Z,
+            role: G,
+            tabIndex: L ? -1 : undefined,
+            to: R,
+            title: H && te()(o) ? o : undefined
+        }, J, i()(F, "onFocus")), p &&  n().createElement(j, null,  n().createElement(ie(), null)),  n().createElement(I, null, A && T === "checkbox" &&  n().createElement(x, {
+            checked: q === "some" ? "indeterminate" : q,
+            disabled: !!u,
             inert: true
-        }), q &&  n().createElement(k, null, q),  n().createElement(E, {
-            $descriptionBottom: !!J,
-            $truncate: R !== null && R !== void 0 ? R : false
+        }), $ &&  n().createElement(k, null, $),  n().createElement(E, {
+            $descriptionBottom: !!U,
+            $truncate: H !== null && H !== void 0 ? H : false
         },  n().createElement(_, {
-            $truncate: R !== null && R !== void 0 ? R : false,
+            $truncate: H !== null && H !== void 0 ? H : false,
             "data-test": "label"
-        }, B, g &&  n().createElement(n().Fragment, null,  n().createElement(pe, null),  n().createElement(le(), null, Y))), J &&  n().createElement(h, {
+        }, z, y &&  n().createElement(n().Fragment, null,  n().createElement(pe, null),  n().createElement(le(), null, ee))), U &&  n().createElement(h, {
             "data-test": "description"
-        }, o), G &&  n().createElement(S, {
+        }, l), Q &&  n().createElement(S, {
             "data-test": "description"
-        }, o)), u &&  n().createElement(k, {
-            endAdornment: u
-        }, u), Q));
+        }, l)), d &&  n().createElement(k, {
+            endAdornment: d
+        }, d), Y));
     }
     he.propTypes = ge;
     he.as = "Item";