style(src/*): fixes lint errors in all files

Author: waynevanson

src/react-sortable.tsx

@@ -37,6 +37,7 @@ const store: Store = { dragging: null };
 export class ReactSortable<T extends ItemInterface> extends Component<
   ReactSortableProps<T>
 > {
+  /* eslint-disable-next-line */
   static defaultProps: Partial<ReactSortableProps<any>> = {
     clone: (item) => item,
   };
@@ -56,7 +57,7 @@ export class ReactSortable<T extends ItemInterface> extends Component<
 
     props.setList(newList, this.sortable, store);
     invariant(
-      //@ts-ignore
+      //@ts-expect-error: Doesn't exist. Will deprecate soon.
       !props.plugins,
       `
 Plugins prop is no longer supported.
@@ -65,13 +66,13 @@ Please read the updated README.md at https://github.com/SortableJS/react-sortabl
       `
     );
   }
-  componentDidMount() {
+  componentDidMount(): void {
     if (this.ref.current === null) return;
     const newOptions = this.makeOptions();
     Sortable.create(this.ref.current, newOptions);
   }
 
-  render() {
+  render(): JSX.Element {
     const { tag, style, className, id } = this.props;
     const classicProps = { style, className, id };
 
@@ -94,17 +95,20 @@ Please read the updated README.md at https://github.com/SortableJS/react-sortabl
       dataIdAttr,
       selectedClass = "sortable-selected",
       chosenClass = "sortable-chosen",
+      /* eslint-disable */
       dragClass = "sortable-drag",
       fallbackClass = "sortable-falback",
       ghostClass = "sortable-ghost",
       swapClass = "sortable-swap-highlight",
+      /* eslint-enable */
       filter = "sortable-filter",
       list,
     } = this.props;
 
     // if no children, don't do anything.
     if (!children || children == null) return null;
     const dataid = dataIdAttr || "data-id";
+    /* eslint-disable-next-line */
     return Children.map(children as ReactElement<any>[], (child, index) => {
       const item = list[index];
       const { className: prevClassName } = child.props;
@@ -137,7 +141,7 @@ Please read the updated README.md at https://github.com/SortableJS/react-sortabl
     if (el === null) return null;
     const key = Object.keys(el).find((k) => k.includes("Sortable"));
     if (!key) return null;
-    //@ts-ignore - I know what I'm doing.
+    //@ts-expect-error: fix me.
     return el[key] as Sortable;
   }
 
@@ -186,36 +190,39 @@ Please read the updated README.md at https://github.com/SortableJS/react-sortabl
   }
 
   /** Prepares a method that will be used in the sortable options to call an `on[Handler]` prop & an `on[Handler]` ReactSortable method.  */
-  prepareOnHandlerPropAndDOM(evtName: HandledMethodNames) {
-    return (evt: SortableEvent) => {
+  prepareOnHandlerPropAndDOM(
+    evtName: HandledMethodNames
+  ): (evt: SortableEvent) => void {
+    return (evt) => {
       // call the component prop
       this.callOnHandlerProp(evt, evtName);
       // calls state change
-      //@ts-ignore - until @types multidrag item is in
+      //@ts-expect-error: until @types multidrag item is in
       this[evtName](evt);
     };
   }
 
   /** Prepares a method that will be used in the sortable options to call an `on[Handler]` prop */
   prepareOnHandlerProp(
     evtName: Exclude<AllMethodsExceptMove, HandledMethodNames>
-  ) {
-    return (evt: SortableEvent) => {
+  ): (evt: SortableEvent) => void {
+    return (evt) => {
       // call the component prop
       this.callOnHandlerProp(evt, evtName);
     };
   }
 
   /** Calls the `props.on[Handler]` function */
-  callOnHandlerProp(evt: SortableEvent, evtName: AllMethodsExceptMove) {
+  callOnHandlerProp(evt: SortableEvent, evtName: AllMethodsExceptMove): void {
     const propEvent = this.props[evtName];
     if (propEvent) propEvent(evt, this.sortable, store);
   }
 
   // SORTABLE DOM HANDLING
 
-  onAdd(evt: MultiDragEvent) {
+  onAdd(evt: MultiDragEvent): void {
     const { list, setList, clone } = this.props;
+    /* eslint-disable-next-line */
     const otherList = [...store.dragging!.props.list];
     const customs = createCustoms(evt, otherList);
     removeNodes(customs);
@@ -226,7 +233,7 @@ Please read the updated README.md at https://github.com/SortableJS/react-sortabl
     setList(newList, this.sortable, store);
   }
 
-  onRemove(evt: MultiDragEvent) {
+  onRemove(evt: MultiDragEvent): void {
     const { list, setList } = this.props;
     const mode = getMode(evt);
     const customs = createCustoms(evt, list);
@@ -248,7 +255,7 @@ Please read the updated README.md at https://github.com/SortableJS/react-sortabl
           }));
           break;
         case "normal":
-          customClones = customs.map((item, index) => ({
+          customClones = customs.map((item) => ({
             ...item,
             element: evt.clone,
           }));
@@ -266,6 +273,7 @@ Please read the updated README.md at https://github.com/SortableJS/react-sortabl
       // replace selected items with cloned items
       customs.forEach((curr) => {
         const index = curr.oldIndex;
+        /* eslint-disable-next-line */
         const newItem = this.props.clone!(curr.item, evt);
         newList.splice(index, 1, newItem);
       });
@@ -276,7 +284,7 @@ Please read the updated README.md at https://github.com/SortableJS/react-sortabl
     setList(newList, this.sortable, store);
   }
 
-  onUpdate(evt: MultiDragEvent) {
+  onUpdate(evt: MultiDragEvent): void {
     const { list, setList } = this.props;
     const customs = createCustoms(evt, list);
     removeNodes(customs);
@@ -285,15 +293,15 @@ Please read the updated README.md at https://github.com/SortableJS/react-sortabl
     return setList(newList, this.sortable, store);
   }
 
-  onStart(evt: SortableEvent) {
+  onStart(): void {
     store.dragging = this;
   }
 
-  onEnd(evt: SortableEvent) {
+  onEnd(): void {
     store.dragging = null;
   }
 
-  onChoose(evt: SortableEvent) {
+  onChoose(evt: SortableEvent): void {
     const { list, setList } = this.props;
     const newList = list.map((item, index) => {
       if (index === evt.oldIndex) {
@@ -307,7 +315,7 @@ Please read the updated README.md at https://github.com/SortableJS/react-sortabl
     setList(newList, this.sortable, store);
   }
 
-  onUnchoose(evt: SortableEvent) {
+  onUnchoose(evt: SortableEvent): void {
     const { list, setList } = this.props;
     const newList = list.map((item, index) => {
       if (index === evt.oldIndex) {
@@ -321,12 +329,12 @@ Please read the updated README.md at https://github.com/SortableJS/react-sortabl
     setList(newList, this.sortable, store);
   }
 
-  onSpill(evt: SortableEvent) {
+  onSpill(evt: SortableEvent): void {
     const { removeOnSpill, revertOnSpill } = this.props;
     if (removeOnSpill && !revertOnSpill) removeNode(evt.item);
   }
 
-  onSelect(evt: MultiDragEvent) {
+  onSelect(evt: MultiDragEvent): void {
     const { list, setList } = this.props;
     const newList = list.map((item) => ({ ...item, selected: false }));
     evt.newIndicies.forEach((curr) => {
@@ -343,7 +351,7 @@ Please read the updated README.md at https://github.com/SortableJS/react-sortabl
     setList(newList, this.sortable, store);
   }
 
-  onDeselect(evt: MultiDragEvent) {
+  onDeselect(evt: MultiDragEvent): void {
     const { list, setList } = this.props;
     const newList = list.map((item) => ({ ...item, selected: false }));
     evt.newIndicies.forEach((curr) => {

src/types.ts

@@ -19,6 +19,7 @@ export interface ItemInterface {
   chosen?: boolean;
   /** When true, it will not be possible to pick this item up in the list. */
   filtered?: boolean;
+  /* eslint-disable-next-line */
   [property: string]: any;
 }
 
@@ -41,6 +42,7 @@ export interface ReactSortableProps<T>
    * @example
    * forwardRef<HTMLElement, YOURPROPS>((props, ref) => <button {...props} ref={ref} />)
    */
+  /* eslint-disable-next-line */
   tag?: ForwardRefExoticComponent<RefAttributes<any>> | keyof ReactHTML;
   /**
    * If this is provided, the function will replace the clone in place.
@@ -63,6 +65,7 @@ export interface ReactSortableProps<T>
  * Mainly used to access `props.list` within another components.
  */
 export interface Store {
+  /* eslint-disable-next-line */
   dragging: null | ReactSortable<any>;
 }
 

src/util.ts

@@ -7,7 +7,7 @@ import { AllMethodNames, ItemInterface, ReactSortableProps } from "./types";
  * Removes the `node` from the DOM
  * @param node
  */
-export function removeNode(node: HTMLElement):void {
+export function removeNode(node: HTMLElement): void {
   if (node.parentElement !== null) node.parentElement.removeChild(node);
 }
 
@@ -21,7 +21,7 @@ export function insertNodeAt(
   parent: HTMLElement,
   newChild: HTMLElement,
   index: number
-):void {
+): void {
   const refChild = parent.children[index] || null;
   parent.insertBefore(newChild, refChild);
 }
@@ -32,16 +32,20 @@ export function insertNodeAt(
 // @todo - do I need parenElement?
 export function handleDOMChanges<T extends ItemInterface>(
   customs: Normalized<T>[]
-):void {
+): void {
   removeNodes(customs);
   insertNodes(customs);
 }
 
-export function removeNodes<T extends ItemInterface>(customs: Normalized<T>[]):void {
+export function removeNodes<T extends ItemInterface>(
+  customs: Normalized<T>[]
+): void {
   customs.forEach((curr) => removeNode(curr.element));
 }
 
-export function insertNodes<T extends ItemInterface>(customs: Normalized<T>[]) :void{
+export function insertNodes<T extends ItemInterface>(
+  customs: Normalized<T>[]
+): void {
   customs.forEach((curr) => {
     insertNodeAt(curr.parentElement, curr.element, curr.oldIndex);
   });
@@ -50,7 +54,7 @@ export function insertNodes<T extends ItemInterface>(customs: Normalized<T>[]) :
 export function createCustoms<T extends ItemInterface>(
   evt: MultiDragEvent,
   list: T[]
-):Normalized<T>[] {
+): Normalized<T>[] {
   const mode = getMode(evt);
   const parentElement = { parentElement: evt.from };
   let custom = [];
@@ -131,7 +135,7 @@ export function handleStateAdd<T extends ItemInterface>(
   return newList;
 }
 
-export function getMode(evt: MultiDragEvent):'multidrag'|"swap"|"normal" {
+export function getMode(evt: MultiDragEvent): "multidrag" | "swap" | "normal" {
   if (evt.oldIndicies && evt.oldIndicies.length > 0) return "multidrag";
   if (evt.swapItem) return "swap";
   return "normal";