Fix "maximize" not using drag rules

Author: ECRomaneli

src/main.ts

@@ -279,14 +279,19 @@ export class Draggable {
           const isDraggable = Draggable.isDraggable(wc, input, options);
           if (isDraggable === false) { return; }
           this.setInitialPosition(dragState); // Not using input.x/y because of inconsistent values
+
+          if (isDraggable === true) { dragState.interval = null; return; }
           (async () => await isDraggable && (dragState.interval = null))();
           return;
         }
 
         // Handle double-click to maximize/unmaximize
         if (input.clickCount === 2 && options.maximize) {
+          const isDraggable = Draggable.isDraggable(wc, input, options);
+          if (isDraggable === false) { return; }
           e.preventDefault();
-          this.toggleMaximize();
+          if (isDraggable === true) { this.toggleMaximize(); return; }
+          (async () => await isDraggable && this.toggleMaximize())();
         }
       }
     };
@@ -311,7 +316,7 @@ export class Draggable {
   }
 
   // eslint-disable-next-line @stylistic/max-len
-  private static isDraggable(webContents: WebContents, point: Point, options: InternalDragOptions): false | Promise<boolean> {
+  private static isDraggable(webContents: WebContents, point: Point, options: InternalDragOptions): boolean | Promise<boolean> {
     if (options.region && !Draggable.isDraggingRegion(options.region, point)) { return false; }
 
     if (options.selector) {
@@ -322,7 +327,7 @@ export class Draggable {
       return Draggable.closest(webContents, point, options.exclude, true);
     }
 
-    return Draggable.TRUE_PROMISE;
+    return true;
   }
 
   private static isDraggingRegion(region: Partial<Rectangle>, point: Point): boolean {

test/sample.js

@@ -3,7 +3,6 @@ const { Draggable } = require('electron-draggable')
 
 app.whenReady().then(() => {  
   const window = setupWindow();
-  setupFindbar(window);
   Menu.setApplicationMenu(null);
   setupApplicationMenu(window);
 })
@@ -21,7 +20,7 @@ function setupWindow() {
   view.setBounds({ x: 0, y: 0, width: 800, height: 600 });
   view.webContents.loadFile(`${__dirname}/sample.html`);
   const dragHandler = Draggable.from(window, { dragZone: { height: 100 } }).attach(view.webContents, { exclude: '.not-drag-1, button'});
-  Draggable.create(window, { selector: '.drag-2', fps: 10 }).attach(view.webContents);
+  Draggable.create(window, { selector: '.drag-2', fps: 10, maximize: true }).attach(view.webContents);
   if (dragHandler !== Draggable.from(window)) {
     window.destroy();
     throw new Error('attach did not return the Draggable instance');