From 1a3a27b73bd31768b89083b57a0db6bc9e29c8e5 Mon Sep 17 00:00:00 2001
From: D8H <Davy.Helard@gmail.com>
Date: Tue, 17 Sep 2024 15:15:25 +0200
Subject: [PATCH] Fix custom object flipping (#6967)

Do not show in changelog
---
 .../Renderers/LegacyRenderedCustomObjectInstance.js  | 11 +++++++++--
 .../Renderers/RenderedCustomObjectInstance.js        |  9 ++++++---
 .../Renderers/RenderedSpriteInstance.js              | 12 ++++++++----
 3 files changed, 23 insertions(+), 9 deletions(-)

diff --git a/newIDE/app/src/ObjectsRendering/Renderers/LegacyRenderedCustomObjectInstance.js b/newIDE/app/src/ObjectsRendering/Renderers/LegacyRenderedCustomObjectInstance.js
index a4691a484a..a718076188 100644
--- a/newIDE/app/src/ObjectsRendering/Renderers/LegacyRenderedCustomObjectInstance.js
+++ b/newIDE/app/src/ObjectsRendering/Renderers/LegacyRenderedCustomObjectInstance.js
@@ -301,6 +301,9 @@ export default class LegacyRenderedCustomObjectInstance
       this._instance.getAngle()
     );
 
+    this._pixiObject.scale.x = 1;
+    this._pixiObject.scale.y = 1;
+
     // Opacity is not handled by 3D objects.
     // TODO Transform 3D objects according to their flipping.
     if (!this._isRenderedIn3D) {
@@ -308,8 +311,12 @@ export default class LegacyRenderedCustomObjectInstance
       const alphaForDisplay = Math.max(this._instance.getOpacity() / 255, 0.5);
       this._pixiObject.alpha = alphaForDisplay;
 
-      if (this._instance.isFlippedX()) this._pixiObject.scale.x *= -1;
-      if (this._instance.isFlippedY()) this._pixiObject.scale.y *= -1;
+      this._pixiObject.scale.x =
+        Math.abs(this._pixiObject.scale.x) *
+        (this._instance.isFlippedX() ? -1 : 1);
+      this._pixiObject.scale.y =
+        Math.abs(this._pixiObject.scale.y) *
+        (this._instance.isFlippedY() ? -1 : 1);
     }
   }
 
diff --git a/newIDE/app/src/ObjectsRendering/Renderers/RenderedCustomObjectInstance.js b/newIDE/app/src/ObjectsRendering/Renderers/RenderedCustomObjectInstance.js
index 5f3a931716..23292ed284 100644
--- a/newIDE/app/src/ObjectsRendering/Renderers/RenderedCustomObjectInstance.js
+++ b/newIDE/app/src/ObjectsRendering/Renderers/RenderedCustomObjectInstance.js
@@ -387,7 +387,6 @@ export default class RenderedCustomObjectInstance extends Rendered3DInstance
       this._pixiObject.rotation = RenderedInstance.toRad(
         this._instance.getAngle()
       );
-
       this._pixiObject.scale.x = 1;
       this._pixiObject.scale.y = 1;
     }
@@ -399,8 +398,12 @@ export default class RenderedCustomObjectInstance extends Rendered3DInstance
       const alphaForDisplay = Math.max(this._instance.getOpacity() / 255, 0.5);
       this._pixiObject.alpha = alphaForDisplay;
 
-      if (this._instance.isFlippedX()) this._pixiObject.scale.x *= -1;
-      if (this._instance.isFlippedY()) this._pixiObject.scale.y *= -1;
+      this._pixiObject.scale.x =
+        Math.abs(this._pixiObject.scale.x) *
+        (this._instance.isFlippedX() ? -1 : 1);
+      this._pixiObject.scale.y =
+        Math.abs(this._pixiObject.scale.y) *
+        (this._instance.isFlippedY() ? -1 : 1);
     }
   }
 
diff --git a/newIDE/app/src/ObjectsRendering/Renderers/RenderedSpriteInstance.js b/newIDE/app/src/ObjectsRendering/Renderers/RenderedSpriteInstance.js
index 802cb7287f..829c548358 100644
--- a/newIDE/app/src/ObjectsRendering/Renderers/RenderedSpriteInstance.js
+++ b/newIDE/app/src/ObjectsRendering/Renderers/RenderedSpriteInstance.js
@@ -123,8 +123,12 @@ export default class RenderedSpriteInstance extends RenderedInstance {
     const alphaForDisplay = Math.max(this._instance.getOpacity() / 255, 0.5);
     this._pixiObject.alpha = alphaForDisplay;
 
-    if (this._instance.isFlippedX()) this._pixiObject.scale.x *= -1;
-    if (this._instance.isFlippedY()) this._pixiObject.scale.y *= -1;
+    this._pixiObject.scale.x =
+      Math.abs(this._pixiObject.scale.x) *
+      (this._instance.isFlippedX() ? -1 : 1);
+    this._pixiObject.scale.y =
+      Math.abs(this._pixiObject.scale.y) *
+      (this._instance.isFlippedY() ? -1 : 1);
   }
 
   updateSprite(): boolean {
@@ -215,13 +219,13 @@ export default class RenderedSpriteInstance extends RenderedInstance {
   getOriginX(): number {
     if (!this._sprite || !this._pixiObject) return 0;
 
-    return this._sprite.getOrigin().getX() * this._pixiObject.scale.x;
+    return this._sprite.getOrigin().getX() * Math.abs(this._pixiObject.scale.x);
   }
 
   getOriginY(): number {
     if (!this._sprite || !this._pixiObject) return 0;
 
-    return this._sprite.getOrigin().getY() * this._pixiObject.scale.y;
+    return this._sprite.getOrigin().getY() * Math.abs(this._pixiObject.scale.y);
   }
 
   getDefaultWidth(): number {