@carbon/ibm-products 1.6.1 → 1.9.0

package/lib/components/EditSidePanel/EditSidePanel.js

@@ -41,7 +41,12 @@ function _objectSpread(target) { for (var i = 1; i < arguments.length; i++) { va
 var blockClass = "".concat(_settings.pkg.prefix, "--edit-side-panel");
 var sidePanelBlockClass = "".concat(_settings.pkg.prefix, "--side-panel");
 var componentName = 'EditSidePanel'; // NOTE: the component SCSS is not imported here: it is rolled up separately.
+// Default values for props
 
+var defaults = {
+  placement: 'right',
+  size: 'md'
+};
 /**
  * TODO: A description of the component.
  */
@@ -55,12 +60,14 @@ var EditSidePanel = /*#__PURE__*/_react.default.forwardRef(function (_ref, ref)
       onRequestClose = _ref.onRequestClose,
       onRequestSubmit = _ref.onRequestSubmit,
       open = _ref.open,
-      placement = _ref.placement,
+      _ref$placement = _ref.placement,
+      placement = _ref$placement === void 0 ? defaults.placement : _ref$placement,
       primaryButtonText = _ref.primaryButtonText,
       secondaryButtonText = _ref.secondaryButtonText,
       selectorPrimaryFocus = _ref.selectorPrimaryFocus,
       selectorPageContent = _ref.selectorPageContent,
-      size = _ref.size,
+      _ref$size = _ref.size,
+      size = _ref$size === void 0 ? defaults.size : _ref$size,
       slideIn = _ref.slideIn,
       subtitle = _ref.subtitle,
       title = _ref.title,
@@ -209,12 +216,4 @@ EditSidePanel.propTypes = {
    * The title of the CreateSidePanel is usually the product or service name.
    */
   title: _propTypes.default.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.
-
-EditSidePanel.defaultProps = {
-  placement: 'right',
-  size: 'md'
 };

package/lib/components/EmptyStates/EmptyState.js

@@ -5,7 +5,7 @@ var _interopRequireDefault = require("@babel/runtime/helpers/interopRequireDefau
 Object.defineProperty(exports, "__esModule", {
   value: true
 });
-exports.EmptyStateDefaultProps = exports.EmptyState = void 0;
+exports.defaults = exports.EmptyState = void 0;
 
 var _defineProperty2 = _interopRequireDefault(require("@babel/runtime/helpers/defineProperty"));
 
@@ -37,7 +37,12 @@ function _objectSpread(target) { for (var i = 1; i < arguments.length; i++) { va
 
 // The block part of our conventional BEM class names (blockClass__E--M).
 var blockClass = "".concat(_settings.pkg.prefix, "--empty-state");
-var componentName = 'EmptyState';
+var componentName = 'EmptyState'; // Default values for props
+
+var defaults = {
+  size: 'lg'
+};
+exports.defaults = defaults;
 
 var EmptyState = /*#__PURE__*/_react.default.forwardRef(function (_ref, ref) {
   var action = _ref.action,
@@ -45,7 +50,8 @@ var EmptyState = /*#__PURE__*/_react.default.forwardRef(function (_ref, ref) {
       illustration = _ref.illustration,
       illustrationDescription = _ref.illustrationDescription,
       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 = (0, _objectWithoutProperties2.default)(_ref, _excluded);
@@ -73,10 +79,6 @@ var EmptyState = /*#__PURE__*/_react.default.forwardRef(function (_ref, ref) {
 
 exports.EmptyState = EmptyState;
 exports.EmptyState = EmptyState = _settings.pkg.checkComponentEnabled(EmptyState, componentName);
-var EmptyStateDefaultProps = {
-  size: 'lg'
-};
-exports.EmptyStateDefaultProps = EmptyStateDefaultProps;
 EmptyState.propTypes = {
   /**
    * Empty state action button
@@ -130,5 +132,4 @@ EmptyState.propTypes = {
    */
   title: _propTypes.default.oneOfType([_propTypes.default.string, _propTypes.default.node]).isRequired
 };
-EmptyState.defaultProps = EmptyStateDefaultProps;
 EmptyState.displayName = componentName;

package/lib/components/EmptyStates/ErrorEmptyState/ErrorEmptyState.js

@@ -46,7 +46,8 @@ var ErrorEmptyState = /*#__PURE__*/_react.default.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 ? _EmptyState.defaults.size : _ref$size,
       subtitle = _ref.subtitle,
       title = _ref.title,
       rest = (0, _objectWithoutProperties2.default)(_ref, _excluded);
@@ -120,9 +121,4 @@ ErrorEmptyState.propTypes = {
    * Empty state title
    */
   title: _propTypes.default.oneOfType([_propTypes.default.string, _propTypes.default.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 = _EmptyState.EmptyStateDefaultProps;
+};

package/lib/components/EmptyStates/NoDataEmptyState/NoDataEmptyState.js

@@ -46,7 +46,8 @@ var NoDataEmptyState = /*#__PURE__*/_react.default.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 ? _EmptyState.defaults.size : _ref$size,
       subtitle = _ref.subtitle,
       title = _ref.title,
       rest = (0, _objectWithoutProperties2.default)(_ref, _excluded);
@@ -120,9 +121,4 @@ NoDataEmptyState.propTypes = {
    * Empty state title
    */
   title: _propTypes.default.oneOfType([_propTypes.default.string, _propTypes.default.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 = _EmptyState.EmptyStateDefaultProps;
+};

package/lib/components/EmptyStates/NoTagsEmptyState/NoTagsEmptyState.js

@@ -46,7 +46,8 @@ var NoTagsEmptyState = /*#__PURE__*/_react.default.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 ? _EmptyState.defaults.size : _ref$size,
       subtitle = _ref.subtitle,
       title = _ref.title,
       rest = (0, _objectWithoutProperties2.default)(_ref, _excluded);
@@ -120,9 +121,4 @@ NoTagsEmptyState.propTypes = {
    * Empty state title
    */
   title: _propTypes.default.oneOfType([_propTypes.default.string, _propTypes.default.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 = _EmptyState.EmptyStateDefaultProps;
+};

package/lib/components/EmptyStates/NotFoundEmptyState/NotFoundEmptyState.js

@@ -46,7 +46,8 @@ var NotFoundEmptyState = /*#__PURE__*/_react.default.forwardRef(function (_ref,
       className = _ref.className,
       illustrationTheme = _ref.illustrationTheme,
       link = _ref.link,
-      size = _ref.size,
+      _ref$size = _ref.size,
+      size = _ref$size === void 0 ? _EmptyState.defaults.size : _ref$size,
       subtitle = _ref.subtitle,
       title = _ref.title,
       rest = (0, _objectWithoutProperties2.default)(_ref, _excluded);
@@ -120,9 +121,4 @@ NotFoundEmptyState.propTypes = {
    * Empty state title
    */
   title: _propTypes.default.oneOfType([_propTypes.default.string, _propTypes.default.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 = _EmptyState.EmptyStateDefaultProps;
+};

package/lib/components/EmptyStates/NotificationsEmptyState/NotificationsEmptyState.js

@@ -46,7 +46,8 @@ var NotificationsEmptyState = /*#__PURE__*/_react.default.forwardRef(function (_
       className = _ref.className,
       illustrationTheme = _ref.illustrationTheme,
       link = _ref.link,
-      size = _ref.size,
+      _ref$size = _ref.size,
+      size = _ref$size === void 0 ? _EmptyState.defaults.size : _ref$size,
       subtitle = _ref.subtitle,
       title = _ref.title,
       rest = (0, _objectWithoutProperties2.default)(_ref, _excluded);
@@ -120,9 +121,4 @@ NotificationsEmptyState.propTypes = {
    * Empty state title
    */
   title: _propTypes.default.oneOfType([_propTypes.default.string, _propTypes.default.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 = _EmptyState.EmptyStateDefaultProps;
+};

package/lib/components/EmptyStates/UnauthorizedEmptyState/UnauthorizedEmptyState.js

@@ -46,7 +46,8 @@ var UnauthorizedEmptyState = /*#__PURE__*/_react.default.forwardRef(function (_r
       className = _ref.className,
       illustrationTheme = _ref.illustrationTheme,
       link = _ref.link,
-      size = _ref.size,
+      _ref$size = _ref.size,
+      size = _ref$size === void 0 ? _EmptyState.defaults.size : _ref$size,
       subtitle = _ref.subtitle,
       title = _ref.title,
       rest = (0, _objectWithoutProperties2.default)(_ref, _excluded);
@@ -120,9 +121,4 @@ UnauthorizedEmptyState.propTypes = {
    * Empty state title
    */
   title: _propTypes.default.oneOfType([_propTypes.default.string, _propTypes.default.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 = _EmptyState.EmptyStateDefaultProps;
+};

package/lib/components/ExampleComponent/ExampleComponent.js

@@ -32,7 +32,13 @@ function _objectSpread(target) { for (var i = 1; i < arguments.length; i++) { va
 // The block part of our conventional BEM class names (blockClass__E--M).
 var blockClass = "".concat(_settings.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.
  */
@@ -45,10 +51,13 @@ var ExampleComponent = /*#__PURE__*/_react.default.forwardRef(function (_ref, re
       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 = (0, _objectWithoutProperties2.default)(_ref, _excluded);
   var modeClass = boxedBorder ? "".concat(blockClass, "--boxed-set") : "".concat(blockClass, "--shadow-set");
@@ -156,14 +165,4 @@ ExampleComponent.propTypes = {
    * Optional style settings for the containing node.
    */
   style: _propTypes.default.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/lib/components/ExportModal/ExportModal.js

@@ -43,7 +43,13 @@ function ownKeys(object, enumerableOnly) { var keys = Object.keys(object); if (O
 
 function _objectSpread(target) { for (var i = 1; i < arguments.length; i++) { var source = null != arguments[i] ? arguments[i] : {}; i % 2 ? ownKeys(Object(source), !0).forEach(function (key) { (0, _defineProperty2.default)(target, key, source[key]); }) : Object.getOwnPropertyDescriptors ? Object.defineProperties(target, Object.getOwnPropertyDescriptors(source)) : ownKeys(Object(source)).forEach(function (key) { Object.defineProperty(target, key, Object.getOwnPropertyDescriptor(source, key)); }); } return target; }
 
-var componentName = 'ExportModal';
+var componentName = 'ExportModal'; // Default values for props
+
+var defaults = {
+  inputType: 'text',
+  preformattedExtensions: Object.freeze([]),
+  validExtensions: Object.freeze([])
+};
 var ExportModal = /*#__PURE__*/(0, _react.forwardRef)(function (_ref, ref) {
   var _preformattedExtensio;
 
@@ -54,14 +60,16 @@ var ExportModal = /*#__PURE__*/(0, _react.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,
@@ -69,7 +77,8 @@ var ExportModal = /*#__PURE__*/(0, _react.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 = (0, _objectWithoutProperties2.default)(_ref, _excluded);
 
   var _useState = (0, _react.useState)(filename),
@@ -317,9 +326,4 @@ ExportModal.propTypes = {
    */
   validExtensions: _propTypes.default.array
 };
-ExportModal.defaultProps = {
-  inputType: 'text',
-  preformattedExtensions: [],
-  validExtensions: []
-};
 ExportModal.displayName = componentName;

package/lib/components/ExpressiveCard/ExpressiveCard.js

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

package/lib/components/ImportModal/ImportModal.js

@@ -47,9 +47,14 @@ function ownKeys(object, enumerableOnly) { var keys = Object.keys(object); if (O
 
 function _objectSpread(target) { for (var i = 1; i < arguments.length; i++) { var source = null != arguments[i] ? arguments[i] : {}; i % 2 ? ownKeys(Object(source), !0).forEach(function (key) { (0, _defineProperty2.default)(target, key, source[key]); }) : Object.getOwnPropertyDescriptors ? Object.defineProperties(target, Object.getOwnPropertyDescriptors(source)) : ownKeys(Object(source)).forEach(function (key) { Object.defineProperty(target, key, Object.getOwnPropertyDescriptor(source, key)); }); } return target; }
 
-var componentName = 'ImportModal';
+var componentName = 'ImportModal'; // Default values for props
+
+var defaults = {
+  accept: Object.freeze([])
+};
 var ImportModal = /*#__PURE__*/(0, _react.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,
@@ -438,7 +443,4 @@ ImportModal.propTypes = {
    */
   title: _propTypes.default.string.isRequired
 };
-ImportModal.defaultProps = {
-  accept: []
-};
 ImportModal.displayName = componentName;

package/lib/components/InlineEdit/InlineEdit.js

@@ -31,7 +31,7 @@ var _carbonComponentsReact = require("carbon-components-react");
 
 var _iconsReact = require("@carbon/icons-react");
 
-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"];
 
 function _getRequireWildcardCache(nodeInterop) { if (typeof WeakMap !== "function") return null; var cacheBabelInterop = new WeakMap(); var cacheNodeInterop = new WeakMap(); return (_getRequireWildcardCache = function _getRequireWildcardCache(nodeInterop) { return nodeInterop ? cacheNodeInterop : cacheBabelInterop; })(nodeInterop); }
 
@@ -41,30 +41,36 @@ function _interopRequireWildcard(obj, nodeInterop) { if (!nodeInterop && obj &&
 var blockClass = "".concat(_settings.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.
  */
 
 var InlineEdit = /*#__PURE__*/_react.default.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,
@@ -83,9 +89,9 @@ var InlineEdit = /*#__PURE__*/_react.default.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.default.createElement(_iconsReact.WarningFilled16, null) : /*#__PURE__*/_react.default.createElement(_iconsReact.WarningAltFilled16, null) : null;
+  var validationIcon = showValidation ? invalid ? /*#__PURE__*/_react.default.createElement(_iconsReact.WarningFilled16, null) : /*#__PURE__*/_react.default.createElement(_iconsReact.WarningAltFilled16, null) : null;
 
   var doSetEditing = function doSetEditing(value) {
     if (value === false) {
@@ -97,6 +103,7 @@ var InlineEdit = /*#__PURE__*/_react.default.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
@@ -137,7 +144,18 @@ var InlineEdit = /*#__PURE__*/_react.default.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
@@ -164,14 +182,6 @@ var InlineEdit = /*#__PURE__*/_react.default.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);
@@ -185,15 +195,21 @@ var InlineEdit = /*#__PURE__*/_react.default.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;
     }
   };
   /*
@@ -235,7 +251,7 @@ var InlineEdit = /*#__PURE__*/_react.default.forwardRef(function (_ref, refIn) {
     }, /*#__PURE__*/_react.default.createElement("div", (0, _extends2.default)({}, rest, (0, _devtools.getDevtoolsProps)(componentName), {
       id: id,
       size: size,
-      className: (0, _classnames.default)("".concat(blockClass, "__input"), (0, _defineProperty2.default)({}, "".concat(blockClass, "__input--empty"), refInput.current && refInput.current.innerText.length === 0)),
+      className: (0, _classnames.default)("".concat(blockClass, "__input"), (0, _defineProperty2.default)({}, "".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",
@@ -256,7 +272,7 @@ var InlineEdit = /*#__PURE__*/_react.default.forwardRef(function (_ref, refIn) {
       "aria-hidden": !editing
     }, "\u2026"), /*#__PURE__*/_react.default.createElement("div", {
       className: (0, _classnames.default)("".concat(blockClass, "__toolbar"), (_cx3 = {}, (0, _defineProperty2.default)(_cx3, "".concat(blockClass, "__toolbar--animation"), toolbarAnimation), (0, _defineProperty2.default)(_cx3, "".concat(blockClass, "__toolbar--saveable"), refInput.current && value !== internalValue), _cx3))
-    }, showValidationText && validationText.length > 0 && /*#__PURE__*/_react.default.createElement("div", {
+    }, showValidation && /*#__PURE__*/_react.default.createElement("div", {
       className: "".concat(blockClass, "__validation-icon")
     }, validationIcon), editing ? /*#__PURE__*/_react.default.createElement(_react.default.Fragment, null, /*#__PURE__*/_react.default.createElement(_carbonComponentsReact.Button, {
       className: "".concat(blockClass, "__cancel"),
@@ -272,10 +288,10 @@ var InlineEdit = /*#__PURE__*/_react.default.forwardRef(function (_ref, refIn) {
       iconDescription: saveDescription,
       onClick: handleSave,
       renderIcon: _iconsReact.Checkmark16,
-      disabled: invalid || saveDisabled || value === internalValue
+      disabled: value === internalValue
     })) : /*#__PURE__*/_react.default.createElement(_carbonComponentsReact.Button, {
       "aria-hidden": "true",
-      className: (0, _classnames.default)("".concat(blockClass, "__edit"), (0, _defineProperty2.default)({}, "".concat(blockClass, "__edit--hover-visible"), editVisibleOnHoverOnly)),
+      className: (0, _classnames.default)("".concat(blockClass, "__edit"), (0, _defineProperty2.default)({}, "".concat(blockClass, "__edit--always-visible"), editAlwaysVisible)),
       kind: "ghost",
       hasIconOnly: true,
       iconDescription: editDescription,
@@ -285,7 +301,7 @@ var InlineEdit = /*#__PURE__*/_react.default.forwardRef(function (_ref, refIn) {
       tabIndex: -1
     }))), /*#__PURE__*/_react.default.createElement("div", {
       className: (0, _classnames.default)("".concat(blockClass, "__disabled-cover"))
-    }), showValidationText && validationText.length > 0 && /*#__PURE__*/_react.default.createElement("div", {
+    }), showValidation && validationText && validationText.length > 0 && /*#__PURE__*/_react.default.createElement("div", {
       className: "".concat(blockClass, "__validation-text ").concat(_settings.carbon.prefix, "--form-requirement")
     }, validationText))
   );
@@ -317,15 +333,14 @@ InlineEdit.propTypes = {
   disabled: _propTypes.default.bool,
 
   /**
-   * Label for the edit button
+   * By default the edit icon is shown on hover only.
    */
-  editDescription: _propTypes.default.string.isRequired,
+  editAlwaysVisible: _propTypes.default.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.default.bool,
+  editDescription: _propTypes.default.string.isRequired,
 
   /**
    * ID for inline edit
@@ -358,12 +373,16 @@ InlineEdit.propTypes = {
   onCancel: _propTypes.default.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.default.func,
 
   /**
    * method called on change event
+   *
+   * NOTE: caller to handle invalid/warn states and associated text
    */
   onSave: _propTypes.default.func,
 
@@ -377,11 +396,6 @@ InlineEdit.propTypes = {
    */
   saveDescription: _propTypes.default.string.isRequired,
 
-  /**
-   * disabled state of the save button
-   */
-  saveDisabled: _propTypes.default.bool,
-
   /**
    * vertical size of control
    */
@@ -401,13 +415,4 @@ InlineEdit.propTypes = {
    * text shown when warn true
    */
   warnText: _propTypes.default.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/lib/components/LoadingBar/LoadingBar.js

@@ -35,7 +35,13 @@ function _interopRequireWildcard(obj, nodeInterop) { if (!nodeInterop && obj &&
 // The block part of our conventional BEM class names (blockClass__E--M).
 var blockClass = "".concat(_settings.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
@@ -52,11 +58,14 @@ var componentName = 'LoadingBar'; // NOTE: the component SCSS is not imported he
 var LoadingBar = /*#__PURE__*/_react.default.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,
@@ -122,7 +131,7 @@ LoadingBar.propTypes = {
   /**
    * Specify whether you want the loading bar indicator to be active or not
    */
-  active: _propTypes.default.bool.isRequired,
+  active: _propTypes.default.bool,
 
   /**
    * Specify a ariaLabel that would be used to best describe the active loading state
@@ -158,17 +167,4 @@ LoadingBar.propTypes = {
    * Specify whether you would like the small variant of this component
    */
   small: _propTypes.default.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/lib/components/ModifiedTabs/ModifiedTabLabelNew.js

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