carbon-react 106.6.3 → 106.6.4

package/esm/components/button-bar/button-bar-test.stories.js

@@ -0,0 +1,122 @@
+function _extends() { _extends = Object.assign || function (target) { for (var i = 1; i < arguments.length; i++) { var source = arguments[i]; for (var key in source) { if (Object.prototype.hasOwnProperty.call(source, key)) { target[key] = source[key]; } } } return target; }; return _extends.apply(this, arguments); }
+
+import React from "react";
+import { action } from "@storybook/addon-actions";
+import Button from "../button";
+import ButtonBar from ".";
+import IconButton from "../icon-button";
+import Icon from "../icon";
+import { BUTTON_BAR_ICON_POSITIONS, BUTTON_BAR_SIZES } from "./button-bar.config";
+export default {
+  title: "Button Bar/Test",
+  parameters: {
+    info: {
+      disable: true
+    },
+    chromatic: {
+      disable: true
+    }
+  }
+};
+const commonArgTypesButtonBar = {
+  size: {
+    options: BUTTON_BAR_SIZES,
+    control: {
+      type: "select"
+    }
+  },
+  iconPosition: {
+    options: BUTTON_BAR_ICON_POSITIONS,
+    control: {
+      type: "select"
+    }
+  }
+};
+const commonArgsButtonBar = {
+  size: "medium",
+  fullWidth: false,
+  iconPosition: "before"
+};
+export const Default = ({ ...args
+}) => /*#__PURE__*/React.createElement(ButtonBar, _extends({
+  onClick: action("click")
+}, args), /*#__PURE__*/React.createElement(Button, {
+  iconType: "search"
+}, "Example Button"), /*#__PURE__*/React.createElement(Button, {
+  iconType: "pdf"
+}, "Example Button"), /*#__PURE__*/React.createElement(Button, {
+  iconType: "csv"
+}, "Example Button"));
+export const Preview = () => {
+  return /*#__PURE__*/React.createElement(React.Fragment, null, /*#__PURE__*/React.createElement(ButtonBar, {
+    ml: 2,
+    mt: 2
+  }, /*#__PURE__*/React.createElement(IconButton, {
+    onAction: () => {}
+  }, /*#__PURE__*/React.createElement(Icon, {
+    type: "pdf"
+  })), /*#__PURE__*/React.createElement(IconButton, {
+    onAction: () => {}
+  }, /*#__PURE__*/React.createElement(Icon, {
+    type: "csv"
+  })), /*#__PURE__*/React.createElement(IconButton, {
+    onAction: () => {}
+  }, /*#__PURE__*/React.createElement(Icon, {
+    type: "search"
+  }))), BUTTON_BAR_SIZES.map(size => /*#__PURE__*/React.createElement(ButtonBar, {
+    key: size,
+    size: size,
+    ml: 2,
+    mt: 2
+  }, /*#__PURE__*/React.createElement(Button, {
+    iconType: "pdf"
+  }), /*#__PURE__*/React.createElement(Button, {
+    iconType: "csv"
+  }), /*#__PURE__*/React.createElement(Button, {
+    iconType: "search"
+  }))), BUTTON_BAR_ICON_POSITIONS.map(iconPosition => /*#__PURE__*/React.createElement(React.Fragment, {
+    key: iconPosition
+  }, BUTTON_BAR_SIZES.map(size => /*#__PURE__*/React.createElement(React.Fragment, null, /*#__PURE__*/React.createElement(ButtonBar, {
+    key: size + iconPosition,
+    iconPosition: iconPosition,
+    size: size,
+    ml: 2,
+    mt: 2
+  }, /*#__PURE__*/React.createElement(Button, {
+    iconType: "pdf"
+  }, iconPosition), /*#__PURE__*/React.createElement(Button, {
+    iconType: "csv"
+  }, iconPosition), /*#__PURE__*/React.createElement(Button, {
+    iconType: "search"
+  }, iconPosition)))))), BUTTON_BAR_ICON_POSITIONS.map(iconPosition => /*#__PURE__*/React.createElement(React.Fragment, {
+    key: iconPosition
+  }, BUTTON_BAR_SIZES.map(size => /*#__PURE__*/React.createElement(React.Fragment, null, /*#__PURE__*/React.createElement(ButtonBar, {
+    fullWidth: true,
+    key: `${size}${iconPosition}fullWidth`,
+    iconPosition: iconPosition,
+    size: size,
+    ml: 2,
+    mt: 2
+  }, /*#__PURE__*/React.createElement(Button, {
+    iconType: "pdf"
+  }, iconPosition), /*#__PURE__*/React.createElement(Button, {
+    iconType: "csv"
+  }, iconPosition), /*#__PURE__*/React.createElement(Button, {
+    iconType: "search"
+  }, iconPosition)))))));
+};
+Default.story = {
+  name: "default",
+  args: { ...commonArgsButtonBar
+  },
+  argTypes: { ...commonArgTypesButtonBar
+  }
+};
+Preview.story = {
+  name: "visual",
+  parameters: {
+    chromatic: {
+      disable: false
+    }
+  }
+};

