react-native-gesture-handler 2.22.1 → 2.23.1

package/src/web/tools/GestureHandlerWebDelegate.ts

@@ -22,7 +22,8 @@ export class GestureHandlerWebDelegate
   implements GestureHandlerDelegate<HTMLElement, IGestureHandler>
 {
   private isInitialized = false;
-  private view!: HTMLElement;
+  private _view!: HTMLElement;
+
   private gestureHandler!: IGestureHandler;
   private eventManagers: EventManager<unknown>[] = [];
   private defaultViewStyles: DefaultViewStyles = {
@@ -30,14 +31,10 @@ export class GestureHandlerWebDelegate
     touchAction: '',
   };
 
-  getView(): HTMLElement {
-    return this.view;
-  }
-
   init(viewRef: number, handler: IGestureHandler): void {
     if (!viewRef) {
       throw new Error(
-        `Cannot find HTML Element for handler ${handler.getTag()}`
+        `Cannot find HTML Element for handler ${handler.handlerTag}`
       );
     }
 
@@ -51,7 +48,7 @@ export class GestureHandlerWebDelegate
       touchAction: this.view.style.touchAction,
     };
 
-    const config = handler.getConfig();
+    const config = handler.config;
 
     this.setUserSelect(config.enabled);
     this.setTouchAction(config.enabled);
@@ -88,12 +85,12 @@ export class GestureHandlerWebDelegate
   }
 
   tryResetCursor() {
-    const config = this.gestureHandler.getConfig();
+    const config = this.gestureHandler.config;
 
     if (
       config.activeCursor &&
       config.activeCursor !== 'auto' &&
-      this.gestureHandler.getState() === State.ACTIVE
+      this.gestureHandler.state === State.ACTIVE
     ) {
       this.view.style.cursor = 'auto';
     }
@@ -132,7 +129,7 @@ export class GestureHandlerWebDelegate
   }
 
   private setUserSelect(isHandlerEnabled: boolean) {
-    const { userSelect } = this.gestureHandler.getConfig();
+    const { userSelect } = this.gestureHandler.config;
 
     this.view.style['userSelect'] = isHandlerEnabled
       ? (userSelect ?? 'none')
@@ -144,7 +141,7 @@ export class GestureHandlerWebDelegate
   }
 
   private setTouchAction(isHandlerEnabled: boolean) {
-    const { touchAction } = this.gestureHandler.getConfig();
+    const { touchAction } = this.gestureHandler.config;
 
     this.view.style['touchAction'] = isHandlerEnabled
       ? (touchAction ?? 'none')
@@ -157,7 +154,7 @@ export class GestureHandlerWebDelegate
   }
 
   private setContextMenu(isHandlerEnabled: boolean) {
-    const config = this.gestureHandler.getConfig();
+    const config = this.gestureHandler.config;
 
     if (isHandlerEnabled) {
       this.addContextMenuListeners(config);
@@ -174,6 +171,21 @@ export class GestureHandlerWebDelegate
     this.setUserSelect(enabled);
     this.setTouchAction(enabled);
     this.setContextMenu(enabled);
+
+    if (enabled) {
+      this.eventManagers.forEach((manager) => {
+        // It may look like managers will be registered twice when handler is mounted for the first time.
+        // However, `init` method is called AFTER `updateGestureConfig` - it means that delegate has not
+        // been initialized yet, so this code won't be executed.
+        //
+        // Also, because we use defined functions, not lambdas, they will not be registered multiple times.
+        manager.registerListeners();
+      });
+    } else {
+      this.eventManagers.forEach((manager) => {
+        manager.unregisterListeners();
+      });
+    }
   }
 
   onBegin(): void {
@@ -181,7 +193,7 @@ export class GestureHandlerWebDelegate
   }
 
   onActivate(): void {
-    const config = this.gestureHandler.getConfig();
+    const config = this.gestureHandler.config;
 
     if (
       (!this.view.style.cursor || this.view.style.cursor === 'auto') &&
@@ -210,4 +222,11 @@ export class GestureHandlerWebDelegate
       manager.unregisterListeners();
     });
   }
+
+  public get view() {
+    return this._view;
+  }
+  public set view(value: HTMLElement) {
+    this._view = value;
+  }
 }

package/src/web/tools/InteractionManager.ts

@@ -3,7 +3,7 @@ import { State } from '../../State';
 import { Config, Handler } from '../interfaces';
 
 export default class InteractionManager {
-  private static instance: InteractionManager;
+  private static _instance: InteractionManager;
   private readonly waitForRelations: Map<number, number[]> = new Map();
   private readonly simultaneousRelations: Map<number, number[]> = new Map();
   private readonly blocksHandlersRelations: Map<number, number[]> = new Map();
@@ -13,7 +13,7 @@ export default class InteractionManager {
   private constructor() {}
 
   public configureInteractions(handler: IGestureHandler, config: Config) {
-    this.dropRelationsForHandlerWithTag(handler.getTag());
+    this.dropRelationsForHandlerWithTag(handler.handlerTag);
 
     if (config.waitFor) {
       const waitFor: number[] = [];
@@ -27,7 +27,7 @@ export default class InteractionManager {
         }
       });
 
-      this.waitForRelations.set(handler.getTag(), waitFor);
+      this.waitForRelations.set(handler.handlerTag, waitFor);
     }
 
     if (config.simultaneousHandlers) {
@@ -40,7 +40,7 @@ export default class InteractionManager {
         }
       });
 
-      this.simultaneousRelations.set(handler.getTag(), simultaneousHandlers);
+      this.simultaneousRelations.set(handler.handlerTag, simultaneousHandlers);
     }
 
     if (config.blocksHandlers) {
@@ -53,7 +53,7 @@ export default class InteractionManager {
         }
       });
 
