Bounding box Gizmo + non uniform scaling + rotation (#16801)

follow up
https://forum.babylonjs.com/t/attaching-bounding-box-gizmo-to-mesh-causes-its-scale-to-change/58595/8

Author: CedricGuillemet

packages/dev/core/src/Gizmos/boundingBoxGizmo.ts

@@ -764,8 +764,26 @@ export class BoundingBoxGizmo extends Gizmo implements IBoundingBoxGizmo {
 
         return this._cornerMesh.clone();
     }
+
+    /**
+     * returns true if the combination of non uniform scaling and rotation of the attached mesh is not supported
+     * In that case, the matrix is skewed and the bounding box gizmo will not work correctly
+     * @returns True if the combination is not supported, otherwise false.
+     */
+    protected _hasInvalidNonUniformScaling() {
+        return (
+            this._attachedMesh?.parent instanceof TransformNode &&
+            this._attachedMesh?.parent.absoluteScaling.isNonUniformWithinEpsilon(0.001) &&
+            ((this._attachedMesh?.rotationQuaternion && !this._attachedMesh?.rotationQuaternion.equalsWithEpsilon(Quaternion.Identity(), Epsilon)) ||
+                this._attachedMesh?.rotation.equalsWithEpsilon(Vector3.Zero(), Epsilon) === false)
+        );
+    }
     protected override _attachedNodeChanged(value: Nullable<AbstractMesh>) {
         if (value) {
+            if (this._hasInvalidNonUniformScaling()) {
+                Logger.Warn("BoundingBoxGizmo controls are not supported on meshes with non-uniform scaling and rotation");
+                return;
+            }
             // Reset anchor mesh to match attached mesh's scale
             // This is needed to avoid invalid box/anchor position on first drag
             this._anchorMesh.scaling.setAll(1);
@@ -814,7 +832,7 @@ export class BoundingBoxGizmo extends Gizmo implements IBoundingBoxGizmo {
      * Updates the bounding box information for the Gizmo
      */
     public updateBoundingBox() {
-        if (this.attachedMesh) {
+        if (this.attachedMesh && !this._hasInvalidNonUniformScaling()) {
             PivotTools._RemoveAndStorePivotPoint(this.attachedMesh);
 
             // Store original parent