victory-stack 36.7.0 → 36.8.2

package/CHANGELOG.md

@@ -1,5 +1,19 @@
 # victory-stack
 
+## 36.8.2
+
+### Patch Changes
+
+- Assign merged props to a const instead of modifying initialProps ([#2718](https://github.com/FormidableLabs/victory/pull/2718))
+
+## 36.8.1
+
+## 36.8.0
+
+### Patch Changes
+
+- Remove usage of defaultProps from components ([#2679](https://github.com/FormidableLabs/victory/pull/2679))
+
 ## 36.7.0
 
 ## 36.6.12

package/dist/victory-stack.js

@@ -348,21 +348,21 @@ __webpack_require__.r(__webpack_exports__);
 /* harmony import */ var lodash_defaults__WEBPACK_IMPORTED_MODULE_1___default = /*#__PURE__*/__webpack_require__.n(lodash_defaults__WEBPACK_IMPORTED_MODULE_1__);
 /* harmony import */ var lodash_assign__WEBPACK_IMPORTED_MODULE_2__ = __webpack_require__(/*! lodash/assign */ "../../../node_modules/.pnpm/lodash@4.17.21/node_modules/lodash/assign.js");
 /* harmony import */ var lodash_assign__WEBPACK_IMPORTED_MODULE_2___default = /*#__PURE__*/__webpack_require__.n(lodash_assign__WEBPACK_IMPORTED_MODULE_2__);
-/* harmony import */ var prop_types__WEBPACK_IMPORTED_MODULE_13__ = __webpack_require__(/*! prop-types */ "../../../node_modules/.pnpm/prop-types@15.8.1/node_modules/prop-types/index.js");
-/* harmony import */ var prop_types__WEBPACK_IMPORTED_MODULE_13___default = /*#__PURE__*/__webpack_require__.n(prop_types__WEBPACK_IMPORTED_MODULE_13__);
+/* harmony import */ var prop_types__WEBPACK_IMPORTED_MODULE_15__ = __webpack_require__(/*! prop-types */ "../../../node_modules/.pnpm/prop-types@15.8.1/node_modules/prop-types/index.js");
+/* harmony import */ var prop_types__WEBPACK_IMPORTED_MODULE_15___default = /*#__PURE__*/__webpack_require__.n(prop_types__WEBPACK_IMPORTED_MODULE_15__);
 /* harmony import */ var react__WEBPACK_IMPORTED_MODULE_3__ = __webpack_require__(/*! react */ "react");
 /* harmony import */ var react__WEBPACK_IMPORTED_MODULE_3___default = /*#__PURE__*/__webpack_require__.n(react__WEBPACK_IMPORTED_MODULE_3__);
-/* harmony import */ var victory_core__WEBPACK_IMPORTED_MODULE_5__ = __webpack_require__(/*! victory-core */ "../../victory-core/es/victory-util/hooks/use-animation-state.js");
-/* harmony import */ var victory_core__WEBPACK_IMPORTED_MODULE_6__ = __webpack_require__(/*! victory-core */ "../../victory-core/es/victory-util/helpers.js");
-/* harmony import */ var victory_core__WEBPACK_IMPORTED_MODULE_8__ = __webpack_require__(/*! victory-core */ "../../victory-core/es/victory-util/user-props.js");
-/* harmony import */ var victory_core__WEBPACK_IMPORTED_MODULE_9__ = __webpack_require__(/*! victory-core */ "../../victory-core/es/victory-util/wrapper.js");
-/* harmony import */ var victory_core__WEBPACK_IMPORTED_MODULE_10__ = __webpack_require__(/*! victory-core */ "../../victory-core/es/victory-util/hooks/use-previous-props.js");
-/* harmony import */ var victory_core__WEBPACK_IMPORTED_MODULE_12__ = __webpack_require__(/*! victory-core */ "../../victory-core/es/victory-util/common-props.js");
-/* harmony import */ var victory_core__WEBPACK_IMPORTED_MODULE_14__ = __webpack_require__(/*! victory-core */ "../../victory-core/es/victory-util/prop-types.js");
-/* harmony import */ var victory_core__WEBPACK_IMPORTED_MODULE_15__ = __webpack_require__(/*! victory-core */ "../../victory-core/es/victory-container/victory-container.js");
-/* harmony import */ var victory_core__WEBPACK_IMPORTED_MODULE_16__ = __webpack_require__(/*! victory-core */ "../../victory-core/es/victory-theme/victory-theme.js");
-/* harmony import */ var victory_shared_events__WEBPACK_IMPORTED_MODULE_11__ = __webpack_require__(/*! victory-shared-events */ "../../victory-shared-events/es/victory-shared-events.js");
-/* harmony import */ var _helper_methods__WEBPACK_IMPORTED_MODULE_7__ = __webpack_require__(/*! ./helper-methods */ "./helper-methods.tsx");
+/* harmony import */ var victory_core__WEBPACK_IMPORTED_MODULE_5__ = __webpack_require__(/*! victory-core */ "../../victory-core/es/victory-container/victory-container.js");
+/* harmony import */ var victory_core__WEBPACK_IMPORTED_MODULE_6__ = __webpack_require__(/*! victory-core */ "../../victory-core/es/victory-theme/victory-theme.js");
+/* harmony import */ var victory_core__WEBPACK_IMPORTED_MODULE_7__ = __webpack_require__(/*! victory-core */ "../../victory-core/es/victory-util/hooks/use-animation-state.js");
+/* harmony import */ var victory_core__WEBPACK_IMPORTED_MODULE_8__ = __webpack_require__(/*! victory-core */ "../../victory-core/es/victory-util/helpers.js");
+/* harmony import */ var victory_core__WEBPACK_IMPORTED_MODULE_10__ = __webpack_require__(/*! victory-core */ "../../victory-core/es/victory-util/user-props.js");
+/* harmony import */ var victory_core__WEBPACK_IMPORTED_MODULE_11__ = __webpack_require__(/*! victory-core */ "../../victory-core/es/victory-util/wrapper.js");
+/* harmony import */ var victory_core__WEBPACK_IMPORTED_MODULE_12__ = __webpack_require__(/*! victory-core */ "../../victory-core/es/victory-util/hooks/use-previous-props.js");
+/* harmony import */ var victory_core__WEBPACK_IMPORTED_MODULE_14__ = __webpack_require__(/*! victory-core */ "../../victory-core/es/victory-util/common-props.js");
+/* harmony import */ var victory_core__WEBPACK_IMPORTED_MODULE_16__ = __webpack_require__(/*! victory-core */ "../../victory-core/es/victory-util/prop-types.js");
+/* harmony import */ var victory_shared_events__WEBPACK_IMPORTED_MODULE_13__ = __webpack_require__(/*! victory-shared-events */ "../../victory-shared-events/es/victory-shared-events.js");
+/* harmony import */ var _helper_methods__WEBPACK_IMPORTED_MODULE_9__ = __webpack_require__(/*! ./helper-methods */ "./helper-methods.tsx");
 /* harmony import */ var react_fast_compare__WEBPACK_IMPORTED_MODULE_4__ = __webpack_require__(/*! react-fast-compare */ "../../../node_modules/.pnpm/react-fast-compare@3.2.0/node_modules/react-fast-compare/index.js");
 /* harmony import */ var react_fast_compare__WEBPACK_IMPORTED_MODULE_4___default = /*#__PURE__*/__webpack_require__.n(react_fast_compare__WEBPACK_IMPORTED_MODULE_4__);
 
@@ -386,18 +386,27 @@ var fallbackProps = {
   height: 300,
   padding: 50
 };
+var defaultProps = {
+  containerComponent: /*#__PURE__*/react__WEBPACK_IMPORTED_MODULE_3___default().createElement(victory_core__WEBPACK_IMPORTED_MODULE_5__.VictoryContainer, null),
+  groupComponent: /*#__PURE__*/react__WEBPACK_IMPORTED_MODULE_3___default().createElement("g", null),
+  standalone: true,
+  theme: victory_core__WEBPACK_IMPORTED_MODULE_6__.VictoryTheme.grayscale,
+  fillInMissingData: true
+};
 
 var VictoryStackBase = function (initialProps) {
-  // eslint-disable-next-line no-use-before-define
   var role = VictoryStack.role;
+  var propsWithDefaults = react__WEBPACK_IMPORTED_MODULE_3___default().useMemo(function () {
+    return _objectSpread(_objectSpread({}, defaultProps), initialProps);
+  }, [initialProps]);
 
-  var _Hooks$useAnimationSt = victory_core__WEBPACK_IMPORTED_MODULE_5__.useAnimationState(),
+  var _Hooks$useAnimationSt = victory_core__WEBPACK_IMPORTED_MODULE_7__.useAnimationState(),
       setAnimationState = _Hooks$useAnimationSt.setAnimationState,
       getAnimationProps = _Hooks$useAnimationSt.getAnimationProps,
       getProps = _Hooks$useAnimationSt.getProps;
 
-  var props = getProps(initialProps);
-  var modifiedProps = victory_core__WEBPACK_IMPORTED_MODULE_6__.modifyProps(props, fallbackProps, role);
+  var props = getProps(propsWithDefaults);
+  var modifiedProps = victory_core__WEBPACK_IMPORTED_MODULE_8__.modifyProps(props, fallbackProps, role);
   var eventKey = modifiedProps.eventKey,
       containerComponent = modifiedProps.containerComponent,
       standalone = modifiedProps.standalone,
@@ -410,12 +419,12 @@ var VictoryStackBase = function (initialProps) {
       horizontal = modifiedProps.horizontal,
       name = modifiedProps.name;
   var childComponents = react__WEBPACK_IMPORTED_MODULE_3___default().Children.toArray(modifiedProps.children);
-  var calculatedProps = (0,_helper_methods__WEBPACK_IMPORTED_MODULE_7__.useMemoizedProps)(modifiedProps);
+  var calculatedProps = (0,_helper_methods__WEBPACK_IMPORTED_MODULE_9__.useMemoizedProps)(modifiedProps);
   var domain = calculatedProps.domain,
       scale = calculatedProps.scale,
       style = calculatedProps.style;
   var newChildren = react__WEBPACK_IMPORTED_MODULE_3___default().useMemo(function () {
-    var children = (0,_helper_methods__WEBPACK_IMPORTED_MODULE_7__.getChildren)(props, childComponents, calculatedProps);
+    var children = (0,_helper_methods__WEBPACK_IMPORTED_MODULE_9__.getChildren)(props, childComponents, calculatedProps);
     var orderedChildren = children.map(function (child, index) {
       var childProps = lodash_assign__WEBPACK_IMPORTED_MODULE_2___default()({
         animate: getAnimationProps(props, child, index)
@@ -450,8 +459,8 @@ var VictoryStackBase = function (initialProps) {
     return {};
   }, [standalone, domain, scale, width, height, theme, style, horizontal, polar, name]);
   var userProps = react__WEBPACK_IMPORTED_MODULE_3___default().useMemo(function () {
-    return victory_core__WEBPACK_IMPORTED_MODULE_8__.getSafeUserProps(initialProps);
-  }, [initialProps]);
+    return victory_core__WEBPACK_IMPORTED_MODULE_10__.getSafeUserProps(propsWithDefaults);
+  }, [propsWithDefaults]);
   var container = react__WEBPACK_IMPORTED_MODULE_3___default().useMemo(function () {
     if (standalone) {
       var defaultContainerProps = lodash_defaults__WEBPACK_IMPORTED_MODULE_1___default()({}, containerComponent.props, containerProps, userProps);
@@ -462,20 +471,20 @@ var VictoryStackBase = function (initialProps) {
     return /*#__PURE__*/react__WEBPACK_IMPORTED_MODULE_3___default().cloneElement(groupComponent, userProps);
   }, [groupComponent, standalone, containerComponent, containerProps, userProps]);
   var events = react__WEBPACK_IMPORTED_MODULE_3___default().useMemo(function () {
-    return victory_core__WEBPACK_IMPORTED_MODULE_9__.getAllEvents(props);
+    return victory_core__WEBPACK_IMPORTED_MODULE_11__.getAllEvents(props);
   }, [props]);
-  var previousProps = victory_core__WEBPACK_IMPORTED_MODULE_10__.usePreviousProps(initialProps);
+  var previousProps = victory_core__WEBPACK_IMPORTED_MODULE_12__.usePreviousProps(propsWithDefaults);
   react__WEBPACK_IMPORTED_MODULE_3___default().useEffect(function () {
     // This is called before dismount to keep state in sync
     return function () {
-      if (initialProps.animate) {
-        setAnimationState(previousProps, initialProps);
+      if (propsWithDefaults.animate) {
+        setAnimationState(previousProps, propsWithDefaults);
       }
     };
-  }, [setAnimationState, previousProps, initialProps]);
+  }, [setAnimationState, previousProps, propsWithDefaults]);
 
   if (!lodash_isEmpty__WEBPACK_IMPORTED_MODULE_0___default()(events)) {
-    return /*#__PURE__*/react__WEBPACK_IMPORTED_MODULE_3___default().createElement(victory_shared_events__WEBPACK_IMPORTED_MODULE_11__["default"], {
+    return /*#__PURE__*/react__WEBPACK_IMPORTED_MODULE_3___default().createElement(victory_shared_events__WEBPACK_IMPORTED_MODULE_13__["default"], {
       container: container,
       eventKey: eventKey,
       events: events,
@@ -486,36 +495,29 @@ var VictoryStackBase = function (initialProps) {
   return /*#__PURE__*/react__WEBPACK_IMPORTED_MODULE_3___default().cloneElement(container, container.props, newChildren);
 };
 
-VictoryStackBase.propTypes = _objectSpread(_objectSpread({}, victory_core__WEBPACK_IMPORTED_MODULE_12__.CommonProps.baseProps), {}, {
-  bins: prop_types__WEBPACK_IMPORTED_MODULE_13___default().oneOfType([prop_types__WEBPACK_IMPORTED_MODULE_13___default().arrayOf(prop_types__WEBPACK_IMPORTED_MODULE_13___default().oneOfType([victory_core__WEBPACK_IMPORTED_MODULE_14__.nonNegative, prop_types__WEBPACK_IMPORTED_MODULE_13___default().instanceOf(Date)])), victory_core__WEBPACK_IMPORTED_MODULE_14__.nonNegative]),
-  categories: prop_types__WEBPACK_IMPORTED_MODULE_13___default().oneOfType([prop_types__WEBPACK_IMPORTED_MODULE_13___default().arrayOf((prop_types__WEBPACK_IMPORTED_MODULE_13___default().string)), prop_types__WEBPACK_IMPORTED_MODULE_13___default().shape({
-    x: prop_types__WEBPACK_IMPORTED_MODULE_13___default().arrayOf((prop_types__WEBPACK_IMPORTED_MODULE_13___default().string)),
-    y: prop_types__WEBPACK_IMPORTED_MODULE_13___default().arrayOf((prop_types__WEBPACK_IMPORTED_MODULE_13___default().string))
+VictoryStackBase.propTypes = _objectSpread(_objectSpread({}, victory_core__WEBPACK_IMPORTED_MODULE_14__.CommonProps.baseProps), {}, {
+  bins: prop_types__WEBPACK_IMPORTED_MODULE_15___default().oneOfType([prop_types__WEBPACK_IMPORTED_MODULE_15___default().arrayOf(prop_types__WEBPACK_IMPORTED_MODULE_15___default().oneOfType([victory_core__WEBPACK_IMPORTED_MODULE_16__.nonNegative, prop_types__WEBPACK_IMPORTED_MODULE_15___default().instanceOf(Date)])), victory_core__WEBPACK_IMPORTED_MODULE_16__.nonNegative]),
+  categories: prop_types__WEBPACK_IMPORTED_MODULE_15___default().oneOfType([prop_types__WEBPACK_IMPORTED_MODULE_15___default().arrayOf((prop_types__WEBPACK_IMPORTED_MODULE_15___default().string)), prop_types__WEBPACK_IMPORTED_MODULE_15___default().shape({
+    x: prop_types__WEBPACK_IMPORTED_MODULE_15___default().arrayOf((prop_types__WEBPACK_IMPORTED_MODULE_15___default().string)),
+    y: prop_types__WEBPACK_IMPORTED_MODULE_15___default().arrayOf((prop_types__WEBPACK_IMPORTED_MODULE_15___default().string))
   })]),
-  children: prop_types__WEBPACK_IMPORTED_MODULE_13___default().oneOfType([prop_types__WEBPACK_IMPORTED_MODULE_13___default().arrayOf((prop_types__WEBPACK_IMPORTED_MODULE_13___default().node)), (prop_types__WEBPACK_IMPORTED_MODULE_13___default().node)]),
-  colorScale: prop_types__WEBPACK_IMPORTED_MODULE_13___default().oneOfType([prop_types__WEBPACK_IMPORTED_MODULE_13___default().arrayOf((prop_types__WEBPACK_IMPORTED_MODULE_13___default().string)), prop_types__WEBPACK_IMPORTED_MODULE_13___default().oneOf(["grayscale", "qualitative", "heatmap", "warm", "cool", "red", "green", "blue"])]),
-  fillInMissingData: (prop_types__WEBPACK_IMPORTED_MODULE_13___default().bool),
-  horizontal: (prop_types__WEBPACK_IMPORTED_MODULE_13___default().bool),
-  labelComponent: (prop_types__WEBPACK_IMPORTED_MODULE_13___default().element),
-  labels: prop_types__WEBPACK_IMPORTED_MODULE_13___default().oneOfType([(prop_types__WEBPACK_IMPORTED_MODULE_13___default().func), (prop_types__WEBPACK_IMPORTED_MODULE_13___default().array)]),
-  style: prop_types__WEBPACK_IMPORTED_MODULE_13___default().shape({
-    parent: (prop_types__WEBPACK_IMPORTED_MODULE_13___default().object),
-    data: (prop_types__WEBPACK_IMPORTED_MODULE_13___default().object),
-    labels: (prop_types__WEBPACK_IMPORTED_MODULE_13___default().object)
+  children: prop_types__WEBPACK_IMPORTED_MODULE_15___default().oneOfType([prop_types__WEBPACK_IMPORTED_MODULE_15___default().arrayOf((prop_types__WEBPACK_IMPORTED_MODULE_15___default().node)), (prop_types__WEBPACK_IMPORTED_MODULE_15___default().node)]),
+  colorScale: prop_types__WEBPACK_IMPORTED_MODULE_15___default().oneOfType([prop_types__WEBPACK_IMPORTED_MODULE_15___default().arrayOf((prop_types__WEBPACK_IMPORTED_MODULE_15___default().string)), prop_types__WEBPACK_IMPORTED_MODULE_15___default().oneOf(["grayscale", "qualitative", "heatmap", "warm", "cool", "red", "green", "blue"])]),
+  fillInMissingData: (prop_types__WEBPACK_IMPORTED_MODULE_15___default().bool),
+  horizontal: (prop_types__WEBPACK_IMPORTED_MODULE_15___default().bool),
+  labelComponent: (prop_types__WEBPACK_IMPORTED_MODULE_15___default().element),
+  labels: prop_types__WEBPACK_IMPORTED_MODULE_15___default().oneOfType([(prop_types__WEBPACK_IMPORTED_MODULE_15___default().func), (prop_types__WEBPACK_IMPORTED_MODULE_15___default().array)]),
+  style: prop_types__WEBPACK_IMPORTED_MODULE_15___default().shape({
+    parent: (prop_types__WEBPACK_IMPORTED_MODULE_15___default().object),
+    data: (prop_types__WEBPACK_IMPORTED_MODULE_15___default().object),
+    labels: (prop_types__WEBPACK_IMPORTED_MODULE_15___default().object)
   }),
-  xOffset: (prop_types__WEBPACK_IMPORTED_MODULE_13___default().number)
+  xOffset: (prop_types__WEBPACK_IMPORTED_MODULE_15___default().number)
 });
-VictoryStackBase.defaultProps = {
-  containerComponent: /*#__PURE__*/react__WEBPACK_IMPORTED_MODULE_3___default().createElement(victory_core__WEBPACK_IMPORTED_MODULE_15__.VictoryContainer, null),
-  groupComponent: /*#__PURE__*/react__WEBPACK_IMPORTED_MODULE_3___default().createElement("g", null),
-  standalone: true,
-  theme: victory_core__WEBPACK_IMPORTED_MODULE_16__.VictoryTheme.grayscale,
-  fillInMissingData: true
-};
 var componentConfig = {
   role: "stack",
   expectedComponents: ["groupComponent", "containerComponent", "labelComponent"],
-  getChildren: _helper_methods__WEBPACK_IMPORTED_MODULE_7__.getChildren
+  getChildren: _helper_methods__WEBPACK_IMPORTED_MODULE_9__.getChildren
 };
 var VictoryStack = Object.assign( /*#__PURE__*/react__WEBPACK_IMPORTED_MODULE_3___default().memo(VictoryStackBase, (react_fast_compare__WEBPACK_IMPORTED_MODULE_4___default())), componentConfig);
 VictoryStack.displayName = "VictoryStack";
@@ -11205,10 +11207,11 @@ function reduceChildren(children, iteratee) {
     }, initialMemo);
   };
 
-  var childNames = children.map(function (c, i) {
+  var validChildren = children.filter(react__WEBPACK_IMPORTED_MODULE_6__.isValidElement);
+  var childNames = validChildren.map(function (c, i) {
     return i;
   });
-  return traverseChildren(children, childNames);
+  return traverseChildren(validChildren, childNames);
 }
 /**
  * @param {Object} props: the props object
@@ -12592,6 +12595,7 @@ __webpack_require__.r(__webpack_exports__);
 /* harmony export */ });
 /* harmony import */ var react__WEBPACK_IMPORTED_MODULE_0__ = __webpack_require__(/*! react */ "react");
 /* harmony import */ var react__WEBPACK_IMPORTED_MODULE_0___default = /*#__PURE__*/__webpack_require__.n(react__WEBPACK_IMPORTED_MODULE_0__);
+/* harmony import */ var _helpers__WEBPACK_IMPORTED_MODULE_1__ = __webpack_require__(/*! ./helpers */ "../../victory-core/es/victory-util/helpers.js");
 function _slicedToArray(arr, i) { return _arrayWithHoles(arr) || _iterableToArrayLimit(arr, i) || _unsupportedIterableToArray(arr, i) || _nonIterableRest(); }
 
 function _nonIterableRest() { throw new TypeError("Invalid attempt to destructure non-iterable instance.\nIn order to be iterable, non-array objects must have a [Symbol.iterator]() method."); }
@@ -12611,6 +12615,7 @@ function _objectSpread(target) { for (var i = 1; i < arguments.length; i++) { va
 function _defineProperty(obj, key, value) { if (key in obj) { Object.defineProperty(obj, key, { value: value, enumerable: true, configurable: true, writable: true }); } else { obj[key] = value; } return obj; }
 
 
+
 /*
   USER_PROPS_SAFELIST is to contain any string deemed safe for user props.
   The startsWidth array will contain the start of any accepted user-prop that
@@ -12665,21 +12670,6 @@ var testIfSafeProp = function (key) {
   if (doesPropStartWith(key) || isExactMatch(key)) return true;
   return false;
 };
-/**
- * Gets the value from props if a function value is provided
- * @param {any} value: maybe function value
- * @param {Object} props: props object
- * @returns {any} newValue
- */
-
-
-var getValue = function (value, props) {
-  if (typeof value === "function") {
-    return value(props);
-  }
-
-  return value;
-};
 /**
  * getSafeUserProps - function that takes in a props object and removes any
  * key-value entries that do not match filter strings in the USER_PROPS_SAFELIST
@@ -12703,7 +12693,7 @@ var getSafeUserProps = function (props) {
         key = _ref4[0],
         value = _ref4[1];
 
-    return [key, getValue(value, props)];
+    return [key, (0,_helpers__WEBPACK_IMPORTED_MODULE_1__.evaluateProp)(value, props)];
   }));
 };
 /**