package/esm/components/button-bar/button-bar.component.d.ts

@@ -1,16 +1,13 @@
-export default ButtonBar;
-declare function ButtonBar({ children, size, iconPosition, buttonType, ...rest }: {
-    [x: string]: any;
-    children: any;
-    size: any;
-    iconPosition: any;
-    buttonType: any;
-}): JSX.Element;
-declare namespace ButtonBar {
-    const propTypes: any;
-    namespace defaultProps {
-        const size: string;
-        const iconPosition: string;
-        const fullWidth: boolean;
-    }
+import React from "react";
+import { SpaceProps } from "styled-system";
+export interface ButtonBarProps extends SpaceProps {
+    children: React.ReactNode;
+    /** Apply fullWidth style to the button bar */
+    fullWidth?: boolean;
+    /** Defines an Icon position for buttons: "before" | "after" */
+    iconPosition?: "before" | "after";
+    /** Assigns a size to the buttons: "small" | "medium" | "large" */
+    size?: "small" | "medium" | "large";
 }
+declare const ButtonBar: ({ children, size, iconPosition, fullWidth, ...rest }: ButtonBarProps) => JSX.Element;
+export default ButtonBar;

package/esm/components/button-bar/button-bar.component.js

@@ -1,19 +1,31 @@
 function _extends() { _extends = Object.assign || function (target) { for (var i = 1; i < arguments.length; i++) { var source = arguments[i]; for (var key in source) { if (Object.prototype.hasOwnProperty.call(source, key)) { target[key] = source[key]; } } } return target; }; return _extends.apply(this, arguments); }
 
-import React from "react";
+import React, { useMemo } from "react";
 import PropTypes from "prop-types";
-import propTypes from "@styled-system/prop-types";
+import invariant from "invariant";
 import StyledButtonBar from "./button-bar.style";
 import Button from "../button";
 import IconButton from "../icon-button";
 
 const ButtonBar = ({
   children,
-  size,
-  iconPosition,
-  buttonType,
+  size = "medium",
+  iconPosition = "before",
+  fullWidth = false,
   ...rest
 }) => {
+  const hasProperChildren = useMemo(() => {
+    const incorrectChild = React.Children.toArray(children).find(child => {
+      if (! /*#__PURE__*/React.isValidElement(child)) {
+        return true;
+      }
+
+      return child.type !== Button && child.type !== IconButton;
+    });
+    return !incorrectChild;
+  }, [children]);
+  invariant(hasProperChildren, "ButtonBar accepts only `Button` or `IconButton` elements.");
+
   const getBtnProps = child => {
     var _child$props, _child$props2, _child$props2$childre, _child$props2$childre2;
 
@@ -21,44 +33,330 @@ const ButtonBar = ({
       buttonType: "secondary",
       size,
       iconPosition,
-      fullWidth: false,
+      fullWidth,
       "aria-label": child.type === IconButton ? ((_child$props = child.props) === null || _child$props === void 0 ? void 0 : _child$props.ariaLabel) || ((_child$props2 = child.props) === null || _child$props2 === void 0 ? void 0 : (_child$props2$childre = _child$props2.children) === null || _child$props2$childre === void 0 ? void 0 : (_child$props2$childre2 = _child$props2$childre.props) === null || _child$props2$childre2 === void 0 ? void 0 : _child$props2$childre2.type) : ""
     };
     return btnProps;
   };
 
   return /*#__PURE__*/React.createElement(StyledButtonBar, _extends({}, rest, {
+    fullWidth: fullWidth,
     size: size
   }), React.Children.map(children, child => /*#__PURE__*/React.createElement(child.type, getBtnProps(child))));
 };
 
 ButtonBar.propTypes = {
-  /** Assigns a size to the button: "small" | "medium" | "large" */
-  size: PropTypes.oneOf(["small", "medium", "large"]),
-
-  /** Defines an Icon position related to the children: "before" | "after" */
-  iconPosition: PropTypes.oneOf(["before", "after"]),
-
-  /** Styled system spacing props */
-  ...propTypes.space,
-
-  /** Apply fullWidth style to the button bar */
-  fullWidth: PropTypes.bool,
-
-  /** The Button / IconButton elements the button bar displays */
-  children: props => {
-    let error = null;
-    React.Children.forEach(props.children, child => {
-      if (child.type !== Button && child.type !== IconButton) {
-        error = new Error("ButtonBar accepts only `Button` or `IconButton` elements.");
-      }
-    });
-    return error;
-  }
-};
-ButtonBar.defaultProps = {
-  size: "medium",
-  iconPosition: "before",
-  fullWidth: false
+  "children": PropTypes.node,
+  "fullWidth": PropTypes.bool,
+  "iconPosition": PropTypes.oneOf(["after", "before"]),
+  "m": PropTypes.oneOfType([PropTypes.arrayOf(PropTypes.oneOfType([PropTypes.oneOf([null]), PropTypes.number, PropTypes.shape({
+    "__@toStringTag": PropTypes.string.isRequired,
+    "description": PropTypes.string,
+    "toString": PropTypes.func.isRequired,
+    "valueOf": PropTypes.func.isRequired
+  }), PropTypes.string])), PropTypes.number, PropTypes.object, PropTypes.shape({
+    "__@toStringTag": PropTypes.string.isRequired,
+    "description": PropTypes.string,
+    "toString": PropTypes.func.isRequired,
+    "valueOf": PropTypes.func.isRequired
+  }), PropTypes.string]),
+  "margin": PropTypes.oneOfType([PropTypes.arrayOf(PropTypes.oneOfType([PropTypes.oneOf([null]), PropTypes.number, PropTypes.shape({
+    "__@toStringTag": PropTypes.string.isRequired,
+    "description": PropTypes.string,
+    "toString": PropTypes.func.isRequired,
+    "valueOf": PropTypes.func.isRequired
+  }), PropTypes.string])), PropTypes.number, PropTypes.object, PropTypes.shape({
+    "__@toStringTag": PropTypes.string.isRequired,
+    "description": PropTypes.string,
+    "toString": PropTypes.func.isRequired,
+    "valueOf": PropTypes.func.isRequired
+  }), PropTypes.string]),
+  "marginBottom": PropTypes.oneOfType([PropTypes.arrayOf(PropTypes.oneOfType([PropTypes.oneOf([null]), PropTypes.number, PropTypes.shape({
+    "__@toStringTag": PropTypes.string.isRequired,
+    "description": PropTypes.string,
+    "toString": PropTypes.func.isRequired,
+    "valueOf": PropTypes.func.isRequired
+  }), PropTypes.string])), PropTypes.number, PropTypes.object, PropTypes.shape({
+    "__@toStringTag": PropTypes.string.isRequired,
+    "description": PropTypes.string,
+    "toString": PropTypes.func.isRequired,
+    "valueOf": PropTypes.func.isRequired
+  }), PropTypes.string]),
+  "marginLeft": PropTypes.oneOfType([PropTypes.arrayOf(PropTypes.oneOfType([PropTypes.oneOf([null]), PropTypes.number, PropTypes.shape({
+    "__@toStringTag": PropTypes.string.isRequired,
+    "description": PropTypes.string,
+    "toString": PropTypes.func.isRequired,
+    "valueOf": PropTypes.func.isRequired
+  }), PropTypes.string])), PropTypes.number, PropTypes.object, PropTypes.shape({
+    "__@toStringTag": PropTypes.string.isRequired,
+    "description": PropTypes.string,
+    "toString": PropTypes.func.isRequired,
+    "valueOf": PropTypes.func.isRequired
+  }), PropTypes.string]),
+  "marginRight": PropTypes.oneOfType([PropTypes.arrayOf(PropTypes.oneOfType([PropTypes.oneOf([null]), PropTypes.number, PropTypes.shape({
+    "__@toStringTag": PropTypes.string.isRequired,
+    "description": PropTypes.string,
+    "toString": PropTypes.func.isRequired,
+    "valueOf": PropTypes.func.isRequired
+  }), PropTypes.string])), PropTypes.number, PropTypes.object, PropTypes.shape({
+    "__@toStringTag": PropTypes.string.isRequired,
+    "description": PropTypes.string,
+    "toString": PropTypes.func.isRequired,
+    "valueOf": PropTypes.func.isRequired
+  }), PropTypes.string]),
+  "marginTop": PropTypes.oneOfType([PropTypes.arrayOf(PropTypes.oneOfType([PropTypes.oneOf([null]), PropTypes.number, PropTypes.shape({
+    "__@toStringTag": PropTypes.string.isRequired,
+    "description": PropTypes.string,
+    "toString": PropTypes.func.isRequired,
+    "valueOf": PropTypes.func.isRequired
+  }), PropTypes.string])), PropTypes.number, PropTypes.object, PropTypes.shape({
+    "__@toStringTag": PropTypes.string.isRequired,
+    "description": PropTypes.string,
+    "toString": PropTypes.func.isRequired,
+    "valueOf": PropTypes.func.isRequired
+  }), PropTypes.string]),
+  "marginX": PropTypes.oneOfType([PropTypes.arrayOf(PropTypes.oneOfType([PropTypes.oneOf([null]), PropTypes.number, PropTypes.shape({
+    "__@toStringTag": PropTypes.string.isRequired,
+    "description": PropTypes.string,
+    "toString": PropTypes.func.isRequired,
+    "valueOf": PropTypes.func.isRequired
+  }), PropTypes.string])), PropTypes.number, PropTypes.object, PropTypes.shape({
+    "__@toStringTag": PropTypes.string.isRequired,
+    "description": PropTypes.string,
+    "toString": PropTypes.func.isRequired,
+    "valueOf": PropTypes.func.isRequired
+  }), PropTypes.string]),
+  "marginY": PropTypes.oneOfType([PropTypes.arrayOf(PropTypes.oneOfType([PropTypes.oneOf([null]), PropTypes.number, PropTypes.shape({
+    "__@toStringTag": PropTypes.string.isRequired,
+    "description": PropTypes.string,
+    "toString": PropTypes.func.isRequired,
+    "valueOf": PropTypes.func.isRequired
+  }), PropTypes.string])), PropTypes.number, PropTypes.object, PropTypes.shape({
+    "__@toStringTag": PropTypes.string.isRequired,
+    "description": PropTypes.string,
+    "toString": PropTypes.func.isRequired,
+    "valueOf": PropTypes.func.isRequired
+  }), PropTypes.string]),
+  "mb": PropTypes.oneOfType([PropTypes.arrayOf(PropTypes.oneOfType([PropTypes.oneOf([null]), PropTypes.number, PropTypes.shape({
+    "__@toStringTag": PropTypes.string.isRequired,
+    "description": PropTypes.string,
+    "toString": PropTypes.func.isRequired,
+    "valueOf": PropTypes.func.isRequired
+  }), PropTypes.string])), PropTypes.number, PropTypes.object, PropTypes.shape({
+    "__@toStringTag": PropTypes.string.isRequired,
+    "description": PropTypes.string,
+    "toString": PropTypes.func.isRequired,
+    "valueOf": PropTypes.func.isRequired
+  }), PropTypes.string]),
+  "ml": PropTypes.oneOfType([PropTypes.arrayOf(PropTypes.oneOfType([PropTypes.oneOf([null]), PropTypes.number, PropTypes.shape({
+    "__@toStringTag": PropTypes.string.isRequired,
+    "description": PropTypes.string,
+    "toString": PropTypes.func.isRequired,
+    "valueOf": PropTypes.func.isRequired
+  }), PropTypes.string])), PropTypes.number, PropTypes.object, PropTypes.shape({
+    "__@toStringTag": PropTypes.string.isRequired,
+    "description": PropTypes.string,
+    "toString": PropTypes.func.isRequired,
+    "valueOf": PropTypes.func.isRequired
+  }), PropTypes.string]),
+  "mr": PropTypes.oneOfType([PropTypes.arrayOf(PropTypes.oneOfType([PropTypes.oneOf([null]), PropTypes.number, PropTypes.shape({
+    "__@toStringTag": PropTypes.string.isRequired,
+    "description": PropTypes.string,
+    "toString": PropTypes.func.isRequired,
+    "valueOf": PropTypes.func.isRequired
+  }), PropTypes.string])), PropTypes.number, PropTypes.object, PropTypes.shape({
+    "__@toStringTag": PropTypes.string.isRequired,
+    "description": PropTypes.string,
+    "toString": PropTypes.func.isRequired,
+    "valueOf": PropTypes.func.isRequired
+  }), PropTypes.string]),
+  "mt": PropTypes.oneOfType([PropTypes.arrayOf(PropTypes.oneOfType([PropTypes.oneOf([null]), PropTypes.number, PropTypes.shape({
+    "__@toStringTag": PropTypes.string.isRequired,
+    "description": PropTypes.string,
+    "toString": PropTypes.func.isRequired,
+    "valueOf": PropTypes.func.isRequired
+  }), PropTypes.string])), PropTypes.number, PropTypes.object, PropTypes.shape({
+    "__@toStringTag": PropTypes.string.isRequired,
+    "description": PropTypes.string,
+    "toString": PropTypes.func.isRequired,
+    "valueOf": PropTypes.func.isRequired
+  }), PropTypes.string]),
+  "mx": PropTypes.oneOfType([PropTypes.arrayOf(PropTypes.oneOfType([PropTypes.oneOf([null]), PropTypes.number, PropTypes.shape({
+    "__@toStringTag": PropTypes.string.isRequired,
+    "description": PropTypes.string,
+    "toString": PropTypes.func.isRequired,
+    "valueOf": PropTypes.func.isRequired
+  }), PropTypes.string])), PropTypes.number, PropTypes.object, PropTypes.shape({
+    "__@toStringTag": PropTypes.string.isRequired,
+    "description": PropTypes.string,
+    "toString": PropTypes.func.isRequired,
+    "valueOf": PropTypes.func.isRequired
+  }), PropTypes.string]),
+  "my": PropTypes.oneOfType([PropTypes.arrayOf(PropTypes.oneOfType([PropTypes.oneOf([null]), PropTypes.number, PropTypes.shape({
+    "__@toStringTag": PropTypes.string.isRequired,
+    "description": PropTypes.string,
+    "toString": PropTypes.func.isRequired,
+    "valueOf": PropTypes.func.isRequired
+  }), PropTypes.string])), PropTypes.number, PropTypes.object, PropTypes.shape({
+    "__@toStringTag": PropTypes.string.isRequired,
+    "description": PropTypes.string,
+    "toString": PropTypes.func.isRequired,
+    "valueOf": PropTypes.func.isRequired
+  }), PropTypes.string]),
+  "p": PropTypes.oneOfType([PropTypes.arrayOf(PropTypes.oneOfType([PropTypes.oneOf([null]), PropTypes.number, PropTypes.shape({
+    "__@toStringTag": PropTypes.string.isRequired,
+    "description": PropTypes.string,
+    "toString": PropTypes.func.isRequired,
+    "valueOf": PropTypes.func.isRequired
+  }), PropTypes.string])), PropTypes.number, PropTypes.object, PropTypes.shape({
+    "__@toStringTag": PropTypes.string.isRequired,
+    "description": PropTypes.string,
+    "toString": PropTypes.func.isRequired,
+    "valueOf": PropTypes.func.isRequired
+  }), PropTypes.string]),
+  "padding": PropTypes.oneOfType([PropTypes.arrayOf(PropTypes.oneOfType([PropTypes.oneOf([null]), PropTypes.number, PropTypes.shape({
+    "__@toStringTag": PropTypes.string.isRequired,
+    "description": PropTypes.string,
+    "toString": PropTypes.func.isRequired,
+    "valueOf": PropTypes.func.isRequired
+  }), PropTypes.string])), PropTypes.number, PropTypes.object, PropTypes.shape({
+    "__@toStringTag": PropTypes.string.isRequired,
+    "description": PropTypes.string,
+    "toString": PropTypes.func.isRequired,
+    "valueOf": PropTypes.func.isRequired
+  }), PropTypes.string]),
+  "paddingBottom": PropTypes.oneOfType([PropTypes.arrayOf(PropTypes.oneOfType([PropTypes.oneOf([null]), PropTypes.number, PropTypes.shape({
+    "__@toStringTag": PropTypes.string.isRequired,
+    "description": PropTypes.string,
+    "toString": PropTypes.func.isRequired,
+    "valueOf": PropTypes.func.isRequired
+  }), PropTypes.string])), PropTypes.number, PropTypes.object, PropTypes.shape({
+    "__@toStringTag": PropTypes.string.isRequired,
+    "description": PropTypes.string,
+    "toString": PropTypes.func.isRequired,
+    "valueOf": PropTypes.func.isRequired
+  }), PropTypes.string]),
+  "paddingLeft": PropTypes.oneOfType([PropTypes.arrayOf(PropTypes.oneOfType([PropTypes.oneOf([null]), PropTypes.number, PropTypes.shape({
+    "__@toStringTag": PropTypes.string.isRequired,
+    "description": PropTypes.string,
+    "toString": PropTypes.func.isRequired,
+    "valueOf": PropTypes.func.isRequired
+  }), PropTypes.string])), PropTypes.number, PropTypes.object, PropTypes.shape({
+    "__@toStringTag": PropTypes.string.isRequired,
+    "description": PropTypes.string,
+    "toString": PropTypes.func.isRequired,
+    "valueOf": PropTypes.func.isRequired
+  }), PropTypes.string]),
+  "paddingRight": PropTypes.oneOfType([PropTypes.arrayOf(PropTypes.oneOfType([PropTypes.oneOf([null]), PropTypes.number, PropTypes.shape({
+    "__@toStringTag": PropTypes.string.isRequired,
+    "description": PropTypes.string,
+    "toString": PropTypes.func.isRequired,
+    "valueOf": PropTypes.func.isRequired
+  }), PropTypes.string])), PropTypes.number, PropTypes.object, PropTypes.shape({
+    "__@toStringTag": PropTypes.string.isRequired,
+    "description": PropTypes.string,
+    "toString": PropTypes.func.isRequired,
+    "valueOf": PropTypes.func.isRequired
+  }), PropTypes.string]),
+  "paddingTop": PropTypes.oneOfType([PropTypes.arrayOf(PropTypes.oneOfType([PropTypes.oneOf([null]), PropTypes.number, PropTypes.shape({
+    "__@toStringTag": PropTypes.string.isRequired,
+    "description": PropTypes.string,
+    "toString": PropTypes.func.isRequired,
+    "valueOf": PropTypes.func.isRequired
+  }), PropTypes.string])), PropTypes.number, PropTypes.object, PropTypes.shape({
+    "__@toStringTag": PropTypes.string.isRequired,
+    "description": PropTypes.string,
+    "toString": PropTypes.func.isRequired,
+    "valueOf": PropTypes.func.isRequired
+  }), PropTypes.string]),
+  "paddingX": PropTypes.oneOfType([PropTypes.arrayOf(PropTypes.oneOfType([PropTypes.oneOf([null]), PropTypes.number, PropTypes.shape({
+    "__@toStringTag": PropTypes.string.isRequired,
+    "description": PropTypes.string,
+    "toString": PropTypes.func.isRequired,
+    "valueOf": PropTypes.func.isRequired
+  }), PropTypes.string])), PropTypes.number, PropTypes.object, PropTypes.shape({
+    "__@toStringTag": PropTypes.string.isRequired,
+    "description": PropTypes.string,
+    "toString": PropTypes.func.isRequired,
+    "valueOf": PropTypes.func.isRequired
+  }), PropTypes.string]),
+  "paddingY": PropTypes.oneOfType([PropTypes.arrayOf(PropTypes.oneOfType([PropTypes.oneOf([null]), PropTypes.number, PropTypes.shape({
+    "__@toStringTag": PropTypes.string.isRequired,
+    "description": PropTypes.string,
+    "toString": PropTypes.func.isRequired,
+    "valueOf": PropTypes.func.isRequired
+  }), PropTypes.string])), PropTypes.number, PropTypes.object, PropTypes.shape({
+    "__@toStringTag": PropTypes.string.isRequired,
+    "description": PropTypes.string,
+    "toString": PropTypes.func.isRequired,
+    "valueOf": PropTypes.func.isRequired
+  }), PropTypes.string]),
+  "pb": PropTypes.oneOfType([PropTypes.arrayOf(PropTypes.oneOfType([PropTypes.oneOf([null]), PropTypes.number, PropTypes.shape({
+    "__@toStringTag": PropTypes.string.isRequired,
+    "description": PropTypes.string,
+    "toString": PropTypes.func.isRequired,
+    "valueOf": PropTypes.func.isRequired
+  }), PropTypes.string])), PropTypes.number, PropTypes.object, PropTypes.shape({
+    "__@toStringTag": PropTypes.string.isRequired,
+    "description": PropTypes.string,
+    "toString": PropTypes.func.isRequired,
+    "valueOf": PropTypes.func.isRequired
+  }), PropTypes.string]),
+  "pl": PropTypes.oneOfType([PropTypes.arrayOf(PropTypes.oneOfType([PropTypes.oneOf([null]), PropTypes.number, PropTypes.shape({
+    "__@toStringTag": PropTypes.string.isRequired,
+    "description": PropTypes.string,
+    "toString": PropTypes.func.isRequired,
+    "valueOf": PropTypes.func.isRequired
+  }), PropTypes.string])), PropTypes.number, PropTypes.object, PropTypes.shape({
+    "__@toStringTag": PropTypes.string.isRequired,
+    "description": PropTypes.string,
+    "toString": PropTypes.func.isRequired,
+    "valueOf": PropTypes.func.isRequired
+  }), PropTypes.string]),
+  "pr": PropTypes.oneOfType([PropTypes.arrayOf(PropTypes.oneOfType([PropTypes.oneOf([null]), PropTypes.number, PropTypes.shape({
+    "__@toStringTag": PropTypes.string.isRequired,
+    "description": PropTypes.string,
+    "toString": PropTypes.func.isRequired,
+    "valueOf": PropTypes.func.isRequired
+  }), PropTypes.string])), PropTypes.number, PropTypes.object, PropTypes.shape({
+    "__@toStringTag": PropTypes.string.isRequired,
+    "description": PropTypes.string,
+    "toString": PropTypes.func.isRequired,
+    "valueOf": PropTypes.func.isRequired
+  }), PropTypes.string]),
+  "pt": PropTypes.oneOfType([PropTypes.arrayOf(PropTypes.oneOfType([PropTypes.oneOf([null]), PropTypes.number, PropTypes.shape({
+    "__@toStringTag": PropTypes.string.isRequired,
+    "description": PropTypes.string,
+    "toString": PropTypes.func.isRequired,
+    "valueOf": PropTypes.func.isRequired
+  }), PropTypes.string])), PropTypes.number, PropTypes.object, PropTypes.shape({
+    "__@toStringTag": PropTypes.string.isRequired,
+    "description": PropTypes.string,
+    "toString": PropTypes.func.isRequired,
+    "valueOf": PropTypes.func.isRequired
+  }), PropTypes.string]),
+  "px": PropTypes.oneOfType([PropTypes.arrayOf(PropTypes.oneOfType([PropTypes.oneOf([null]), PropTypes.number, PropTypes.shape({
+    "__@toStringTag": PropTypes.string.isRequired,
+    "description": PropTypes.string,
+    "toString": PropTypes.func.isRequired,
+    "valueOf": PropTypes.func.isRequired
+  }), PropTypes.string])), PropTypes.number, PropTypes.object, PropTypes.shape({
+    "__@toStringTag": PropTypes.string.isRequired,
+    "description": PropTypes.string,
+    "toString": PropTypes.func.isRequired,
+    "valueOf": PropTypes.func.isRequired
+  }), PropTypes.string]),
+  "py": PropTypes.oneOfType([PropTypes.arrayOf(PropTypes.oneOfType([PropTypes.oneOf([null]), PropTypes.number, PropTypes.shape({
+    "__@toStringTag": PropTypes.string.isRequired,
+    "description": PropTypes.string,
+    "toString": PropTypes.func.isRequired,
+    "valueOf": PropTypes.func.isRequired
+  }), PropTypes.string])), PropTypes.number, PropTypes.object, PropTypes.shape({
+    "__@toStringTag": PropTypes.string.isRequired,
+    "description": PropTypes.string,
+    "toString": PropTypes.func.isRequired,
+    "valueOf": PropTypes.func.isRequired
+  }), PropTypes.string]),
+  "size": PropTypes.oneOf(["large", "medium", "small"])
 };
 export default ButtonBar;

package/esm/components/button-bar/button-bar.config.d.ts

@@ -1,2 +1,2 @@
-export const BUTTON_BAR_SIZES: string[];
-export const BUTTON_BAR_ICON_POSITIONS: string[];
+export declare const BUTTON_BAR_SIZES: string[];
+export declare const BUTTON_BAR_ICON_POSITIONS: string[];

package/esm/components/button-bar/button-bar.style.d.ts

@@ -1,4 +1,4 @@
-export default ButtonBar;
-declare const ButtonBar: import("styled-components").StyledComponent<"div", any, {
-    size: any;
-}, never>;
+import { SpaceProps } from "styled-system";
+import { ButtonBarProps } from "./button-bar.component";
+declare const StyledButtonBar: import("styled-components").StyledComponent<"div", any, SpaceProps<Required<import("styled-system").Theme<import("styled-system").TLengthStyledSystem>>, string | number | symbol> & Pick<ButtonBarProps, "size" | "fullWidth">, never>;
+export default StyledButtonBar;

package/esm/components/button-bar/button-bar.style.js

@@ -1,32 +1,26 @@
 import styled, { css } from "styled-components";
 import { space } from "styled-system";
-import PropTypes from "prop-types";
-import propTypes from "@styled-system/prop-types";
 import BaseTheme from "../../style/themes/base";
 import StyledIcon from "../icon/icon.style";
-const ButtonBar = styled.div`
+const StyledButtonBar = styled.div`
   ${space}
-  ${stylingForType}
-`;
 
-function stylingForType({
+  ${({
+  fullWidth,
   size
-}) {
-  return css`
-    ${({
-    fullWidth
-  }) => fullWidth && css`
+}) => css`
+    ${fullWidth && css`
+      width: 100%;
+      display: flex;
+      button {
+        box-sizing: content-box;
+        padding: 0;
         width: 100%;
-        display: flex;
-        button {
-          box-sizing: content-box;
-          padding: 0;
-          width: 100%;
-          ${size === "small" && "min-height: 28px"}
-          ${size === "medium" && "min-height: 36px"}
+        ${size === "small" && "min-height: 28px"}
+        ${size === "medium" && "min-height: 36px"}
           ${size === "large" && "min-height: 44px"}
-        }
-      `}
+      }
+    `}
 
     button {
       margin: 0;
@@ -57,19 +51,10 @@ function stylingForType({
         color: var(--colorsActionMajor500);
       }
     }
-  `;
-}
-
-ButtonBar.defaultProps = {
+  `}
+`;
+StyledButtonBar.defaultProps = {
   theme: BaseTheme,
   size: "medium"
 };
-ButtonBar.propTypes = {
-  /** Styled system spacing props */
-  ...propTypes.space,
-
-  /** Assigns a size to the button: "small" | "medium" | "large" */
-  size: PropTypes.oneOf(["small", "medium", "large"]),
-  children: PropTypes.node.isRequired
-};
-export default ButtonBar;
+export default StyledButtonBar;

package/esm/components/button-bar/index.d.ts

@@ -1 +1,2 @@
-export { default } from "./button-bar";
+export { default } from "./button-bar.component";
+export type { ButtonBarProps } from "./button-bar.component";

package/esm/components/link/link.component.js

@@ -86,11 +86,7 @@ const Link = /*#__PURE__*/React.forwardRef(({
       type = "button";
     }
 
-    return /*#__PURE__*/React.createElement(type, { ...componentProps,
-      ...(type === "button" && {
-        role: "link"
-      })
-    }, /*#__PURE__*/React.createElement(React.Fragment, null, renderLinkIcon(), /*#__PURE__*/React.createElement(StyledContent, null, isSkipLink ? l.link.skipLinkLabel() : children), renderLinkIcon("right")));
+    return /*#__PURE__*/React.createElement(type, componentProps, /*#__PURE__*/React.createElement(React.Fragment, null, renderLinkIcon(), /*#__PURE__*/React.createElement(StyledContent, null, isSkipLink ? l.link.skipLinkLabel() : children), renderLinkIcon("right")));
   };
 
   return /*#__PURE__*/React.createElement(StyledLink, _extends({

package/esm/components/menu/__internal__/spec-helper/index.js

@@ -9,7 +9,8 @@ const events = {
 };
 
 const openSubmenu = wrapper => {
-  const menuItem = wrapper.find('[data-component="submenu-wrapper"]').find("a");
+  const menuWrapper = wrapper.find('[data-component="submenu-wrapper"]');
+  const menuItem = menuWrapper.exists("a") ? menuWrapper.find("a") : menuWrapper.find("button");
   menuItem.getDOMNode().focus();
   act(() => {
     wrapper.find(StyledMenuItemWrapper).at(0).props().onKeyDown(events.space);