-      this.blocksHandlersRelations.set(handler.getTag(), blocksHandlers);
+      this.blocksHandlersRelations.set(handler.handlerTag, blocksHandlers);
     }
   }
 
@@ -62,12 +62,12 @@ export default class InteractionManager {
     otherHandler: IGestureHandler
   ): boolean {
     const waitFor: number[] | undefined = this.waitForRelations.get(
-      handler.getTag()
+      handler.handlerTag
     );
 
     return (
       waitFor?.find((tag: number) => {
-        return tag === otherHandler.getTag();
+        return tag === otherHandler.handlerTag;
       }) !== undefined
     );
   }
@@ -77,11 +77,11 @@ export default class InteractionManager {
     otherHandler: IGestureHandler
   ): boolean {
     const simultaneousHandlers: number[] | undefined =
-      this.simultaneousRelations.get(handler.getTag());
+      this.simultaneousRelations.get(handler.handlerTag);
 
     return (
       simultaneousHandlers?.find((tag: number) => {
-        return tag === otherHandler.getTag();
+        return tag === otherHandler.handlerTag;
       }) !== undefined
     );
   }
@@ -91,12 +91,12 @@ export default class InteractionManager {
     otherHandler: IGestureHandler
   ): boolean {
     const waitFor: number[] | undefined = this.blocksHandlersRelations.get(
-      handler.getTag()
+      handler.handlerTag
     );
 
     return (
       waitFor?.find((tag: number) => {
-        return tag === otherHandler.getTag();
+        return tag === otherHandler.handlerTag;
       }) !== undefined
     );
   }
@@ -108,7 +108,7 @@ export default class InteractionManager {
     // We check constructor name instead of using `instanceof` in order do avoid circular dependencies
     const isNativeHandler =
       otherHandler.constructor.name === 'NativeViewGestureHandler';
-    const isActive = otherHandler.getState() === State.ACTIVE;
+    const isActive = otherHandler.state === State.ACTIVE;
     const isButton = otherHandler.isButton?.() === true;
 
     return isNativeHandler && isActive && !isButton;
@@ -126,11 +126,11 @@ export default class InteractionManager {
     this.blocksHandlersRelations.clear();
   }
 
-  public static getInstance(): InteractionManager {
-    if (!this.instance) {
-      this.instance = new InteractionManager();
+  public static get instance(): InteractionManager {
+    if (!this._instance) {
+      this._instance = new InteractionManager();
     }
 
-    return this.instance;
+    return this._instance;
   }
 }

package/src/web/tools/NodeManager.ts

@@ -28,7 +28,7 @@ export default abstract class NodeManager {
     }
 
     this.gestures[handlerTag] = handler;
-    this.gestures[handlerTag].setTag(handlerTag);
+    this.gestures[handlerTag].handlerTag = handlerTag;
   }
 
   public static dropGestureHandler(handlerTag: number): void {
@@ -42,7 +42,7 @@ export default abstract class NodeManager {
     delete this.gestures[handlerTag];
   }
 
-  public static getNodes() {
+  public static get nodes() {
     return { ...this.gestures };
   }
 }

package/src/web/tools/PointerTracker.ts

@@ -13,7 +13,7 @@ const MAX_POINTERS = 20;
 
 export default class PointerTracker {
   private velocityTracker = new VelocityTracker();
-  private trackedPointers: Map<number, TrackerElement> = new Map<
+  private readonly _trackedPointers: Map<number, TrackerElement> = new Map<
     number,
     TrackerElement
   >();
@@ -72,7 +72,7 @@ export default class PointerTracker {
     this.lastMovedPointerId = event.pointerId;
 
     this.velocityTracker.add(event);
-    const [velocityX, velocityY] = this.velocityTracker.getVelocity();
+    const [velocityX, velocityY] = this.velocityTracker.velocity;
 
     element.velocityX = velocityX;
     element.velocityY = velocityY;
@@ -189,24 +189,6 @@ export default class PointerTracker {
     return sum;
   }
 
-  public getTrackedPointersCount(): number {
-    return this.trackedPointers.size;
-  }
-
-  public getTrackedPointersID(): number[] {
-    const keys: number[] = [];
-
-    this.trackedPointers.forEach((_value, key) => {
-      keys.push(key);
-    });
-
-    return keys;
-  }
-
-  public getData(): Map<number, TrackerElement> {
-    return this.trackedPointers;
-  }
-
   public resetTracker(): void {
     this.velocityTracker.reset();
     this.trackedPointers.clear();
@@ -223,4 +205,22 @@ export default class PointerTracker {
   ): boolean {
     return stPointers.some((pointerId) => ndPointers.includes(pointerId));
   }
+
+  public get trackedPointersCount(): number {
+    return this.trackedPointers.size;
+  }
+
+  public get trackedPointersIDs() {
+    const keys: number[] = [];
+
+    this.trackedPointers.forEach((_value, key) => {
+      keys.push(key);
+    });
+
+    return keys;
+  }
+
+  public get trackedPointers() {
+    return this._trackedPointers;
+  }
 }

package/src/web/tools/Vector.ts

@@ -30,7 +30,7 @@ export default class Vector {
     return new Vector(velocity.x, velocity.y);
   }
 
-  get magnitude() {
+  public get magnitude() {
     return this._magnitude;
   }
 

package/src/web/tools/VelocityTracker.ts

@@ -84,7 +84,7 @@ export default class VelocityTracker {
     return null;
   }
 
-  public getVelocity(): [number, number] {
+  public get velocity(): [number, number] {
     const estimate = this.getVelocityEstimate();
     if (estimate !== null) {
       return estimate;