@carbon/ibm-products 1.6.1 → 1.9.0

package/es/components/EmptyStates/ErrorEmptyState/ErrorEmptyState.js

@@ -23,7 +23,7 @@ import { getDevtoolsProps } from '../../../global/js/utils/devtools';
 import { pkg } from '../../../settings';
 import { EmptyStateContent } from '../EmptyStateContent';
 import { ErrorIllustration } from '../assets/ErrorIllustration';
-import { EmptyStateDefaultProps } from '../EmptyState'; // The block part of our conventional BEM class names (blockClass__E--M).
+import { defaults } from '../EmptyState'; // The block part of our conventional BEM class names (blockClass__E--M).
 
 var blockClass = "".concat(pkg.prefix, "--empty-state");
 var componentName = 'ErrorEmptyState';
@@ -32,7 +32,8 @@ export var ErrorEmptyState = /*#__PURE__*/React.forwardRef(function (_ref, ref)
       className = _ref.className,
       illustrationTheme = _ref.illustrationTheme,
       link = _ref.link,
-      size = _ref.size,
+      _ref$size = _ref.size,
+      size = _ref$size === void 0 ? defaults.size : _ref$size,
       subtitle = _ref.subtitle,
       title = _ref.title,
       rest = _objectWithoutProperties(_ref, _excluded);
@@ -105,9 +106,4 @@ ErrorEmptyState.propTypes = {
    * Empty state title
    */
   title: PropTypes.oneOfType([PropTypes.string, PropTypes.node]).isRequired
-}; // Default values for component props. Default values are not required for
-// props that are required, nor for props where the component can apply
-// 'undefined' values reasonably. Default values should be provided when the
-// component needs to make a choice or assumption when a prop is not supplied.
-
-ErrorEmptyState.defaultProps = EmptyStateDefaultProps;
+};

package/es/components/EmptyStates/NoDataEmptyState/NoDataEmptyState.js

@@ -23,7 +23,7 @@ import { getDevtoolsProps } from '../../../global/js/utils/devtools';
 import { pkg } from '../../../settings';
 import { EmptyStateContent } from '../EmptyStateContent';
 import { NoDataIllustration } from '../assets/NoDataIllustration';
-import { EmptyStateDefaultProps } from '../EmptyState'; // The block part of our conventional BEM class names (blockClass__E--M).
+import { defaults } from '../EmptyState'; // The block part of our conventional BEM class names (blockClass__E--M).
 
 var blockClass = "".concat(pkg.prefix, "--empty-state");
 var componentName = 'NoDataEmptyState';
@@ -32,7 +32,8 @@ export var NoDataEmptyState = /*#__PURE__*/React.forwardRef(function (_ref, ref)
       className = _ref.className,
       illustrationTheme = _ref.illustrationTheme,
       link = _ref.link,
-      size = _ref.size,
+      _ref$size = _ref.size,
+      size = _ref$size === void 0 ? defaults.size : _ref$size,
       subtitle = _ref.subtitle,
       title = _ref.title,
       rest = _objectWithoutProperties(_ref, _excluded);
@@ -105,9 +106,4 @@ NoDataEmptyState.propTypes = {
    * Empty state title
    */
   title: PropTypes.oneOfType([PropTypes.string, PropTypes.node]).isRequired
-}; // Default values for component props. Default values are not required for
-// props that are required, nor for props where the component can apply
-// 'undefined' values reasonably. Default values should be provided when the
-// component needs to make a choice or assumption when a prop is not supplied.
-
-NoDataEmptyState.defaultProps = EmptyStateDefaultProps;
+};

package/es/components/EmptyStates/NoTagsEmptyState/NoTagsEmptyState.js

@@ -23,7 +23,7 @@ import { getDevtoolsProps } from '../../../global/js/utils/devtools';
 import { pkg } from '../../../settings';
 import { EmptyStateContent } from '../EmptyStateContent';
 import { NoTagsIllustration } from '../assets/NoTagsIllustration';
