victory-stack 36.7.0 → 36.8.2

package/es/victory-stack.d.ts

@@ -70,13 +70,6 @@ export declare const VictoryStack: React.NamedExoticComponent<VictoryStackProps>
             standalone?: React.Validator<boolean | null | undefined> | undefined;
             width?: React.Validator<number | null | undefined> | undefined;
         };
-        defaultProps: {
-            containerComponent: JSX.Element;
-            groupComponent: JSX.Element;
-            standalone: boolean;
-            theme: import("victory-core").VictoryThemeDefinition;
-            fillInMissingData: boolean;
-        };
     };
 } & VictoryComponentConfiguration<VictoryStackProps>;
 //# sourceMappingURL=victory-stack.d.ts.map

package/es/victory-stack.js

@@ -19,17 +19,26 @@ var fallbackProps = {
   height: 300,
   padding: 50
 };
+var defaultProps = {
+  containerComponent: /*#__PURE__*/React.createElement(VictoryContainer, null),
+  groupComponent: /*#__PURE__*/React.createElement("g", null),
+  standalone: true,
+  theme: VictoryTheme.grayscale,
+  fillInMissingData: true
+};
 
 var VictoryStackBase = function (initialProps) {
-  // eslint-disable-next-line no-use-before-define
   var role = VictoryStack.role;
+  var propsWithDefaults = React.useMemo(function () {
+    return _objectSpread(_objectSpread({}, defaultProps), initialProps);
+  }, [initialProps]);
 
   var _Hooks$useAnimationSt = Hooks.useAnimationState(),
       setAnimationState = _Hooks$useAnimationSt.setAnimationState,
       getAnimationProps = _Hooks$useAnimationSt.getAnimationProps,
       getProps = _Hooks$useAnimationSt.getProps;
 
-  var props = getProps(initialProps);
+  var props = getProps(propsWithDefaults);
   var modifiedProps = Helpers.modifyProps(props, fallbackProps, role);
   var eventKey = modifiedProps.eventKey,
       containerComponent = modifiedProps.containerComponent,
@@ -83,8 +92,8 @@ var VictoryStackBase = function (initialProps) {
     return {};
   }, [standalone, domain, scale, width, height, theme, style, horizontal, polar, name]);
   var userProps = React.useMemo(function () {
-    return UserProps.getSafeUserProps(initialProps);
-  }, [initialProps]);
+    return UserProps.getSafeUserProps(propsWithDefaults);
+  }, [propsWithDefaults]);
   var container = React.useMemo(function () {
     if (standalone) {
       var defaultContainerProps = _defaults({}, containerComponent.props, containerProps, userProps);
@@ -97,15 +106,15 @@ var VictoryStackBase = function (initialProps) {
   var events = React.useMemo(function () {
     return Wrapper.getAllEvents(props);
   }, [props]);
-  var previousProps = Hooks.usePreviousProps(initialProps);
+  var previousProps = Hooks.usePreviousProps(propsWithDefaults);
   React.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 (!_isEmpty(events)) {
     return /*#__PURE__*/React.createElement(VictorySharedEvents, {
@@ -138,13 +147,6 @@ VictoryStackBase.propTypes = _objectSpread(_objectSpread({}, CommonProps.basePro
   }),
   xOffset: PropTypes.number
 });
-VictoryStackBase.defaultProps = {
-  containerComponent: /*#__PURE__*/React.createElement(VictoryContainer, null),
-  groupComponent: /*#__PURE__*/React.createElement("g", null),
-  standalone: true,
-  theme: VictoryTheme.grayscale,
-  fillInMissingData: true
-};
 var componentConfig = {
   role: "stack",
   expectedComponents: ["groupComponent", "containerComponent", "labelComponent"],

package/lib/victory-stack.d.ts

@@ -70,13 +70,6 @@ export declare const VictoryStack: React.NamedExoticComponent<VictoryStackProps>
             standalone?: React.Validator<boolean | null | undefined> | undefined;
             width?: React.Validator<number | null | undefined> | undefined;
         };
-        defaultProps: {
-            containerComponent: JSX.Element;
-            groupComponent: JSX.Element;
-            standalone: boolean;
-            theme: import("victory-core").VictoryThemeDefinition;
-            fillInMissingData: boolean;
-        };
     };
 } & VictoryComponentConfiguration<VictoryStackProps>;
 //# sourceMappingURL=victory-stack.d.ts.map

package/lib/victory-stack.js

@@ -36,17 +36,27 @@ var fallbackProps = {
   height: 300,
   padding: 50
 };
+var defaultProps = {
+  containerComponent: /*#__PURE__*/_react.default.createElement(_victoryCore.VictoryContainer, null),
+  groupComponent: /*#__PURE__*/_react.default.createElement("g", null),
+  standalone: true,
+  theme: _victoryCore.VictoryTheme.grayscale,
+  fillInMissingData: true
+};
 
 var VictoryStackBase = function (initialProps) {
-  // eslint-disable-next-line no-use-before-define
   var role = VictoryStack.role;
 
+  var propsWithDefaults = _react.default.useMemo(function () {
+    return _objectSpread(_objectSpread({}, defaultProps), initialProps);
+  }, [initialProps]);
+
   var _Hooks$useAnimationSt = _victoryCore.Hooks.useAnimationState(),
       setAnimationState = _Hooks$useAnimationSt.setAnimationState,
       getAnimationProps = _Hooks$useAnimationSt.getAnimationProps,
       getProps = _Hooks$useAnimationSt.getProps;
 
-  var props = getProps(initialProps);
+  var props = getProps(propsWithDefaults);
 
   var modifiedProps = _victoryCore.Helpers.modifyProps(props, fallbackProps, role);
 
@@ -106,8 +116,8 @@ var VictoryStackBase = function (initialProps) {
   }, [standalone, domain, scale, width, height, theme, style, horizontal, polar, name]);
 
   var userProps = _react.default.useMemo(function () {
-    return _victoryCore.UserProps.getSafeUserProps(initialProps);
-  }, [initialProps]);
+    return _victoryCore.UserProps.getSafeUserProps(propsWithDefaults);
+  }, [propsWithDefaults]);
 
   var container = _react.default.useMemo(function () {
     if (standalone) {
@@ -122,16 +132,16 @@ var VictoryStackBase = function (initialProps) {
     return _victoryCore.Wrapper.getAllEvents(props);
   }, [props]);
 
-  var previousProps = _victoryCore.Hooks.usePreviousProps(initialProps);
+  var previousProps = _victoryCore.Hooks.usePreviousProps(propsWithDefaults);
 
   _react.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 (!(0, _isEmpty2.default)(events)) {
     return /*#__PURE__*/_react.default.createElement(_victorySharedEvents.VictorySharedEvents, {
@@ -164,13 +174,6 @@ VictoryStackBase.propTypes = _objectSpread(_objectSpread({}, _victoryCore.Common
   }),
   xOffset: _propTypes.default.number
 });
-VictoryStackBase.defaultProps = {
-  containerComponent: /*#__PURE__*/_react.default.createElement(_victoryCore.VictoryContainer, null),
-  groupComponent: /*#__PURE__*/_react.default.createElement("g", null),
-  standalone: true,
-  theme: _victoryCore.VictoryTheme.grayscale,
-  fillInMissingData: true
-};
 var componentConfig = {
   role: "stack",
   expectedComponents: ["groupComponent", "containerComponent", "labelComponent"],

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "victory-stack",
-  "version": "36.7.0",
+  "version": "36.8.2",
   "description": "Stack Layout Component for Victory",
   "keywords": [
     "data visualization",
@@ -23,8 +23,8 @@
     "lodash": "^4.17.19",
     "prop-types": "^15.8.1",
     "react-fast-compare": "^3.2.0",
-    "victory-core": "^36.7.0",
-    "victory-shared-events": "^36.7.0"
+    "victory-core": "^36.8.2",
+    "victory-shared-events": "^36.8.2"
   },
   "peerDependencies": {
     "react": ">=16.6.0"

package/src/victory-stack.tsx

@@ -52,13 +52,24 @@ export interface VictoryStackProps
   xOffset?: number;
 }
 
+const defaultProps = {
+  containerComponent: <VictoryContainer />,
+  groupComponent: <g />,
+  standalone: true,
+  theme: VictoryTheme.grayscale,
+  fillInMissingData: true,
+};
+
 const VictoryStackBase = (initialProps: VictoryStackProps) => {
-  // eslint-disable-next-line no-use-before-define
   const { role } = VictoryStack;
+  const propsWithDefaults = React.useMemo(
+    () => ({ ...defaultProps, ...initialProps }),
+    [initialProps],
+  );
   const { setAnimationState, getAnimationProps, getProps } =
     Hooks.useAnimationState();
 
-  const props = getProps(initialProps);
+  const props = getProps(propsWithDefaults);
 
   const modifiedProps = Helpers.modifyProps(props, fallbackProps, role);
   const {
@@ -125,8 +136,8 @@ const VictoryStackBase = (initialProps: VictoryStackProps) => {
     name,
   ]);
   const userProps = React.useMemo(
-    () => UserProps.getSafeUserProps(initialProps),
-    [initialProps],
+    () => UserProps.getSafeUserProps(propsWithDefaults),
+    [propsWithDefaults],
   );
 
   const container = React.useMemo(() => {
@@ -152,16 +163,16 @@ const VictoryStackBase = (initialProps: VictoryStackProps) => {
     return Wrapper.getAllEvents(props);
   }, [props]);
 
-  const previousProps = Hooks.usePreviousProps(initialProps);
+  const previousProps = Hooks.usePreviousProps(propsWithDefaults);
 
   React.useEffect(() => {
     // This is called before dismount to keep state in sync
     return () => {
-      if (initialProps.animate) {
-        setAnimationState(previousProps, initialProps);
+      if (propsWithDefaults.animate) {
+        setAnimationState(previousProps, propsWithDefaults);
       }
     };
-  }, [setAnimationState, previousProps, initialProps]);
+  }, [setAnimationState, previousProps, propsWithDefaults]);
 
   if (!isEmpty(events)) {
     return (
@@ -226,14 +237,6 @@ VictoryStackBase.propTypes = {
   xOffset: PropTypes.number,
 };
 
-VictoryStackBase.defaultProps = {
-  containerComponent: <VictoryContainer />,
-  groupComponent: <g />,
-  standalone: true,
-  theme: VictoryTheme.grayscale,
-  fillInMissingData: true,
-};
-
 const componentConfig: VictoryComponentConfiguration<VictoryStackProps> = {
   role: "stack",
   expectedComponents: [