feat: hide some context menu options when opening menu with mouse (#442)

Author: maribethb

src/actions/edit.ts

@@ -51,7 +51,8 @@ export class EditAction {
   private registerContextMenuAction() {
     const editAboveItem: ContextMenuRegistry.RegistryItem = {
       displayText: 'Edit Block contents (→︎)',
-      preconditionFn: (scope: ContextMenuRegistry.Scope) => {
+      preconditionFn: (scope: ContextMenuRegistry.Scope, menuOpenEvent) => {
+        if (menuOpenEvent instanceof PointerEvent) return 'hidden';
         const workspace = scope.block?.workspace;
         if (!workspace || !this.navigation.canCurrentlyNavigate(workspace)) {
           return 'disabled';

src/actions/insert.ts

@@ -71,7 +71,11 @@ export class InsertAction {
       displayText: () => {
         return 'Insert Block (I)';
       },
-      preconditionFn: (scope: ContextMenuRegistry.Scope) => {
+      preconditionFn: (
+        scope: ContextMenuRegistry.Scope,
+        menuOpenEvent: Event,
+      ) => {
+        if (menuOpenEvent instanceof PointerEvent) return 'hidden';
         let block;
         if (scope.focusedNode instanceof Blockly.Block) {
           block = scope.focusedNode;

src/actions/move.ts

@@ -127,9 +127,10 @@ export class MoveActions {
   menuItems: ContextMenuRegistry.RegistryItem[] = [
     {
       displayText: 'Move Block (M)',
-      preconditionFn: (scope) => {
+      preconditionFn: (scope, menuOpenEvent) => {
         const workspace = scope.block?.workspace as WorkspaceSvg | null;
-        if (!workspace) return 'hidden';
+        if (!workspace || menuOpenEvent instanceof PointerEvent)
+          return 'hidden';
         return this.mover.canMove(workspace) ? 'enabled' : 'disabled';
       },
       callback: (scope) => {