-import { EmptyStateDefaultProps } from '../EmptyState'; // The block part of our conventional BEM class names (blockClass__E--M).
+import { defaults } from '../EmptyState'; // The block part of our conventional BEM class names (blockClass__E--M).
 
 var blockClass = "".concat(pkg.prefix, "--empty-state");
 var componentName = 'NoTagsEmptyState';
@@ -32,7 +32,8 @@ export var NoTagsEmptyState = /*#__PURE__*/React.forwardRef(function (_ref, ref)
       className = _ref.className,
       illustrationTheme = _ref.illustrationTheme,
       link = _ref.link,
-      size = _ref.size,
+      _ref$size = _ref.size,
+      size = _ref$size === void 0 ? defaults.size : _ref$size,
       subtitle = _ref.subtitle,
       title = _ref.title,
       rest = _objectWithoutProperties(_ref, _excluded);
@@ -105,9 +106,4 @@ NoTagsEmptyState.propTypes = {
    * Empty state title
    */
   title: PropTypes.oneOfType([PropTypes.string, PropTypes.node]).isRequired
-}; // Default values for component props. Default values are not required for
-// props that are required, nor for props where the component can apply
-// 'undefined' values reasonably. Default values should be provided when the
-// component needs to make a choice or assumption when a prop is not supplied.
-
-NoTagsEmptyState.defaultProps = EmptyStateDefaultProps;
+};

package/es/components/EmptyStates/NotFoundEmptyState/NotFoundEmptyState.js

@@ -23,7 +23,7 @@ import { getDevtoolsProps } from '../../../global/js/utils/devtools';
 import { pkg } from '../../../settings';
 import { EmptyStateContent } from '../EmptyStateContent';
 import { NotFoundIllustration } from '../assets/NotFoundIllustration';
-import { EmptyStateDefaultProps } from '../EmptyState'; // The block part of our conventional BEM class names (blockClass__E--M).
+import { defaults } from '../EmptyState'; // The block part of our conventional BEM class names (blockClass__E--M).
 
 var blockClass = "".concat(pkg.prefix, "--empty-state");
 var componentName = 'NotFoundEmptyState';
