victory-core 37.3.0 → 37.3.2

package/src/victory-container/victory-container.tsx

@@ -155,9 +155,9 @@ export const VictoryContainer = (initialProps: VictoryContainerProps) => {
         ...style,
         width: responsive ? style?.width : dimensions.width,
         height: responsive ? style?.height : dimensions.height,
-        pointerEvents: "none",
-        touchAction: "none",
-        position: "relative",
+        pointerEvents: style?.pointerEvents ?? "none",
+        touchAction: style?.touchAction ?? "none",
+        position: style?.position ?? "relative",
       }}
       data-ouia-component-id={ouiaId}
       data-ouia-component-type={ouiaType}

package/src/victory-primitives/point.tsx

@@ -12,9 +12,9 @@ export interface PointProps extends VictoryCommonPrimitiveProps {
   datum?: any;
   getPath?: (x: number, y: number, size: number) => string;
   pathComponent?: React.ReactElement;
-  // eslint-disable-next-line @typescript-eslint/ban-types
+  // eslint-disable-next-line @typescript-eslint/no-unsafe-function-type
   size?: number | Function;
-  // eslint-disable-next-line @typescript-eslint/ban-types
+  // eslint-disable-next-line @typescript-eslint/no-unsafe-function-type
   symbol?: ScatterSymbolType | Function;
   x?: number;
   y?: number;

package/src/victory-theme/types.ts

@@ -9,6 +9,7 @@ import {
   PaddingOrCallback,
   StringOrNumberOrCallback,
 } from "../types/callbacks";
+import { DomainPropType } from "../types/prop-types";
 
 export type BlockProps = {
   top?: number;
@@ -49,7 +50,7 @@ export interface VictoryAxisCommonProps {
   axisLabelComponent?: React.ReactElement;
   axisValue?: number | string | object | Date;
   dependentAxis?: boolean;
-  disableInlineStyles?: boolean;
+  domain?: DomainPropType;
   gridComponent?: React.ReactElement;
   invertAxis?: boolean;
   style?: {

package/src/victory-transition/victory-transition.tsx

@@ -66,7 +66,7 @@ export class VictoryTransition extends React.Component<
   }
 
   componentDidMount() {
-    this.setState({ nodesShouldLoad: true }); // eslint-disable-line react/no-did-mount-set-state
+    this.setState({ nodesShouldLoad: true });
   }
 
   shouldComponentUpdate(nextProps: VictoryTransitionProps) {

package/src/victory-util/add-events.tsx

@@ -127,7 +127,7 @@ export function addEvents<
   TBase extends WrappedComponentClass<TProps>,
   TProps extends EventMixinCommonProps,
 >(WrappedComponent: TBase, options: MixinOptions = {}) {
-  // eslint-disable-next-line @typescript-eslint/no-empty-interface
+  // eslint-disable-next-line @typescript-eslint/no-empty-object-type
   interface AddEventsMixin extends EventMixinCalculatedValues {}
 
   // @ts-expect-error "TS2545: A mixin class must have a constructor with a single rest parameter of type 'any[]'."

package/src/victory-util/axis.tsx

@@ -313,7 +313,7 @@ export function getTicks(props, scale: D3Scale, filterZero = false) {
  * @param {String} axis: either x or y
  * @returns {Array} returns a domain from tickValues
  */
-// eslint-disable-next-line max-statements
+
 function getDomainFromData(props, axis) {
   const { polar, startAngle = 0, endAngle = 360 } = props;
   const tickArray = getTickArray(props);

package/src/victory-util/common-props.tsx

@@ -57,7 +57,7 @@ export interface VictoryCommonThemeProps {
         x?: ScalePropType | D3Scale;
         y?: ScalePropType | D3Scale;
       };
-  // eslint-disable-next-line @typescript-eslint/ban-types
+  // eslint-disable-next-line @typescript-eslint/no-unsafe-function-type
   sharedEvents?: { events: any[]; getEventState: Function };
   singleQuadrantDomainPadding?: boolean | { x?: boolean; y?: boolean };
   standalone?: boolean;
@@ -72,11 +72,11 @@ export interface VictoryCommonPrimitiveProps {
   className?: string;
   clipPath?: string;
   data?: any;
-  // eslint-disable-next-line @typescript-eslint/ban-types
+  // eslint-disable-next-line @typescript-eslint/no-unsafe-function-type
   desc?: string | Function;
   disableInlineStyles?: boolean;
   events?: object;
-  // eslint-disable-next-line @typescript-eslint/ban-types
+  // eslint-disable-next-line @typescript-eslint/no-unsafe-function-type
   id?: number | string | Function;
   index?: number | string;
   origin?: OriginType;

package/src/victory-util/data.ts

@@ -1,4 +1,3 @@
-/* eslint-disable no-use-before-define */
 import React from "react";
 import {
   uniq,
@@ -141,7 +140,6 @@ export function formatDataFromDomain(dataset, domain, defaultBaseline?) {
   const isOverMaxX = overMax(maxDomainX);
   const isOverMaxY = overMax(maxDomainY);
 
-  // eslint-disable-next-line complexity
   return dataset.map((datum) => {
     let { _x, _y, _y0, _y1 } = datum;
 
@@ -283,7 +281,6 @@ export function formatData(
   const data = preformattedData
     ? dataset
     : dataset.reduce((dataArr, datum, index) => {
-        // eslint-disable-line complexity
         const parsedDatum = parseDatum(datum);
         const fallbackValues = { x: index, y: parsedDatum };
         const processedValues = expectedKeys!.reduce((memo, type) => {

package/src/victory-util/domain.ts

@@ -1,4 +1,3 @@
-/* eslint-disable no-use-before-define */
 import React from "react";
 import { isPlainObject, sortedUniq, isDate } from "lodash";
 import * as Data from "./data";
@@ -67,7 +66,6 @@ function getExtremeFromData(dataset, axis, type = "min") {
   return containsDate ? new Date(result) : result;
 }
 
-// eslint-disable-next-line max-statements
 function padDomain(domain, props, axis) {
   if (!props.domainPadding) {
     return domain;

package/src/victory-util/events.ts

@@ -1,4 +1,5 @@
-/* eslint-disable no-use-before-define */
+import React from "react";
+
 import { isEmpty, pickBy, omitBy, uniq } from "lodash";
 import type { EventMixinCalculatedValues } from "./add-events";
 import { isFunction } from "./helpers";
@@ -72,9 +73,12 @@ export function getEvents(
     const selectedEvents = getSelectedEvents();
     return (
       Array.isArray(selectedEvents) &&
-      selectedEvents.reduce((memo, event) => {
-        return event ? Object.assign(memo, event.eventHandlers) : memo;
-      }, {} as ComponentEvent["eventHandlers"])
+      selectedEvents.reduce(
+        (memo, event) => {
+          return event ? Object.assign(memo, event.eventHandlers) : memo;
+        },
+        {} as ComponentEvent["eventHandlers"],
+      )
     );
   };
 
@@ -84,13 +88,13 @@ export function getEvents(
    */
   const getAllEvents = () => {
     // Mandatory usage: `getEvents.bind(this)`
-    /* eslint-disable no-invalid-this */
+
     if (Array.isArray(this.componentEvents)) {
       return Array.isArray(props.events)
         ? this.componentEvents.concat(...props.events)
         : this.componentEvents;
     }
-    /* eslint-enable no-invalid-this */
+
     return props.events;
   };
 
@@ -127,12 +131,12 @@ export function getScopedEvents(
   }
 
   // Mandatory usage: `getScopedEvents.bind(this)`
-  // eslint-disable-next-line no-invalid-this
+
   const newBaseProps = baseProps || this.baseProps;
   // returns the original base props or base state of a given target element
   const getTargetProps = (identifier, type) => {
     const { childName, target, key } = identifier;
-    // eslint-disable-next-line no-invalid-this
+
     const baseType = type === "props" ? newBaseProps : this.state || {};
     const base =
       childName === undefined || childName === null || !baseType[childName]
@@ -173,7 +177,6 @@ export function getScopedEvents(
 
     // returns the state object with mutated props applied for a single key
     const getMutationObject = (key, childName) => {
-      // eslint-disable-next-line no-invalid-this
       const baseState = this.state || {};
       if (!isFunction(eventReturn.mutation)) {
         return baseState;
@@ -270,11 +273,10 @@ export function getScopedEvents(
   // into a state mutation, and calls setState
   // eslint-disable-next-line max-params
   const onEvent = (evt, childProps, eventKey, eventName) => {
-    // eslint-disable-next-line no-invalid-this
     const eventReturn = events[eventName](evt, childProps, eventKey, this);
     if (!isEmpty(eventReturn)) {
       const callbacks = compileCallbacks(eventReturn);
-      // eslint-disable-next-line no-invalid-this
+
       this.setState(parseEventReturn(eventReturn, eventKey), callbacks);
     }
   };
@@ -319,7 +321,7 @@ export function getEventState(
   childType?: string,
 ) {
   // Mandatory usage: `getEventState.bind(this)`
-  // eslint-disable-next-line no-invalid-this
+
   const state = this.state || {};
   if (!childType) {
     return eventKey === "parent"

package/src/victory-util/helpers.ts

@@ -1,4 +1,3 @@
-/* eslint-disable no-use-before-define */
 import React, { isValidElement } from "react";
 import { defaults, property, pick } from "lodash";
 import { CallbackArgs } from "../types/callbacks";

package/src/victory-util/label-helpers.ts

@@ -1,4 +1,3 @@
-/* eslint-disable no-use-before-define */
 import { VictoryLabelProps } from "../victory-label/victory-label";
 import * as Helpers from "./helpers";
 import { defaults } from "lodash";

package/src/victory-util/merge-refs.ts

@@ -1,3 +1,5 @@
+import React from "react";
+
 import * as Helpers from "./helpers";
 
 type Ref<T> = React.MutableRefObject<T> | React.LegacyRef<T> | undefined | null;

package/src/victory-util/scale.ts

@@ -1,4 +1,3 @@
-/* eslint-disable no-use-before-define */
 import { isPlainObject } from "lodash";
 import * as Helpers from "./helpers";
 import * as Collection from "./collection";

package/src/victory-util/transitions.ts

@@ -156,7 +156,6 @@ function getChildBeforeLoad(animate, child, data, cb): TransitionProps {
   return { animate: newAnimate, data: newData, clipWidth: 0 };
 }
 
-// eslint-disable-next-line max-params
 function getChildOnLoad(animate, data, cb): TransitionProps {
   const newAnimate = Object.assign({}, animate, { onEnd: cb });
   let newData = data;
@@ -174,7 +173,7 @@ function getChildOnLoad(animate, data, cb): TransitionProps {
   return { animate: newAnimate, data: newData };
 }
 
-// eslint-disable-next-line max-params, max-len
+// eslint-disable-next-line max-params
 function getChildPropsOnExit(
   animate,
   child,
@@ -208,7 +207,7 @@ function getChildPropsOnExit(
   return { animate: newAnimate, data: newData };
 }
 
-// eslint-disable-next-line max-params,max-len
+// eslint-disable-next-line max-params
 function getChildPropsBeforeEnter(
   animate,
   child,
@@ -240,7 +239,7 @@ function getChildPropsBeforeEnter(
   return { animate: newAnimate, data: newData };
 }
 
-// eslint-disable-next-line max-params, max-len
+// eslint-disable-next-line max-params
 function getChildPropsOnEnter(
   animate,
   data,
@@ -357,7 +356,6 @@ export function getTransitionPropsFactory(props, state, setState) {
     return {};
   };
 
-  // eslint-disable-next-line max-statements, complexity, max-len
   return function getTransitionProps(child, index): TransitionProps {
     const data = getChildData(child) || [];
     const animate: AnimatePropTypeInterface = defaults(

package/src/victory-util/type-helpers.ts

@@ -30,7 +30,7 @@ export function getValueForAxis<T = unknown>(
   return value;
 }
 
-// eslint-disable-next-line @typescript-eslint/ban-types
+// eslint-disable-next-line @typescript-eslint/no-unsafe-function-type
 export function isFunction<T = Function>(func?: unknown): func is T {
   return typeof func === "function";
 }

package/src/victory-util/wrapper.tsx

@@ -1,4 +1,4 @@
-import { defaults, uniq, groupBy, uniqBy, isPlainObject } from "lodash";
+import { defaults, uniq, groupBy, uniqBy } from "lodash";
 import React from "react";
 import * as Axis from "./axis";
 import * as Style from "./style";
@@ -188,7 +188,6 @@ export function getDefaultDomainPadding(props, axis, childComponents) {
 }
 
 export function getDomainFromChildren(props, axis, childComponents) {
-  // eslint-disable-line max-statements, complexity, max-len
   const children = childComponents
     ? childComponents.slice(0)
     : React.Children.toArray(props.children);