feat: update code

Author: cptbtptpbcptdtptp

packages/core/src/input/InputManager.ts

@@ -156,9 +156,9 @@ export class InputManager {
     // @ts-ignore
     const canvas = engine._canvas._webCanvas;
     if (typeof OffscreenCanvas === "undefined" || !(canvas instanceof OffscreenCanvas)) {
-      this._wheelManager = new WheelManager(engine, inputOptions?.wheelTarget);
-      this._pointerManager = new PointerManager(engine, inputOptions?.pointerTarget);
-      this._keyboardManager = new KeyboardManager(engine, inputOptions?.keyboardTarget);
+      this._wheelManager = new WheelManager(engine, inputOptions?.wheelTarget ?? canvas);
+      this._pointerManager = new PointerManager(engine, inputOptions?.pointerTarget ?? canvas);
+      this._keyboardManager = new KeyboardManager(engine, inputOptions?.keyboardTarget ?? window);
       this._initialized = true;
     }
   }

packages/core/src/input/keyboard/KeyboardManager.ts

@@ -32,7 +32,7 @@ export class KeyboardManager implements IInput {
   /**
    * @internal
    */
-  constructor(engine: Engine, target: EventTarget = window) {
+  constructor(engine: Engine, target: EventTarget) {
     this._engine = engine;
     this._onBlur = this._onBlur.bind(this);
     this._onKeyEvent = this._onKeyEvent.bind(this);

packages/core/src/input/pointer/PointerManager.ts

@@ -45,15 +45,18 @@ export class PointerManager implements IInput {
   /**
    * @internal
    */
-  constructor(engine: Engine, target?: EventTarget) {
+  constructor(engine: Engine, target: EventTarget) {
+    if (target instanceof Window) {
+      throw "Do not set window as target because window cannot listen to pointer leave event.";
+    }
     this._engine = engine;
+    this._target = target;
     this._canvas = engine.canvas;
-    this._onPointerEvent = this._onPointerEvent.bind(this);
+    // @ts-ignore
+    this._htmlCanvas = engine._canvas._webCanvas;
     // If there are no compatibility issues, navigator.maxTouchPoints should be used here
     this._pointerPool = new Array<Pointer>(11);
-    // @ts-ignore
-    const htmlCanvas = (this._htmlCanvas = engine._canvas._webCanvas);
-    this._target = target ?? htmlCanvas;
+    this._onPointerEvent = this._onPointerEvent.bind(this);
     this._addEventListener();
   }
 

packages/core/src/input/wheel/WheelManager.ts

@@ -17,7 +17,7 @@ export class WheelManager implements IInput {
   /**
    * @internal
    */
-  constructor(engine: Engine, target: EventTarget = window) {
+  constructor(engine: Engine, target: EventTarget) {
     this._onWheelEvent = this._onWheelEvent.bind(this);
     this._target = target;
     this._addEventListener();

packages/design/src/input/IInputOptions.ts

@@ -4,20 +4,20 @@
 export interface IInputOptions {
   /**
    * The target element of the pointer event, defaults is the current canvas.
+   * @remarks
    * @remarks: When setting the pointer target you need to specify:
-   *  1. Do not set window as target because window cannot listen to pointer leave event.
-   *  2. On mobile, pointer move may trigger the default slide event, thereby removing the point from the screen,
-   *     so in most cases you need to set HtmlElement.style.touchAction = "none";
+   *  - Do not set window as target because window cannot listen to pointer leave event.
+   *  - On mobile, pointer move may trigger the default slide event, thereby removing the point from the screen, so in most cases you need to set HtmlElement.style.touchAction = "none".
    */
-  pointerTarget?: EventTarget;
+  pointerTarget?: Exclude<EventTarget, Window & typeof globalThis>;
+
   /**
    * The target element of the keyboard event, defaults is the window.
-   * @remarks: When setting the keyboard target you need to specify:
-   * If the type of listening source is HtmlElement, you need to set tabIndex to make the element focus.
+   * @remarks
+   * If the type of listening target is `HtmlElement`, you need to set tabIndex to make the element focus.
    */
   keyboardTarget?: EventTarget;
-  /**
-   * The target element of the wheel event, defaults is the window.
-   */
+
+  /** The target element of the wheel event, defaults is the current canvas. */
   wheelTarget?: EventTarget;
 }