@@ -32,7 +32,8 @@ export var NotFoundEmptyState = /*#__PURE__*/React.forwardRef(function (_ref, re
       className = _ref.className,
       illustrationTheme = _ref.illustrationTheme,
       link = _ref.link,
-      size = _ref.size,
+      _ref$size = _ref.size,
+      size = _ref$size === void 0 ? defaults.size : _ref$size,
       subtitle = _ref.subtitle,
       title = _ref.title,
       rest = _objectWithoutProperties(_ref, _excluded);
@@ -105,9 +106,4 @@ NotFoundEmptyState.propTypes = {
    * Empty state title
    */
   title: PropTypes.oneOfType([PropTypes.string, PropTypes.node]).isRequired
-}; // Default values for component props. Default values are not required for
-// props that are required, nor for props where the component can apply
-// 'undefined' values reasonably. Default values should be provided when the
-// component needs to make a choice or assumption when a prop is not supplied.
-
-NotFoundEmptyState.defaultProps = EmptyStateDefaultProps;
+};

package/es/components/EmptyStates/NotificationsEmptyState/NotificationsEmptyState.js

@@ -23,7 +23,7 @@ import { getDevtoolsProps } from '../../../global/js/utils/devtools';
 import { pkg } from '../../../settings';
 import { EmptyStateContent } from '../EmptyStateContent';
 import { NotificationsIllustration } from '../assets/NotificationsIllustration';
-import { EmptyStateDefaultProps } from '../EmptyState'; // The block part of our conventional BEM class names (blockClass__E--M).
+import { defaults } from '../EmptyState'; // The block part of our conventional BEM class names (blockClass__E--M).
 
 var blockClass = "".concat(pkg.prefix, "--empty-state");
 var componentName = 'NotificationsEmptyState';
@@ -32,7 +32,8 @@ export var NotificationsEmptyState = /*#__PURE__*/React.forwardRef(function (_re
       className = _ref.className,
       illustrationTheme = _ref.illustrationTheme,
       link = _ref.link,
-      size = _ref.size,
+      _ref$size = _ref.size,
+      size = _ref$size === void 0 ? defaults.size : _ref$size,
       subtitle = _ref.subtitle,
       title = _ref.title,
       rest = _objectWithoutProperties(_ref, _excluded);
@@ -105,9 +106,4 @@ NotificationsEmptyState.propTypes = {
    * Empty state title
    */
   title: PropTypes.oneOfType([PropTypes.string, PropTypes.node]).isRequired
-}; // Default values for component props. Default values are not required for
-// props that are required, nor for props where the component can apply
-// 'undefined' values reasonably. Default values should be provided when the
-// component needs to make a choice or assumption when a prop is not supplied.
-
-NotificationsEmptyState.defaultProps = EmptyStateDefaultProps;
+};

package/es/components/EmptyStates/UnauthorizedEmptyState/UnauthorizedEmptyState.js

@@ -23,7 +23,7 @@ import { getDevtoolsProps } from '../../../global/js/utils/devtools';
 import { pkg } from '../../../settings';
 import { EmptyStateContent } from '../EmptyStateContent';
 import { UnauthorizedIllustration } from '../assets/UnauthorizedIllustration';
-import { EmptyStateDefaultProps } from '../EmptyState'; // The block part of our conventional BEM class names (blockClass__E--M).
+import { defaults } from '../EmptyState'; // The block part of our conventional BEM class names (blockClass__E--M).
 
 var blockClass = "".concat(pkg.prefix, "--empty-state");
 var componentName = 'UnauthorizedEmptyState';
@@ -32,7 +32,8 @@ export var UnauthorizedEmptyState = /*#__PURE__*/React.forwardRef(function (_ref
       className = _ref.className,
       illustrationTheme = _ref.illustrationTheme,
       link = _ref.link,
-      size = _ref.size,
+      _ref$size = _ref.size,
+      size = _ref$size === void 0 ? defaults.size : _ref$size,
       subtitle = _ref.subtitle,
       title = _ref.title,
       rest = _objectWithoutProperties(_ref, _excluded);
@@ -105,9 +106,4 @@ UnauthorizedEmptyState.propTypes = {
    * Empty state title
    */
   title: PropTypes.oneOfType([PropTypes.string, PropTypes.node]).isRequired
-}; // Default values for component props. Default values are not required for
-// props that are required, nor for props where the component can apply
-// 'undefined' values reasonably. Default values should be provided when the
-// component needs to make a choice or assumption when a prop is not supplied.
-
-UnauthorizedEmptyState.defaultProps = EmptyStateDefaultProps;
+};

package/es/components/ExampleComponent/ExampleComponent.js

@@ -24,7 +24,13 @@ import { Button, ButtonSet } from 'carbon-components-react'; // The block part o
 
 var blockClass = "".concat(pkg.prefix, "--example-component");
 var componentName = 'ExampleComponent'; // NOTE: the component SCSS is not imported here: it is rolled up separately.
+// Default values for props
 
+var defaults = {
+  primaryKind: 'primary',
+  secondaryKind: 'secondary',
+  size: 'default'
+};
 /**
  * This is an example component to show relevant conventions and usage.
  */
@@ -37,10 +43,13 @@ export var ExampleComponent = /*#__PURE__*/React.forwardRef(function (_ref, ref)
       onPrimaryClick = _ref.onPrimaryClick,
       onSecondaryClick = _ref.onSecondaryClick,
       primaryButtonLabel = _ref.primaryButtonLabel,
-      primaryKind = _ref.primaryKind,
+      _ref$primaryKind = _ref.primaryKind,
+      primaryKind = _ref$primaryKind === void 0 ? defaults.primaryKind : _ref$primaryKind,
       secondaryButtonLabel = _ref.secondaryButtonLabel,
-      secondaryKind = _ref.secondaryKind,
-      size = _ref.size,
+      _ref$secondaryKind = _ref.secondaryKind,
+      secondaryKind = _ref$secondaryKind === void 0 ? defaults.secondaryKind : _ref$secondaryKind,
+      _ref$size = _ref.size,
+      size = _ref$size === void 0 ? defaults.size : _ref$size,
       style = _ref.style,
       rest = _objectWithoutProperties(_ref, _excluded);
 
@@ -147,14 +156,4 @@ ExampleComponent.propTypes = {
    * Optional style settings for the containing node.
    */
   style: PropTypes.object
-}; // Default values for component props. Default values are not required for
-// props that are required, nor for props where the component can apply
-// 'undefined' values reasonably. Default values should be provided when the
-// component needs to make a choice or assumption when a prop is not supplied.
-
-ExampleComponent.defaultProps = {
-  boxedBorder: false,
-  primaryKind: 'primary',
-  secondaryKind: 'secondary',
-  size: 'default'
 };

package/es/components/ExportModal/ExportModal.js

@@ -22,7 +22,13 @@ import PropTypes from 'prop-types';
 import { getDevtoolsProps } from '../../global/js/utils/devtools';
 import uuidv4 from '../../global/js/utils/uuidv4';
 import { pkg } from '../../settings';
-var componentName = 'ExportModal';
+var componentName = 'ExportModal'; // Default values for props
+
+var defaults = {
+  inputType: 'text',
+  preformattedExtensions: Object.freeze([]),
+  validExtensions: Object.freeze([])
+};
 export var ExportModal = /*#__PURE__*/forwardRef(function (_ref, ref) {
   var _preformattedExtensio;
 
@@ -33,14 +39,16 @@ export var ExportModal = /*#__PURE__*/forwardRef(function (_ref, ref) {
       filename = _ref.filename,
       hidePasswordLabel = _ref.hidePasswordLabel,
       inputLabel = _ref.inputLabel,
-      inputType = _ref.inputType,
+      _ref$inputType = _ref.inputType,
+      inputType = _ref$inputType === void 0 ? defaults.inputType : _ref$inputType,
       invalidInputText = _ref.invalidInputText,
       loading = _ref.loading,
       loadingMessage = _ref.loadingMessage,
       onClose = _ref.onClose,
       onRequestSubmit = _ref.onRequestSubmit,
       open = _ref.open,
-      preformattedExtensions = _ref.preformattedExtensions,
+      _ref$preformattedExte = _ref.preformattedExtensions,
+      preformattedExtensions = _ref$preformattedExte === void 0 ? defaults.preformattedExtensions : _ref$preformattedExte,
       preformattedExtensionsLabel = _ref.preformattedExtensionsLabel,
       primaryButtonText = _ref.primaryButtonText,
       secondaryButtonText = _ref.secondaryButtonText,
@@ -48,7 +56,8 @@ export var ExportModal = /*#__PURE__*/forwardRef(function (_ref, ref) {
       successMessage = _ref.successMessage,
       successful = _ref.successful,
       title = _ref.title,
-      validExtensions = _ref.validExtensions,
+      _ref$validExtensions = _ref.validExtensions,
+      validExtensions = _ref$validExtensions === void 0 ? defaults.validExtensions : _ref$validExtensions,
       rest = _objectWithoutProperties(_ref, _excluded);
 
   var _useState = useState(filename),
@@ -297,9 +306,4 @@ ExportModal.propTypes = {
    */
   validExtensions: PropTypes.array
 };
-ExportModal.defaultProps = {
-  inputType: 'text',
-  preformattedExtensions: [],
-  validExtensions: []
-};
 ExportModal.displayName = componentName;

package/es/components/ExpressiveCard/ExpressiveCard.js

@@ -128,12 +128,4 @@ ExpressiveCard.propTypes = {
    */
   title: PropTypes.string
 };
-ExpressiveCard.defaultProps = {
-  actionIcons: [],
-  actionIconsPosition: 'bottom',
-  mediaPosition: 'top',
-  overflowActions: [],
-  primaryButtonKind: 'primary',
-  secondaryButtonKind: 'secondary'
-};
 ExpressiveCard.displayName = componentName;

package/es/components/ImportModal/ImportModal.js

@@ -24,9 +24,14 @@ import PropTypes from 'prop-types';
 import { getDevtoolsProps } from '../../global/js/utils/devtools';
 import uuidv4 from '../../global/js/utils/uuidv4';
 import { pkg } from '../../settings';
-var componentName = 'ImportModal';
+var componentName = 'ImportModal'; // Default values for props
+
+var defaults = {
+  accept: Object.freeze([])
+};
 export var ImportModal = /*#__PURE__*/forwardRef(function (_ref, ref) {
-  var accept = _ref.accept,
+  var _ref$accept = _ref.accept,
+      accept = _ref$accept === void 0 ? defaults.accept : _ref$accept,
       className = _ref.className,
       defaultErrorBody = _ref.defaultErrorBody,
       defaultErrorHeader = _ref.defaultErrorHeader,
@@ -416,7 +421,4 @@ ImportModal.propTypes = {
    */
   title: PropTypes.string.isRequired
 };
-ImportModal.defaultProps = {
-  accept: []
-};
 ImportModal.displayName = componentName;

package/es/components/InlineEdit/InlineEdit.js

@@ -2,7 +2,7 @@ import _extends from "@babel/runtime/helpers/extends";
 import _defineProperty from "@babel/runtime/helpers/defineProperty";
 import _slicedToArray from "@babel/runtime/helpers/slicedToArray";
 import _objectWithoutProperties from "@babel/runtime/helpers/objectWithoutProperties";
-var _excluded = ["cancelDescription", "className", "disabled", "editDescription", "editVisibleOnHoverOnly", "id", "invalid", "invalidText", "labelText", "light", "onCancel", "onSave", "onChange", "placeholder", "saveDescription", "saveDisabled", "size", "value", "warn", "warnText"];
+var _excluded = ["cancelDescription", "className", "disabled", "editAlwaysVisible", "editDescription", "id", "invalid", "invalidText", "labelText", "light", "onCancel", "onSave", "onChange", "placeholder", "saveDescription", "size", "value", "warn", "warnText"];
 
 /**
  * Copyright IBM Corp. 2022, 2022
@@ -26,30 +26,36 @@ import { Close16, Edit16, EditOff16, Checkmark16, WarningFilled16, WarningAltFil
 var blockClass = "".concat(pkg.prefix, "--inline-edit");
 var componentName = 'InlineEdit'; // NOTE: the component SCSS is not imported here: it is rolled up separately.
 
+var defaults = {
+  light: true,
+  // defaults to true to reflect design
+  size: 'md'
+};
 /**
  * TODO: A description of the component.
  */
 
 export var InlineEdit = /*#__PURE__*/React.forwardRef(function (_ref, refIn) {
-  var _cx, _cx3;
+  var _cx, _refInput$current, _refInput$current$inn, _cx3;
 
   var cancelDescription = _ref.cancelDescription,
       className = _ref.className,
       disabled = _ref.disabled,
+      editAlwaysVisible = _ref.editAlwaysVisible,
       editDescription = _ref.editDescription,
-      editVisibleOnHoverOnly = _ref.editVisibleOnHoverOnly,
       id = _ref.id,
       invalid = _ref.invalid,
       invalidText = _ref.invalidText,
       labelText = _ref.labelText,
-      light = _ref.light,
+      _ref$light = _ref.light,
+      light = _ref$light === void 0 ? defaults.light : _ref$light,
       onCancel = _ref.onCancel,
       onSave = _ref.onSave,
       onChange = _ref.onChange,
       placeholder = _ref.placeholder,
       saveDescription = _ref.saveDescription,
-      saveDisabled = _ref.saveDisabled,
-      size = _ref.size,
+      _ref$size = _ref.size,
+      size = _ref$size === void 0 ? defaults.size : _ref$size,
       value = _ref.value,
       warn = _ref.warn,
       warnText = _ref.warnText,
@@ -69,9 +75,9 @@ export var InlineEdit = /*#__PURE__*/React.forwardRef(function (_ref, refIn) {
       internalValue = _useState4[0],
       setInternalValue = _useState4[1];
 
-  var showValidationText = invalid || warn;
+  var showValidation = invalid || warn;
   var validationText = invalidText || warnText;
-  var validationIcon = showValidationText ? invalid ? /*#__PURE__*/React.createElement(WarningFilled16, null) : /*#__PURE__*/React.createElement(WarningAltFilled16, null) : null;
+  var validationIcon = showValidation ? invalid ? /*#__PURE__*/React.createElement(WarningFilled16, null) : /*#__PURE__*/React.createElement(WarningAltFilled16, null) : null;
 
   var doSetEditing = function doSetEditing(value) {
     if (value === false) {
@@ -83,6 +89,7 @@ export var InlineEdit = /*#__PURE__*/React.forwardRef(function (_ref, refIn) {
   };
 
   var handleEdit = function handleEdit(ev) {
+    /* istanbul ignore else */
     if (!disabled) {
       var rightOfInput = ev.currentTarget.classList.contains("".concat(blockClass, "__edit")) || ev.target.classList.contains("".concat(blockClass, "__after-input-elements"));
       var leftOfInput = ev.currentTarget = ev.target.classList.contains("".concat(blockClass)); // clicking on the content editable element should not set either of these to true
@@ -123,7 +130,18 @@ export var InlineEdit = /*#__PURE__*/React.forwardRef(function (_ref, refIn) {
     }
   };
 
-  var handlePaste = function handlePaste(ev) {
+  var handleInput = function handleInput() {
+    setInternalValue(refInput.current.innerText);
+
+    if (onChange) {
+      onChange(refInput.current.innerText);
+    }
+  }; // pasting into contentEditable not supported by userEvent
+
+
+  var handlePaste =
+  /* istanbul ignore next */
+  function handlePaste(ev) {
     ev.preventDefault(); // Get clipboard as plain text
 
     var text = (ev.clipboardData || window.clipboardData).getData('text/plain'); // remove \n
@@ -150,14 +168,6 @@ export var InlineEdit = /*#__PURE__*/React.forwardRef(function (_ref, refIn) {
     }
   };
 
-  var handleInput = function handleInput() {
-    setInternalValue(refInput.current.innerText);
-
-    if (onChange) {
-      onChange(refInput.current.innerText);
-    }
-  };
-
   var handleCancel = function handleCancel() {
     refInput.current.innerText = value;
     handleInput(value);
@@ -171,15 +181,21 @@ export var InlineEdit = /*#__PURE__*/React.forwardRef(function (_ref, refIn) {
 
   var handleBlur = function handleBlur(ev) {
     if (!ref.current.contains(ev.relatedTarget)) {
-      doSetEditing(false);
       handleSave();
     }
   };
 
   var handleKeyDown = function handleKeyDown(ev) {
-    if (ev.key === 'Enter') {
-      ev.preventDefault();
-      refInput.current.blur(); // will cause save
+    switch (ev.key) {
+      case 'Enter':
+        ev.preventDefault();
+        refInput.current.blur(); // will cause save
+
+        break;
+
+      case 'Escape':
+        handleCancel();
+        break;
     }
   };
   /*
@@ -221,7 +237,7 @@ export var InlineEdit = /*#__PURE__*/React.forwardRef(function (_ref, refIn) {
     }, /*#__PURE__*/React.createElement("div", _extends({}, rest, getDevtoolsProps(componentName), {
       id: id,
       size: size,
-      className: cx("".concat(blockClass, "__input"), _defineProperty({}, "".concat(blockClass, "__input--empty"), refInput.current && refInput.current.innerText.length === 0)),
+      className: cx("".concat(blockClass, "__input"), _defineProperty({}, "".concat(blockClass, "__input--empty"), ((_refInput$current = refInput.current) === null || _refInput$current === void 0 ? void 0 : (_refInput$current$inn = _refInput$current.innerText) === null || _refInput$current$inn === void 0 ? void 0 : _refInput$current$inn.length) === 0)),
       contentEditable: true,
       "aria-label": labelText,
       role: "textbox",
@@ -242,7 +258,7 @@ export var InlineEdit = /*#__PURE__*/React.forwardRef(function (_ref, refIn) {
       "aria-hidden": !editing
     }, "\u2026"), /*#__PURE__*/React.createElement("div", {
       className: cx("".concat(blockClass, "__toolbar"), (_cx3 = {}, _defineProperty(_cx3, "".concat(blockClass, "__toolbar--animation"), toolbarAnimation), _defineProperty(_cx3, "".concat(blockClass, "__toolbar--saveable"), refInput.current && value !== internalValue), _cx3))
-    }, showValidationText && validationText.length > 0 && /*#__PURE__*/React.createElement("div", {
+    }, showValidation && /*#__PURE__*/React.createElement("div", {
       className: "".concat(blockClass, "__validation-icon")
     }, validationIcon), editing ? /*#__PURE__*/React.createElement(React.Fragment, null, /*#__PURE__*/React.createElement(Button, {
       className: "".concat(blockClass, "__cancel"),
@@ -258,10 +274,10 @@ export var InlineEdit = /*#__PURE__*/React.forwardRef(function (_ref, refIn) {
       iconDescription: saveDescription,
       onClick: handleSave,
       renderIcon: Checkmark16,
-      disabled: invalid || saveDisabled || value === internalValue
+      disabled: value === internalValue
     })) : /*#__PURE__*/React.createElement(Button, {
       "aria-hidden": "true",
-      className: cx("".concat(blockClass, "__edit"), _defineProperty({}, "".concat(blockClass, "__edit--hover-visible"), editVisibleOnHoverOnly)),
+      className: cx("".concat(blockClass, "__edit"), _defineProperty({}, "".concat(blockClass, "__edit--always-visible"), editAlwaysVisible)),
       kind: "ghost",
       hasIconOnly: true,
       iconDescription: editDescription,
@@ -271,7 +287,7 @@ export var InlineEdit = /*#__PURE__*/React.forwardRef(function (_ref, refIn) {
       tabIndex: -1
     }))), /*#__PURE__*/React.createElement("div", {
       className: cx("".concat(blockClass, "__disabled-cover"))
-    }), showValidationText && validationText.length > 0 && /*#__PURE__*/React.createElement("div", {
+    }), showValidation && validationText && validationText.length > 0 && /*#__PURE__*/React.createElement("div", {
       className: "".concat(blockClass, "__validation-text ").concat(carbon.prefix, "--form-requirement")
     }, validationText))
   );
@@ -301,15 +317,14 @@ InlineEdit.propTypes = {
   disabled: PropTypes.bool,
 
   /**
-   * Label for the edit button
+   * By default the edit icon is shown on hover only.
    */
-  editDescription: PropTypes.string.isRequired,
+  editAlwaysVisible: PropTypes.bool,
 
   /**
-   * In some scenarios the edit icon only needs to be shown on hover. These cases are where continual visibility of
-   * the edit icon is redundant. E.g. a spreadsheet a property panel.
+   * Label for the edit button
    */
-  editVisibleOnHoverOnly: PropTypes.bool,
+  editDescription: PropTypes.string.isRequired,
 
   /**
    * ID for inline edit
@@ -342,12 +357,16 @@ InlineEdit.propTypes = {
   onCancel: PropTypes.func,
 
   /**
-   * method called on input event (it's a React thing onChange behaves like on input)
+   * method called on input event (it's a React thing onChange behaves like on input).
+   *
+   * NOTE: caller to handle invalid/warn states and associated text
    */
   onChange: PropTypes.func,
 
   /**
    * method called on change event
+   *
+   * NOTE: caller to handle invalid/warn states and associated text
    */
   onSave: PropTypes.func,
 
@@ -361,11 +380,6 @@ InlineEdit.propTypes = {
    */
   saveDescription: PropTypes.string.isRequired,
 
-  /**
-   * disabled state of the save button
-   */
-  saveDisabled: PropTypes.bool,
-
   /**
    * vertical size of control
    */
@@ -385,13 +399,4 @@ InlineEdit.propTypes = {
    * text shown when warn true
    */
   warnText: PropTypes.string
-}; // Default values for component props. Default values are not required for
-// props that are required, nor for props where the component can apply
-// 'undefined' values reasonably. Default values should be provided when the
-// component needs to make a choice or assumption when a prop is not supplied.
-
-InlineEdit.defaultProps = {
-  light: true,
-  // defaults to true to reflect design
-  size: 'md'
 };

package/es/components/LoadingBar/LoadingBar.js

@@ -23,7 +23,13 @@ import { pkg
 
 var blockClass = "".concat(pkg.prefix, "--loading-bar");
 var componentName = 'LoadingBar'; // NOTE: the component SCSS is not imported here: it is rolled up separately.
+// Default values for props
 
+var defaults = {
+  active: true,
+  percentage: undefined,
+  ariaLabel: 'Active loading indicator'
+};
 /**
  * The LoadingBar component provides a way to communicate the loading state to users.
  * It is intended to fill the space of where it's used, and should persist until the
@@ -40,11 +46,14 @@ var componentName = 'LoadingBar'; // NOTE: the component SCSS is not imported he
 export var LoadingBar = /*#__PURE__*/React.forwardRef(function (_ref, ref) {
   var _cx2, _cx3;
 
-  var active = _ref.active,
-      ariaLabel = _ref.ariaLabel,
+  var _ref$active = _ref.active,
+      active = _ref$active === void 0 ? defaults.active : _ref$active,
+      _ref$ariaLabel = _ref.ariaLabel,
+      ariaLabel = _ref$ariaLabel === void 0 ? defaults.ariaLabel : _ref$ariaLabel,
       className = _ref.className,
       id = _ref.id,
-      percentage = _ref.percentage,
+      _ref$percentage = _ref.percentage,
+      percentage = _ref$percentage === void 0 ? defaults.percentage : _ref$percentage,
       percentageIndicatorText = _ref.percentageIndicatorText,
       showPercentageIndicator = _ref.showPercentageIndicator,
       small = _ref.small,
@@ -108,7 +117,7 @@ LoadingBar.propTypes = {
   /**
    * Specify whether you want the loading bar indicator to be active or not
    */
-  active: PropTypes.bool.isRequired,
+  active: PropTypes.bool,
 
   /**
    * Specify a ariaLabel that would be used to best describe the active loading state
@@ -144,17 +153,4 @@ LoadingBar.propTypes = {
    * Specify whether you would like the small variant of this component
    */
   small: PropTypes.bool
-}; // Default values for component props. Default values are not required for
-// props that are required, nor for props where the component can apply
-// 'undefined' values reasonably. Default values should be provided when the
-// component needs to make a choice or assumption when a prop is not supplied.
-
-LoadingBar.defaultProps = {
-  /* add defaults for relevant props. */
-  active: true,
-  small: false,
-  percentage: undefined,
-  showPercentageIndicator: false,
-  percentageIndicatorText: undefined,
-  ariaLabel: 'Active loading indicator'
 };

package/es/components/ModifiedTabs/ModifiedTabLabelNew.js

@@ -27,8 +27,4 @@ ModifiedTabLabelNew.propTypes = {
    * Optional onClick handler
    */
   onClick: PropTypes.func
-};
-ModifiedTabLabelNew.defaultProps = {
-  label: '',
-  onClick: undefined
 };

package/es/components/ModifiedTabs/ModifiedTabLabelWithClose.js

@@ -46,9 +46,4 @@ ModifiedTabLabelWithClose.propTypes = {
    * unsavedContent indicates tab contents have not been saved
    */
   unsavedContent: PropTypes.bool
-};
-ModifiedTabLabelWithClose.defaultProps = {
-  unsavedContent: false,
-  label: '',
-  onClose: undefined
 };