TS Strict fixes

Author: snowystinger

packages/@react-aria/form/src/useFormValidation.ts

@@ -68,8 +68,8 @@ export function useFormValidation<T>(props: Validation<T>, state: FormValidation
     input.addEventListener('change', onChange);
     form?.addEventListener('reset', onReset);
     return () => {
-      input.removeEventListener('invalid', onInvalid);
-      input.removeEventListener('change', onChange);
+      input!.removeEventListener('invalid', onInvalid);
+      input!.removeEventListener('change', onChange);
       form?.removeEventListener('reset', onReset);
     };
   }, [ref, onInvalid, onChange, onReset, validationBehavior]);

packages/@react-aria/interactions/src/useInteractOutside.ts

@@ -37,7 +37,7 @@ export function useInteractOutside(props: InteractOutsideProps) {
     ignoreEmulatedMouseEvents: false
   });
 
-  let onPointerDown = useEffectEvent((e: PointerEvent) => {
+  let onPointerDown = useEffectEvent((e) => {
     if (onInteractOutside && isValidEvent(e, ref)) {
       if (onInteractOutsideStart) {
         onInteractOutsideStart(e);

packages/@react-aria/interactions/src/usePress.ts

@@ -157,7 +157,7 @@ export function usePress(props: PressHookProps): PressResult {
   let triggerPressStart = useEffectEvent((originalEvent: EventBase, pointerType: PointerType) => {
     let state = ref.current;
     if (isDisabled || state.didFirePressStart) {
-      return;
+      return false;
     }
 
     let shouldStopPropagation = true;
@@ -181,7 +181,7 @@ export function usePress(props: PressHookProps): PressResult {
   let triggerPressEnd = useEffectEvent((originalEvent: EventBase, pointerType: PointerType, wasPressed = true) => {
     let state = ref.current;
     if (!state.didFirePressStart) {
-      return;
+      return false;
     }
 
     state.ignoreClickAfterPress = true;
@@ -214,7 +214,7 @@ export function usePress(props: PressHookProps): PressResult {
   let triggerPressUp = useEffectEvent((originalEvent: EventBase, pointerType: PointerType) => {
     let state = ref.current;
     if (isDisabled) {
-      return;
+      return false;
     }
 
     if (onPressUp) {
@@ -231,7 +231,7 @@ export function usePress(props: PressHookProps): PressResult {
   let cancel = useEffectEvent((e: EventBase) => {
     let state = ref.current;
     if (state.isPressed && state.target) {
-      if (state.isOverTarget) {
+      if (state.isOverTarget && state.pointerType) {
         triggerPressEnd(createEvent(state.target, e), state.pointerType, false);
       }
       state.isPressed = false;
@@ -460,11 +460,11 @@ export function usePress(props: PressHookProps): PressResult {
         }
 
         if (state.target && isOverTarget(e, state.target)) {
-          if (!state.isOverTarget) {
+          if (!state.isOverTarget && state.pointerType) {
             state.isOverTarget = true;
             triggerPressStart(createEvent(state.target, e), state.pointerType);
           }
-        } else if (state.target && state.isOverTarget) {
+        } else if (state.target && state.isOverTarget && state.pointerType) {
           state.isOverTarget = false;
           triggerPressEnd(createEvent(state.target, e), state.pointerType, false);
           cancelOnPointerExit(e);
@@ -473,9 +473,9 @@ export function usePress(props: PressHookProps): PressResult {
 
       let onPointerUp = (e: PointerEvent) => {
         if (e.pointerId === state.activePointerId && state.isPressed && e.button === 0 && state.target) {
-          if (isOverTarget(e, state.target)) {
+          if (isOverTarget(e, state.target) && state.pointerType) {
             triggerPressEnd(createEvent(state.target, e), state.pointerType);
-          } else if (state.isOverTarget) {
+          } else if (state.isOverTarget && state.pointerType) {
             triggerPressEnd(createEvent(state.target, e), state.pointerType, false);
           }
 
@@ -543,7 +543,7 @@ export function usePress(props: PressHookProps): PressResult {
         }
 
         let shouldStopPropagation = true;
-        if (state.isPressed && !state.ignoreEmulatedMouseEvents) {
+        if (state.isPressed && !state.ignoreEmulatedMouseEvents && state.pointerType) {
           state.isOverTarget = true;
           shouldStopPropagation = triggerPressStart(e, state.pointerType);
         }
@@ -559,7 +559,7 @@ export function usePress(props: PressHookProps): PressResult {
         }
 
         let shouldStopPropagation = true;
-        if (state.isPressed && !state.ignoreEmulatedMouseEvents) {
+        if (state.isPressed && !state.ignoreEmulatedMouseEvents && state.pointerType) {
           state.isOverTarget = false;
           shouldStopPropagation = triggerPressEnd(e, state.pointerType, false);
           cancelOnPointerExit(e);
@@ -594,9 +594,9 @@ export function usePress(props: PressHookProps): PressResult {
           return;
         }
 
-        if (state.target && isOverTarget(e, state.target)) {
+        if (state.target && isOverTarget(e, state.target) && state.pointerType) {
           triggerPressEnd(createEvent(state.target, e), state.pointerType);
-        } else if (state.target && state.isOverTarget) {
+        } else if (state.target && state.isOverTarget && state.pointerType) {
           triggerPressEnd(createEvent(state.target, e), state.pointerType, false);
         }
 
@@ -650,11 +650,11 @@ export function usePress(props: PressHookProps): PressResult {
         let touch = getTouchById(e.nativeEvent, state.activePointerId);
         let shouldStopPropagation = true;
         if (touch && isOverTarget(touch, e.currentTarget)) {
-          if (!state.isOverTarget) {
+          if (!state.isOverTarget && state.pointerType) {
             state.isOverTarget = true;
             shouldStopPropagation = triggerPressStart(e, state.pointerType);
           }
-        } else if (state.isOverTarget) {
+        } else if (state.isOverTarget && state.pointerType) {
           state.isOverTarget = false;
           shouldStopPropagation = triggerPressEnd(e, state.pointerType, false);
           cancelOnPointerExit(e);
@@ -677,10 +677,10 @@ export function usePress(props: PressHookProps): PressResult {
 
         let touch = getTouchById(e.nativeEvent, state.activePointerId);
         let shouldStopPropagation = true;
-        if (touch && isOverTarget(touch, e.currentTarget)) {
+        if (touch && isOverTarget(touch, e.currentTarget) && state.pointerType) {
           triggerPressUp(e, state.pointerType);
           shouldStopPropagation = triggerPressEnd(e, state.pointerType);
-        } else if (state.isOverTarget) {
+        } else if (state.isOverTarget && state.pointerType) {
           shouldStopPropagation = triggerPressEnd(e, state.pointerType, false);
         }