From 822c6b5cd1efa270a49f40bd55147dc18158320f Mon Sep 17 00:00:00 2001 From: Igor Zinken <730069+igorski@users.noreply.github.com> Date: Wed, 5 Mar 2025 13:15:25 +0100 Subject: [PATCH] Use crosshair icon while drawing selection --- src/components/document-canvas/document-canvas.vue | 6 +++++- src/rendering/canvas-elements/interaction-pane.ts | 4 ++-- 2 files changed, 7 insertions(+), 3 deletions(-) diff --git a/src/components/document-canvas/document-canvas.vue b/src/components/document-canvas/document-canvas.vue index 55600f6..5a323d6 100644 --- a/src/components/document-canvas/document-canvas.vue +++ b/src/components/document-canvas/document-canvas.vue @@ -396,6 +396,10 @@ export default { case ToolTypes.DRAG: canvasClasses.add( "cursor-drag" ); break; + case ToolTypes.SELECTION: + case ToolTypes.LASSO: + canvasClasses.add( "cursor-crosshair" ); + break; case ToolTypes.BRUSH: case ToolTypes.CLONE: case ToolTypes.ERASER: @@ -429,7 +433,7 @@ export default { } else if ( this.layerSelectMode ) { mode = InteractionModes.MODE_LAYER_SELECT; } else if ( this.selectMode ) { - mode =InteractionModes.MODE_SELECTION; + mode = InteractionModes.MODE_SELECTION; } zCanvas.interactionPane.setState( enabled, mode, this.activeTool, this.activeToolOptions ); if ( enabled ) { diff --git a/src/rendering/canvas-elements/interaction-pane.ts b/src/rendering/canvas-elements/interaction-pane.ts index 8f5162a..e3f4d16 100644 --- a/src/rendering/canvas-elements/interaction-pane.ts +++ b/src/rendering/canvas-elements/interaction-pane.ts @@ -383,8 +383,8 @@ class InteractionPane extends sprite { } update( _now: DOMHighResTimeStamp, framesSinceLastUpdate: number ): void { - if ( this.getActiveDocument().activeSelection?.length > 0 ) { - this._dashOffset -= ( DASH_SPEED * framesSinceLastUpdate ); // advance the outline animation + if ( this._selectionClosed && this.getActiveDocument().activeSelection?.length ) { + this._dashOffset -= ( DASH_SPEED * framesSinceLastUpdate ); // advance the selection outline animation } }