Window start drag bugfix for windows that moved outside drag events

Author: hpcreery

WobblyWindows.xcodeproj/project.xcworkspace/xcuserdata/Professional.xcuserdatad/UserInterfaceState.xcuserstate

@@ -148,6 +148,7 @@ extension NSScreen {
   }
 
   @objc public func startDrag(at point: CGPoint) {
+    self.alreadySetFrame = false
     let distances = particles
       .map { $0.position.distanceTo(point: point) }
 
@@ -183,7 +184,9 @@ extension NSScreen {
     }
   }
 
+  var alreadySetFrame = false
   @objc public func endDrag() {
+    if alreadySetFrame { return }
     let idx = GRID_WIDTH * (GRID_HEIGHT - 1) + GRID_HEIGHT * (GRID_WIDTH - 1)
     springs.removeLast(springs.count - idx)
     if particles.indices.contains(GRID_WIDTH * GRID_HEIGHT) { particles.remove(at: GRID_WIDTH * GRID_HEIGHT) }
@@ -208,6 +211,7 @@ extension NSScreen {
         )
 
         self.window.setFrameDirty(frame)
+        self.alreadySetFrame = true
 
         self.window.styleMask.insert(NSWindow.StyleMask.resizable)
       } else {

WobblyWindows/Warp.swift

@@ -62,10 +62,8 @@ + (void) willMove:(id) notification {
 //  NSLog(@"willMove");
   NSWindow* window = (NSWindow*)[(NSNotification*)notification object];
 
-  // Fallback for whatsapp and IDEA
-  if (window.warp == NULL) {
-    window.warp = [[Warp alloc] initWithWindow:window];
-  }
+  //Reinit window since it may have moved in different method
+  window.warp = [[Warp alloc] initWithWindow:window];
 
   [window.warp startDragAt: NSEvent.mouseLocation];
   [window windowMoves: notification];
@@ -78,9 +76,9 @@ - (void) windowMoves:(id) notification {
   NSWindow* window = (NSWindow*)[(NSNotification*)notification object];
   timer = [NSTimer scheduledTimerWithTimeInterval:(1.0f / 60.0f) target:self selector:@selector(windowMoved:) userInfo:window repeats:YES];
 
-//  if (monitor != NULL) { // only disable mouseup monitor when we move a window again, because sometimes the first event does not fully trigger.
-//    [NSEvent removeMonitor:monitor];
-//  }
+  if (monitor != NULL) { // only disable mouseup monitor when we move a window again, because sometimes the first event does not fully trigger.
+    [NSEvent removeMonitor:monitor];
+  }
   monitor = [NSEvent addGlobalMonitorForEventsMatchingMask:NSEventMaskLeftMouseUp | NSEventMaskRightMouseUp handler:^(NSEvent *event) {
     [window moveStopped];
   }];