bare-pointer 0.1.2 → 0.1.3

package/README.md

@@ -32,7 +32,9 @@ pointer.destroy()
 | Android  | `bare-ndk`     | Motion events dispatched to the root of the view       |
 | Windows  | `bare-win-ui`  | `SendInput()` through `bare-user32`                    |
 
-On Linux and Windows the input goes through the system, so the window of the view has to be the one on top at the point. On macOS, a control tracks the mouse until it is released, so press one with `tap()` rather than `down()`.
+On Linux the input goes through the system, so the window of the view has to be the one on top at the point. On Windows it does too, and a window covered at the point is brought to the foreground first. On macOS, a control tracks the mouse until it is released, so press one with `tap()` rather than `down()`.
+
+On Windows, a press that no element marks as handled can leave windows opened after it without pointer input once one of them has closed. Have something take every press, such as the root of each window.
 
 ## License
 

package/lib/darwin.js

@@ -16,6 +16,18 @@ module.exports = class DarwinPointer {
     return [width, height]
   }
 
+  covered(view) {
+    const window = view.window
+
+    if (window.keyWindow || !window.canBecomeKeyWindow || !AppKitApplication.active) {
+      return false
+    }
+
+    window.makeKeyAndOrderFront()
+
+    return true
+  }
+
   watch(view, onhidden) {
     let request = null
 

package/lib/win32.js

@@ -16,10 +16,6 @@ const {
 } = user32.constants
 
 module.exports = class Win32Pointer {
-  constructor() {
-    this._point = null
-  }
-
   size(view) {
     return [view.actualWidth, view.actualHeight]
   }
@@ -36,35 +32,32 @@ module.exports = class Win32Pointer {
   }
 
   hover(view, x, y) {
-    this._point = normalize(locate(view, x, y).screen)
-    this._send(0)
+    this._move(view, x, y)
   }
 
   down() {
-    this._send(MOUSEEVENTF_LEFTDOWN)
+    this._send({ flags: MOUSEEVENTF_LEFTDOWN })
   }
 
   move(view, x, y) {
-    this._point = normalize(locate(view, x, y).screen)
-    this._send(0)
+    this._move(view, x, y)
   }
 
   up() {
-    this._send(MOUSEEVENTF_LEFTUP)
+    this._send({ flags: MOUSEEVENTF_LEFTUP })
   }
 
   destroy() {}
 
-  _send(flags) {
-    const sent = user32.sendInput([
-      {
-        type: INPUT_MOUSE,
-        mi: {
-          ...this._point,
-          flags: flags | MOUSEEVENTF_MOVE | MOUSEEVENTF_ABSOLUTE | MOUSEEVENTF_VIRTUALDESK
-        }
-      }
-    ])
+  _move(view, x, y) {
+    this._send({
+      ...normalize(locate(view, x, y).screen),
+      flags: MOUSEEVENTF_MOVE | MOUSEEVENTF_ABSOLUTE | MOUSEEVENTF_VIRTUALDESK
+    })
+  }
+
+  _send(mi) {
+    const sent = user32.sendInput([{ type: INPUT_MOUSE, mi }])
 
     if (sent === 0) throw new Error('Input was blocked')
   }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "bare-pointer",
-  "version": "0.1.2",
+  "version": "0.1.3",
   "description": "Simulated pointer input for testing Bare apps",
   "exports": {
     "./package": "./package.json",