fix: Allow workspace comments to be moved in constrained mode. (#646)

* fix: Allow workspace comments to be moved in constrained mode.

* chore: Add test for moving workspace comments in constrained mode.

* refactor: Use arrow wrapper functions in test.

Author: gonfunko

src/actions/mover.ts

@@ -318,6 +318,10 @@ export class Mover {
     const info = this.moves.get(workspace);
     if (!info) throw new Error('no move info for workspace');
 
+    if (info.draggable instanceof comments.RenderedWorkspaceComment) {
+      return this.moveUnconstrained(workspace, direction);
+    }
+
     info.dragger.onDrag(
       info.fakePointerEvent('pointermove', direction),
       info.totalDelta.clone().scale(workspace.scale),

test/webdriverio/test/workspace_comment_test.ts

@@ -189,7 +189,7 @@ suite('Workspace comment navigation', function () {
     );
   });
 
-  test('Workspace comments can be moved', async function () {
+  test('Workspace comments can be moved in unconstrained mode', async function () {
     await focusOnWorkspaceComment(this.browser, this.commentId1);
 
     const initialPosition = await this.getCommentLocation(this.commentId1);
@@ -203,4 +203,19 @@ suite('Workspace comment navigation', function () {
     const newPosition = await this.getCommentLocation(this.commentId1);
     chai.assert.deepEqual(newPosition, [220, 240]);
   });
+
+  test('Workspace comments can be moved in constrained mode', async function () {
+    await focusOnWorkspaceComment(this.browser, this.commentId1);
+
+    const initialPosition = await this.getCommentLocation(this.commentId1);
+    chai.assert.deepEqual(initialPosition, [200, 200]);
+
+    await sendKeyAndWait(this.browser, 'm');
+    await keyUp(this.browser, 2);
+    await keyLeft(this.browser);
+    await sendKeyAndWait(this.browser, Key.Enter);
+
+    const newPosition = await this.getCommentLocation(this.commentId1);
+    chai.assert.deepEqual(newPosition, [180, 160]);
+  });
 });