@splunk/react-ui 5.0.0-beta.4 → 5.0.0-beta.5

package/Clickable.js

@@ -72,15 +72,15 @@
     var t =  e.n(r);
     // CONCATENATED MODULE: external "prop-types"
     const i = require("prop-types");
-    var o =  e.n(i);
+    var a =  e.n(i);
     // CONCATENATED MODULE: external "lodash/has"
-    const a = require("lodash/has");
+    const o = require("lodash/has");
     // CONCATENATED MODULE: external "lodash/isString"
     const l = require("lodash/isString");
     var u =  e.n(l);
     // CONCATENATED MODULE: external "lodash/omit"
-    const c = require("lodash/omit");
-    var d =  e.n(c);
+    const d = require("lodash/omit");
+    var c =  e.n(d);
     // CONCATENATED MODULE: external "styled-components"
     const s = require("styled-components");
     var f =  e.n(s);
@@ -94,25 +94,25 @@
     // CONCATENATED MODULE: ./src/Clickable/NavigationProvider.tsx
     /** @public */
     var p = {
-        children: o().node,
-        onClick: o().func,
-        onLinkClick: o().func,
-        transformUrl: o().func
+        children: a().node,
+        onClick: a().func,
+        onLinkClick: a().func,
+        transformUrl: a().func
     };
     var m =  t().createContext({});
     /**
  * Used to provide an override for the `onClick` for links for single page applications so that
  * internal links can navigate without a page reload.
  */    function y(e) {
-        var n = e.children, i = e.onClick, o = e.onLinkClick, a = e.transformUrl;
+        var n = e.children, i = e.onClick, a = e.onLinkClick, o = e.transformUrl;
         // @docs-props-type NavigationProviderProps
                 var l = (0, r.useMemo)((function() {
             return {
                 onClick: i,
-                onLinkClick: o,
-                transformUrl: a
+                onLinkClick: a,
+                transformUrl: o
             };
-        }), [ i, o, a ]);
+        }), [ i, a, o ]);
         
         return t().createElement(m.Provider, {
             value: l
@@ -152,16 +152,16 @@
     }
     function g(e, n) {
         if (null == e) return {};
-        var r, t, i = h(e, n);
+        var r, t, i = O(e, n);
         if (Object.getOwnPropertySymbols) {
-            var o = Object.getOwnPropertySymbols(e);
-            for (t = 0; t < o.length; t++) {
-                r = o[t], n.includes(r) || {}.propertyIsEnumerable.call(e, r) && (i[r] = e[r]);
+            var a = Object.getOwnPropertySymbols(e);
+            for (t = 0; t < a.length; t++) {
+                r = a[t], n.includes(r) || {}.propertyIsEnumerable.call(e, r) && (i[r] = e[r]);
             }
         }
         return i;
     }
-    function h(e, n) {
+    function O(e, n) {
         if (null == e) return {};
         var r = {};
         for (var t in e) {
@@ -172,127 +172,130 @@
         }
         return r;
     }
-    var O = /^[a-z0-9]+:/;
+    var h = /^[a-z0-9]+:/;
     var w = /^\/\//;
     var P = /^\//;
     var S = function e(n) {
         return n != null && P.test(n);
     };
     var j = function e(n) {
-        return n != null && !O.test(n) && !w.test(n);
+        return n != null && !h.test(n) && !w.test(n);
     };
     var x = {
-        children: o().node,
-        disabled: o().oneOfType([ o().bool, o().oneOf([ "dimmed", "disabled" ]) ]),
-        elementRef: o().oneOfType([ o().func, o().object ]),
-        onClick: o().func,
-        openInNewContext: o().bool,
-        navigationLabel: o().string,
-        to: o().string
+        children: a().node,
+        disabled: a().oneOfType([ a().bool, a().oneOf([ "dimmed", "disabled" ]) ]),
+        elementRef: a().oneOfType([ a().func, a().object ]),
+        onClick: a().func,
+        openInNewContext: a().bool,
+        navigationLabel: a().string,
+        to: a().oneOfType([ a().string, a().oneOf([ undefined ]) ])
     };
-    function L(e, n) {
-        return !!n && !e;
+    function L(e) {
+        return e !== undefined;
     }
     /**
  * `Clickable` renders as a `button` element, or as an `a` element if the `to` prop is set
  * and the `disabled` prop is `false`. This is called link mode.
- */    var I =  t().forwardRef((function(e, n) {
-        var i = e.children, o = e.disabled, a = e.elementRef, l = e.navigationLabel, c = e.onClick, s = e.openInNewContext, f = e.to, v = g(e, [ "children", "disabled", "elementRef", "navigationLabel", "onClick", "openInNewContext", "to" ]);
+ */    var I =  t().forwardRef((function(e, 
+    // SUI-7623 - needs to be specified for react-docgen versions less than 7.1.0
+    n) {
+        var i = e.children, a = e.disabled, o = e.elementRef, l = e.navigationLabel, d = e.onClick, s = e.openInNewContext, f = e.tag, v = e.to, p = g(e, [ "children", "disabled", "elementRef", "navigationLabel", "onClick", "openInNewContext", "tag", "to" ]);
         // @docs-props-type ClickablePropsBase
-                var p = (0, r.useContext)(m), y = p.onClick, h = p.onLinkClick, O = p.transformUrl;
+                var y = (0, r.useContext)(m), O = y.onClick, h = y.onLinkClick, w = y.transformUrl;
         (0, r.useEffect)((function() {
             if (false) {}
             // this effect only runs once
             // eslint-disable-next-line react-hooks/exhaustive-deps
                 }), []);
-        var w = (0, r.useCallback)((function(e) {
-            k(a, e);
+        var P = (0, r.useCallback)((function(e) {
+            k(o, e);
             k(n, e);
-        }), [ a, n ]);
+        }), [ o, n ]);
         // memoizing this result causes TS to lose track of the type guard established by isLinkCheck
-                var P = L(o, f);
+                var x = L(v);
         // ideally we wouldn't need to pass "to" as "toValue" here
         // but without doing that TS thinks the return value of this is
         // string | undefined which doesn't work for usages of this function
-                var x = (0, r.useCallback)((function(e) {
-            return O != null ? O(e, {
+                var I = (0, r.useCallback)((function(e) {
+            return w != null ? w(e, {
                 isInternal: j(e),
                 isRootRelative: S(e)
             }) : e;
-        }), [ O ]);
-        var I = (0, r.useCallback)((function(e) {
-            if (P) {
+        }), [ w ]);
+        var R = (0, r.useCallback)((function(e) {
+            if (x) {
                 // when user command-click on mac or ctrl-click on other platforms, and
                 // Tag is an <a>, let the click pass through, let the <a> to achieve user's
                 // intent of 'open in new context'
                 // on mac, ctrl-click will be caught and open option menu even before hitting
                 // the DOM, so we're safe to check both metaKey and ctrlKey here
                 // without platform sniffing
-                if ((e.metaKey || e.ctrlKey) && f) {
+                if ((e.metaKey || e.ctrlKey) && v) {
                     return;
                 }
             }
-            c === null || c === void 0 ? void 0 : c(e);
-            var n = c == null && y != null;
-            if (P && (n || h != null)) {
+            d === null || d === void 0 ? void 0 : d(e);
+            var n = d == null && O != null;
+            if (x && (n || h != null)) {
                 var r = l;
                 if (!r && u()(i)) {
                     r = i;
                 }
                 var t = {
-                    to: x(f),
-                    originalTo: f,
+                    to: I(v),
+                    originalTo: v,
                     openInNewContext: s,
                     label: r
                 };
                 if (n) {
-                    y(e, t);
+                    O(e, t);
                 }
                 h === null || h === void 0 ? void 0 : h(e, t);
             }
-        }), [ i, x, P, l, c, y, h, s, f ]);
-        // Only set the href attribute when enabled, and therefore using an <a> tag
-                if (P) {
-            var R = s ? "_blank" : undefined;
-            var q = s && j(f) ? "noopener noreferrer" : undefined;
+        }), [ i, I, x, l, d, O, h, s, v ]);
+        // Only set the href attribute when enabled and therefore using an <a> tag
+                if (x && f !== "button") {
+            if (a) {
+                
+                return t().createElement(b, C({
+                    "data-test": "clickable",
+                    "data-test-disabled": a,
+                    ref: P,
+                    role: "link",
+                    "aria-disabled": "true"
+                }, p), i);
+            }
+            var T = s ? "_blank" : undefined;
+            var q = s && j(v) ? "noopener noreferrer" : undefined;
             
             return t().createElement(b, C({
                 "data-test": "clickable",
-                ref: w,
-                href: x(f),
-                target: R,
+                ref: P,
+                href: I(v),
+                target: T,
                 rel: q,
-                onClick: I
-            }, d()(v)), i);
-        }
-        if (o && f != null) {
-            
-            return t().createElement(b, C({
-                "data-test": "clickable",
-                ref: w,
-                role: "link",
-                "aria-disabled": "true"
-            }, v), i);
+                onClick: R
+            }, c()(p)), i);
         }
         // button variant
-                var N = o === true || o === "dimmed" || undefined;
-        var T;
+                var N = a === true || a === "dimmed" || undefined;
+        var _;
         if (N) {
-            T = "dimmed";
-        } else if (o === "disabled") {
-            T = "disabled";
+            _ = "dimmed";
+        } else if (a === "disabled") {
+            _ = "disabled";
         }
         
         return t().createElement(b, C({
             as: "button",
             "aria-disabled": N,
             "data-test": "clickable",
-            "data-test-disabled": T,
-            ref: w,
+            "data-test-disabled": _,
+            ref: P,
             type: "button",
-            disabled: o === "disabled",
-            onClick: c && !o ? I : undefined
-        }, d()(v, "disabled")), i);
+            disabled: a === "disabled",
+            onClick: d && !a ? R : undefined
+        }, c()(p, "disabled")), i);
     }));
     I.propTypes = x;
     /* harmony default export */    const R = I;

package/ComboBox.js

@@ -200,7 +200,7 @@
             "data-test-value": b,
             description: o,
             descriptionPosition: i,
-            disabled: l,
+            disabled: l ? "disabled" : undefined,
             elementRef: g,
             startAdornment: c,
             matchRanges: f,