victory-create-container 31.0.0 → 31.2.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "victory-create-container",
-  "version": "31.0.0",
+  "version": "31.2.0",
   "description": "Container Helper for Victory",
   "keywords": [
     "data visualization",
@@ -20,12 +20,12 @@
   "license": "MIT",
   "dependencies": {
     "lodash": "^4.17.5",
-    "victory-brush-container": "^31.0.0",
-    "victory-core": "^31.0.0",
-    "victory-cursor-container": "^31.0.0",
-    "victory-selection-container": "^31.0.0",
-    "victory-voronoi-container": "^31.0.0",
-    "victory-zoom-container": "^31.0.0"
+    "victory-brush-container": "^31.2.0",
+    "victory-core": "^31.2.0",
+    "victory-cursor-container": "^31.2.0",
+    "victory-selection-container": "^31.2.0",
+    "victory-voronoi-container": "^31.2.0",
+    "victory-zoom-container": "^31.2.0"
   },
   "scripts": {
     "version": "nps build-libs && nps build-dists"

package/src/create-container.js

@@ -24,7 +24,8 @@ const combineEventHandlers = (eventHandlersArray) => {
       const existingHandler = finalHandlers[eventName];
       if (existingHandler) {
         // create new handler for event that concats the existing handler's mutations with new ones
-        finalHandlers[eventName] = function combinedHandler(...params) { // named for debug clarity
+        finalHandlers[eventName] = function combinedHandler(...params) {
+          // named for debug clarity
           // sometimes handlers return undefined; use empty array instead, for concat()
           const existingMutations = ensureArray(existingHandler(...params));
           const localMutations = ensureArray(localHandler(...params));
@@ -42,16 +43,16 @@ const combineDefaultEvents = (defaultEvents) => {
   // takes a defaultEvents array and returns one equal or lesser length,
   // by combining any events that have the same target
   const eventsByTarget = groupBy(defaultEvents, "target");
-  const events = toPairs(eventsByTarget).map(
-    ([target, eventsArray]) => {
-      eventsArray = eventsArray.filter(Boolean);
-      return isEmpty(eventsArray) ? null : {
-        target,
-        eventHandlers: combineEventHandlers(eventsArray.map((event) => event.eventHandlers))
-        // note: does not currently handle eventKey or childName
-      };
-    }
-  );
+  const events = toPairs(eventsByTarget).map(([target, eventsArray]) => {
+    eventsArray = eventsArray.filter(Boolean);
+    return isEmpty(eventsArray)
+      ? null
+      : {
+          target,
+          eventHandlers: combineEventHandlers(eventsArray.map((event) => event.eventHandlers))
+          // note: does not currently handle eventKey or childName
+        };
+  });
   return events.filter(Boolean);
 };
 
@@ -73,24 +74,23 @@ const combineContainerMixins = (mixins, Container) => {
   return class VictoryCombinedContainer extends NaiveCombinedContainer {
     static displayName = `Victory${displayType}Container`;
 
-    static propTypes =
-      Classes.reduce(
-        (propTypes, Class) => ({ ...propTypes, ...Class.propTypes }),
-        {}
-      );
+    static propTypes = Classes.reduce(
+      (propTypes, Class) => ({ ...propTypes, ...Class.propTypes }),
+      {}
+    );
 
-    static defaultProps =
-      Classes.reduce(
-        (defaultProps, Class) => ({ ...defaultProps, ...Class.defaultProps }),
-        {}
-      );
+    static defaultProps = Classes.reduce(
+      (defaultProps, Class) => ({ ...defaultProps, ...Class.defaultProps }),
+      {}
+    );
 
     static defaultEvents = (props) => {
       return combineDefaultEvents(
         Classes.reduce((defaultEvents, Class) => {
-          const events = isFunction(Class.defaultEvents) ?
-            Class.defaultEvents(props) : Class.defaultEvents;
-          return ([...defaultEvents, ...events]);
+          const events = isFunction(Class.defaultEvents)
+            ? Class.defaultEvents(props)
+            : Class.defaultEvents;
+          return [...defaultEvents, ...events];
         }, [])
       );
     };
@@ -106,13 +106,12 @@ const combineContainerMixins = (mixins, Container) => {
 
 const checkBehaviorName = (behavior, behaviors) => {
   if (behavior && !includes(behaviors, behavior)) {
-    Log.warn(
-      `"${behavior}" is not a valid behavior. Choose from [${behaviors.join(", ")}].`
-    );
+    Log.warn(`"${behavior}" is not a valid behavior. Choose from [${behaviors.join(", ")}].`);
   }
 };
 
-const makeCreateContainerFunction = (mixinMap, Container) => (behaviorA, behaviorB, ...invalid) => { // eslint-disable-line
+const makeCreateContainerFunction = (mixinMap, Container) => (behaviorA, behaviorB, ...invalid) => {
+  // eslint-disable-line
   const behaviors = Object.keys(mixinMap);
 
   checkBehaviorName(behaviorA, behaviors);
@@ -132,13 +131,15 @@ const makeCreateContainerFunction = (mixinMap, Container) => (behaviorA, behavio
   return combineContainerMixins([...firstMixins, ...secondMixins], Container);
 };
 
-const createContainer = makeCreateContainerFunction({
-  zoom: [zoomContainerMixin],
-  voronoi: [voronoiContainerMixin],
-  selection: [selectionContainerMixin],
-  cursor: [cursorContainerMixin],
-  brush: [brushContainerMixin]
-}, VictoryContainer);
+const createContainer = makeCreateContainerFunction(
+  {
+    zoom: [zoomContainerMixin],
+    voronoi: [voronoiContainerMixin],
+    selection: [selectionContainerMixin],
+    cursor: [cursorContainerMixin],
+    brush: [brushContainerMixin]
+  },
+  VictoryContainer
+);
 
 export { createContainer, makeCreateContainerFunction, combineContainerMixins };
-

package/src/index.js

@@ -1,3 +1,5 @@
 export {
-  combineContainerMixins, makeCreateContainerFunction, createContainer
+  combineContainerMixins,
+  makeCreateContainerFunction,
+  createContainer
 } from "./create-container";