Fix bug when action conditions are not met but cursor is changed (#17414)

Co-authored-by: David Catuhe <david@catuhe.com>

Author: deltakosh

packages/dev/core/src/Actions/action.ts

@@ -65,6 +65,12 @@ export interface IAction {
      * @see https://www.babylonjs-playground.com/#1T30HR#0
      */
     then(action: IAction): IAction;
+
+    /**
+     * Internal only - Returns if the current condition allows to run the action
+     * @internal
+     */
+    _evaluateConditionForCurrentFrame(): boolean;
 }
 
 /**

packages/dev/core/src/Actions/actionManager.ts

@@ -245,7 +245,9 @@ export class ActionManager extends AbstractActionManager {
             const action = this.actions[index];
 
             if (action.trigger >= ActionManager.OnPickTrigger && action.trigger <= ActionManager.OnPointerOutTrigger) {
-                return true;
+                if (action._evaluateConditionForCurrentFrame()) {
+                    return true;
+                }
             }
         }
 
@@ -260,7 +262,9 @@ export class ActionManager extends AbstractActionManager {
             const action = this.actions[index];
 
             if (action.trigger >= ActionManager.OnPickTrigger && action.trigger <= ActionManager.OnPickUpTrigger) {
-                return true;
+                if (action._evaluateConditionForCurrentFrame()) {
+                    return true;
+                }
             }
         }