MACOS: Tested OK in multi-monitor setup

Author: Kalmat

CHANGES.txt

@@ -1,3 +1,4 @@
+0.5, 2023/09/06 -- MACOS: Tested OK in multi-monitor setup
 0.4, 2023/08/25 -- Reorganized to avoid IDEs showing external and / or private elements
 0.3, 2023/08/21 -- Linux: Improved window geometry (thanks to elraymond - https://github.com/elraymond)
                    ALL: Improved default onQuery/onSet methods

README.md

@@ -7,8 +7,6 @@
 Cross-Platform and multi-monitor module which allows to manage window areas (position and
 size) and all their properties, as well as any rectangular area.
 
-**WARNING: Multi-monitor support is still experimental and not fully tested in macOS.**
-
 ## Rectangular areas
 
 You just need to instantiate the PyWinBox class, passing custom callbacks to be called when any property is 

dist/PyWinBox-0.4-py3-none-any.whl dist/PyWinBox-0.5-py3-none-any.whldist/PyWinBox-0.4-py3-none-any.whl renamed to dist/PyWinBox-0.5-py3-none-any.whl

@@ -5,7 +5,7 @@
     "version", "PyWinBox", "Box", "Rect", "Point", "Size", "pointInBox"
 ]
 
-__version__ = "0.4"
+__version__ = "0.5"
 
 
 def version(numberOnly: bool = True) -> str:

src/pywinbox/__init__.py

@@ -83,22 +83,33 @@ def __init__(self, onQuery: Optional[Callable[[], Box]] = None, onSet: Optional[
         In this case, if your custom functions do not properly retrieve or set the actual window position and size, the
         information contained in the PyWinBox class, and returned by all properties, will likely become obsolete.
 
-        It can raise ValueError if no parameters are passed.
+        It can raise ValueError if no parameters or not valid window handle are passed
         """
-        self._box: Box = Box(0, 0, 0, 0)
-        if handle is not None:
-            self._handle = _getHandle(handle) if handle is not None else None
-        elif onSet is None and onQuery is None:
+        self._handle = _getHandle(handle) if handle is not None else None
+        if self._handle is None and (onSet is None or onQuery is None):
             raise ValueError
         self._onQuery: Callable[[], Box] = onQuery or self.onQuery
         self._onSet: Callable[[Box], None] = onSet or self.onSet
+        self._box: Box = Box(0, 0, 0, 0)
 
     def onQuery(self) -> Box:
+        """
+        Default method to retrieve current window position and size values when a property is queried.
+        It requires to pass valid window handle when instantiating the main class (PyWinBox class)
+
+        :return: window Box struct (x, y, width, height)
+        """
         if self._handle is not None:
             self._box = _getWindowBox(self._handle)
         return self._box
 
     def onSet(self, newBox: Box):
+        """
+        Default method to actually place / resize the window when a property is changed.
+        It requires to pass valid window handle when instantiating the main class (PyWinBox class)
+
+        :param newBox: target position and or size in Box struct format (x, y, width, height)
+        """
         if self._handle is not None:
             _moveResizeWindow(self._handle, newBox)
 
@@ -114,7 +125,7 @@ def __repr__(self):
 
     def __str__(self):
         """Return a string representation of this Box object."""
-        return "(%s, %s, w=%s, h=%s)" % (
+        return "(%s, %s, %s, %s)" % (
             self._box.left,
             self._box.top,
             self._box.width,

src/pywinbox/_main.py

@@ -154,7 +154,7 @@ def _unflipTop(window: AppKit.NSWindow, box: Box) -> int:
 
 def _NSgetWindowBox(window: AppKit.NSWindow, flipValues: bool = False) -> Box:
     # frame = window.frame()
-    frame = window.screen().convertRectToBacking_(window.frame())
+    frame = window.frame()
     x = int(frame.origin.x)
     y = int(frame.origin.y)
     w = int(frame.size.width)

src/pywinbox/_pywinbox_macos.py

@@ -50,7 +50,6 @@ def applicationDidFinishLaunching_(self, aNotification: None):
 
             if win:
                 print("ACTIVE WINDOW:", win.title)
-
                 myPyBox = pywinbox.PyWinBox(onQuery=None, onSet=None, handle=win.getHandle())
 
             else:
@@ -59,6 +58,8 @@ def applicationDidFinishLaunching_(self, aNotification: None):
 
             timelap = 0.3
 
+            print("INIT", myPyBox.box, myPyBox._handle.window.frame())
+
             print("MOVE left = 200", myPyBox.box, myPyBox.rect)
             myPyBox.left = 200
             time.sleep(timelap)
@@ -175,7 +176,7 @@ def demo():
     a.setDelegate_(delegate)
 
     # Now we can start to create the window ...
-    frame = NSMakeRect(400, 400, 250, 100)
+    frame = NSMakeRect(400, 400, 250, 128)
     # (Don't worry about these parameters for the moment. They just specify
     # the type of window, its size and position etc)
     mask = NSWindowStyleMaskTitled | NSWindowStyleMaskClosable | NSWindowStyleMaskMiniaturizable | NSWindowStyleMaskResizable

tests/test_MacNSBox.py

@@ -141,10 +141,10 @@ def test_basic():
         print(npw.box, npw.rect)
         assert npw.height == 500
 
-        myPyBox.size = (801, 601)
+        myPyBox.size = (301, 301)
         time.sleep(timelap)
         print(npw.box, npw.rect)
-        assert npw.size == (801, 601)
+        assert npw.size == (301, 301)
 
         # Test closing
         npw.close()