@witchcraft/layout 0.5.8 → 0.5.9

package/dist/module.json

@@ -1,7 +1,7 @@
 {
   "name": "@witchcraft/layout",
   "configKey": "witchcraftLayout",
-  "version": "0.5.8",
+  "version": "0.5.9",
   "builder": {
     "@nuxt/module-builder": "1.0.2",
     "unbuild": "unknown"

package/dist/runtime/components/LayoutWindow.vue

@@ -98,7 +98,6 @@ const actionHandler = props.actionHandler ?? markRaw(new ActionHandler(
   ],
   {
     onEvent: (e, cancel) => {
-      showMoving.value = true;
       if (e instanceof KeyboardEvent && e.key === "Escape") {
         cancel();
       }
@@ -108,7 +107,6 @@ const actionHandler = props.actionHandler ?? markRaw(new ActionHandler(
     },
     onEnd: () => {
       requestType.value = void 0;
-      showMoving.value = true;
     }
   }
 ));

package/dist/runtime/move/ActionHandler.d.ts

@@ -43,7 +43,7 @@ export declare class ActionHandler<TRawActions extends IAction[], TActions exten
      * The default prevents movement when the edge is a window edge and when the edge is touching a collapsed frame.
      */
     defaultOnMoveChange?: IAction["onMoveChange"]);
-    eventHandler(e: KeyboardEvent | PointerEvent, state: MoveState, forceRecalculateEdges: () => void): undefined;
+    eventHandler(e: KeyboardEvent | PointerEvent | undefined, state: MoveState, forceRecalculateEdges: () => void): undefined;
     onMoveChange<T extends "start" | "move" | "end">(type: T, e: T extends "end" | "start" ? PointerEvent | undefined : PointerEvent, state: MoveState, forceRecalculateEdges: () => void, cancel: (e: PointerEvent | KeyboardEvent | undefined, state: MoveState) => void, resolve: T extends "end" ? undefined : ((opts: ActionResolve) => void)): MoveChangeResult;
     onMoveApply(state: MoveState, forceRecalculateEdges: () => void): ActionHandlerApplyResult;
     onMoveEnded(): void;

package/dist/runtime/move/ActionHandler.js

@@ -36,7 +36,7 @@ export class ActionHandler {
   }
   eventHandler(e, state, forceRecalculateEdges) {
     if (state.isMoving) {
-      e.preventDefault();
+      e?.preventDefault();
     }
     let cancelled = false;
     this.hooks.onEvent?.(e, () => {

package/dist/runtime/move/CloseAction.d.ts

@@ -29,7 +29,7 @@ export declare class CloseAction implements IAction {
         }) => string[]);
         transformError: (e: KnownError) => string;
     };
-    handleEvent: (e: PointerEvent | KeyboardEvent, state: MoveState) => boolean | "force";
+    handleEvent: (e: PointerEvent | KeyboardEvent | undefined, state: MoveState) => boolean | "force";
     modifyDecos: (shapes: CloseDeco[]) => void;
     hooks: {
         onStart?: (active: boolean) => void;
@@ -51,7 +51,7 @@ export declare class CloseAction implements IAction {
         actions: string[];
         errors: string[];
     };
-    canHandleRequest(e: PointerEvent | KeyboardEvent, state: MoveState): boolean;
+    canHandleRequest(e: PointerEvent | KeyboardEvent | undefined, state: MoveState): boolean;
     onMoveChange(type: "start" | "end" | "move", _e: PointerEvent | undefined, state: MoveState): ActionChangeResult;
     onMoveApply(state: MoveState): ActionApplyResult;
     cancel(): void;

package/dist/runtime/move/CloseAction.js

@@ -17,10 +17,10 @@ export class CloseAction {
     transformError: (e) => e.message
   };
   handleEvent = (e) => {
-    if (e.ctrlKey && e.shiftKey) {
+    if (e?.ctrlKey && e?.shiftKey) {
       return "force";
     }
-    if (e.shiftKey) return true;
+    if (e?.shiftKey) return true;
     return false;
   };
   modifyDecos = () => {

package/dist/runtime/move/FrameDragAction.d.ts

@@ -11,7 +11,7 @@ export declare class FrameDragAction implements IAction {
         actions: string[];
         errors: string[];
     };
-    handleEvent: (e: PointerEvent | KeyboardEvent, state: MoveState) => boolean;
+    handleEvent: (e: PointerEvent | KeyboardEvent | undefined, state: MoveState) => boolean;
     dragHints: {
         actions: {
             split?: string;
@@ -43,7 +43,7 @@ export declare class FrameDragAction implements IAction {
         errors: string[];
     };
     getDecos(matchedZone: Zone | undefined, state: MoveState, result: LayoutChange<"split" | "swap" | "rearrange" | "dock"> | KnownError | undefined): FrameDeco[];
-    canHandleRequest(e: PointerEvent | KeyboardEvent, state: MoveState): boolean;
+    canHandleRequest(e: PointerEvent | KeyboardEvent | undefined, state: MoveState): boolean;
     onMoveChange<T extends "start" | "move" | "end">(type: T, _e: PointerEvent | undefined, state: MoveState): ActionChangeResult;
     onMoveApply(state: MoveState): ActionApplyResult;
     onMoveEnded(): void;

package/dist/runtime/move/SplitAction.d.ts

@@ -26,7 +26,7 @@ export declare class SplitAction implements IAction {
         actions: string[] | ((type: DragChangeType) => string[]);
         transformError: (e: KnownError) => string;
     };
-    handleEvent: (e: PointerEvent | KeyboardEvent, state: MoveState) => boolean;
+    handleEvent: (e: PointerEvent | KeyboardEvent | undefined, state: MoveState) => boolean;
     modifyDecos: (shapes: SplitDeco[]) => void;
     hooks: {
         onStart?: () => void;
@@ -43,7 +43,7 @@ export declare class SplitAction implements IAction {
     });
     reset(): void;
     getDecos(state: MoveState): SplitDeco[];
-    canHandleRequest(e: PointerEvent | KeyboardEvent, state: MoveState): boolean;
+    canHandleRequest(e: PointerEvent | KeyboardEvent | undefined, state: MoveState): boolean;
     setTextHints(result: true | SplitInfo | KnownError | undefined): void;
     getTextHints(type: "start" | "move" | "end"): {
         actions: string[];

package/dist/runtime/move/SplitAction.js

@@ -15,7 +15,7 @@ export class SplitAction {
     actions: ["Hold Alt to Split"],
     transformError: (e) => e.message
   };
-  handleEvent = (e, state) => e.altKey || state.isMovingFromWindowEdge;
+  handleEvent = (e, state) => e?.altKey || state.isMovingFromWindowEdge;
   modifyDecos = () => {
   };
   hooks;

package/dist/runtime/types/index.d.ts

@@ -1956,8 +1956,10 @@ export interface IAction {
      * The user is not necessarily dragging at this point, though they might also change actions mid drag. So it does not necessarily mean the event is allowed.
      *
      * Here is where you should initiate your state. Don't allow the action by default unless it can always be allowed.
+     *
+     * Event might be undefined if it was initiated manually via moveStart without an event.
      */
-    canHandleRequest(e: KeyboardEvent | PointerEvent, state: MoveState, forceRecalculateEdges: () => void): boolean;
+    canHandleRequest(e: KeyboardEvent | PointerEvent | undefined, state: MoveState, forceRecalculateEdges: () => void): boolean;
     /**
      * Plugins should implement some basic debug logs by calling {@link ActionHandler.debugState } at least before and after applying actions in onMoveApply. Debug can be a string because it can be an object key to filter on (see the debugState function).
      *

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@witchcraft/layout",
   "description": "Headless layout manager.",
-  "version": "0.5.8",
+  "version": "0.5.9",
   "main": "./dist/runtime/index.js",
   "type": "module",
   "sideEffects": false,

package/src/runtime/components/LayoutWindow.vue

@@ -128,7 +128,6 @@ const actionHandler = props.actionHandler ?? markRaw(new ActionHandler(
 	],
 	{
 		onEvent: (e, cancel) => {
-			showMoving.value = true
 			if (e instanceof KeyboardEvent && e.key === "Escape") {
 				cancel()
 			}
@@ -138,7 +137,6 @@ const actionHandler = props.actionHandler ?? markRaw(new ActionHandler(
 		},
 		onEnd: () => {
 			requestType.value = undefined
-			showMoving.value = true
 		},
 	}
 ))

package/src/runtime/move/ActionHandler.ts

@@ -77,12 +77,12 @@ export class ActionHandler<
 	}
 
 	eventHandler(
-		e: KeyboardEvent | PointerEvent,
+		e: KeyboardEvent | PointerEvent | undefined,
 		state: MoveState,
 		forceRecalculateEdges: () => void
 	) {
 		if (state.isMoving) {
-			e.preventDefault()
+			e?.preventDefault()
 		}
 		let cancelled = false
 		this.hooks.onEvent?.(e, () => {
@@ -131,7 +131,7 @@ export class ActionHandler<
 	): MoveChangeResult {
 		if (type === "start") {
 			this.eventCanceller = cancel
-			this.eventHandler(e!, state, forceRecalculateEdges)
+			this.eventHandler(e, state, forceRecalculateEdges)
 		}
 
 		if (this.activeAction) {

package/src/runtime/move/CloseAction.ts

@@ -40,11 +40,11 @@ export class CloseAction implements IAction {
 		transformError: e => e.message
 	}
 
-	handleEvent: (e: PointerEvent | KeyboardEvent, state: MoveState) => boolean | "force" = (e: PointerEvent | KeyboardEvent) => {
-		if (e.ctrlKey && e.shiftKey) {
+	handleEvent: (e: PointerEvent | KeyboardEvent | undefined, state: MoveState) => boolean | "force" = (e: PointerEvent | KeyboardEvent | undefined) => {
+		if (e?.ctrlKey && e?.shiftKey) {
 			return "force"
 		}
-		if (e.shiftKey) return true
+		if (e?.shiftKey) return true
 		return false
 	}
 
@@ -136,7 +136,7 @@ export class CloseAction implements IAction {
 	}
 
 
-	canHandleRequest(e: PointerEvent | KeyboardEvent, state: MoveState): boolean {
+	canHandleRequest(e: PointerEvent | KeyboardEvent | undefined, state: MoveState): boolean {
 		const { movingEdges } = state
 		if (movingEdges.length !== 1) return false
 		const res = this.handleEvent(e, state)

package/src/runtime/move/FrameDragAction.ts

@@ -25,8 +25,8 @@ export class FrameDragAction implements IAction {
 	debug: boolean | string = false
 	textHints: { actions: string[], errors: string[] } = { actions: [], errors: [] }
 
-	handleEvent: (e: PointerEvent | KeyboardEvent, state: MoveState) => boolean
-		= (_e: PointerEvent | KeyboardEvent, state: MoveState) => state.isMoving === "frame"
+	handleEvent: (e: PointerEvent | KeyboardEvent | undefined, state: MoveState) => boolean
+		= (_e: PointerEvent | KeyboardEvent | undefined, state: MoveState) => state.isMoving === "frame"
 
 	dragHints: {
 		actions: { split?: string, swap?: string, rearrange?: string, dock?: string } | ((state: { type: "split" | "swap" | "rearrange" | "dock" }) => string[])
@@ -139,7 +139,7 @@ export class FrameDragAction implements IAction {
 		return decos
 	}
 
-	canHandleRequest(e: PointerEvent | KeyboardEvent, state: MoveState): boolean {
+	canHandleRequest(e: PointerEvent | KeyboardEvent | undefined, state: MoveState): boolean {
 		const res = this.handleEvent(e, state)
 		this.setTextHints(undefined)
 		if (res) {

package/src/runtime/move/SplitAction.ts

@@ -40,8 +40,8 @@ export class SplitAction implements IAction {
 		transformError: e => e.message
 	}
 
-	handleEvent: (e: PointerEvent | KeyboardEvent, state: MoveState) => boolean
-		= (e: PointerEvent | KeyboardEvent, state: MoveState) => e.altKey || state.isMovingFromWindowEdge
+	handleEvent: (e: PointerEvent | KeyboardEvent | undefined, state: MoveState) => boolean
+		= (e: PointerEvent | KeyboardEvent | undefined, state: MoveState) => e?.altKey || state.isMovingFromWindowEdge
 
 	modifyDecos: (shapes: SplitDeco[]) => void = () => { }
 	hooks: {
@@ -107,7 +107,7 @@ export class SplitAction implements IAction {
 		return decos
 	}
 
-	canHandleRequest(e: PointerEvent | KeyboardEvent, state: MoveState): boolean {
+	canHandleRequest(e: PointerEvent | KeyboardEvent | undefined, state: MoveState): boolean {
 		const { movingEdges } = state
 		if (movingEdges.length !== 1) return false
 		// hint should not be shown when dragging from window edge but we should still handle the event

package/src/runtime/types/index.ts

@@ -574,9 +574,11 @@ export interface IAction {
 	 * The user is not necessarily dragging at this point, though they might also change actions mid drag. So it does not necessarily mean the event is allowed.
 	 *
 	 * Here is where you should initiate your state. Don't allow the action by default unless it can always be allowed.
+	 *
+	 * Event might be undefined if it was initiated manually via moveStart without an event.
 	 */
 	canHandleRequest(
-		e: KeyboardEvent | PointerEvent,
+		e: KeyboardEvent | PointerEvent | undefined,
 		state: MoveState,
 		forceRecalculateEdges: () => void
 	): boolean