Prevent double transform application in clip() with absolute coordinates

Author: VANSH3104

src/core/p5.Renderer2D.js

@@ -305,7 +305,7 @@ class Renderer2D extends Renderer {
     // Start a new path. Everything from here on out should become part of this
     // one path so that we can clip to the whole thing.
     this.clipPath = new Path2D();
-    this.initialClipTransform = this.drawingContext.getTransform().inverse();
+    this.initialTransform = this.drawingContext.getTransform();
 
     if (this._clipInvert) {
       // Slight hack: draw a big rectangle over everything with reverse winding
@@ -331,7 +331,10 @@ class Renderer2D extends Renderer {
   }
 
   endClip() {
+    const currentTransform = this.drawingContext.getTransform();
+    this.drawingContext.setTransform(1, 0, 0, 1, 0, 0);
     this.drawingContext.clip(this.clipPath);
+    this.drawingContext.setTransform(currentTransform);
     this.clipPath = null;
 
     super.endClip();
@@ -728,25 +731,34 @@ class Renderer2D extends Renderer {
       radiusY = h / 2;
     if (this._clipping) {
       const tempPath = new Path2D();
-      tempPath.ellipse(centerX, centerY, radiusX, radiusY, 0, 0, 2 * Math.PI);
-
-      const currentTransform = this.drawingContext.getTransform();
-      const initialClip = this.initialClipTransform;
-      if (currentTransform.a < 0 || currentTransform.d < 0) {
-        const fixedTransform = new DOMMatrix([
-          Math.abs(currentTransform.a), currentTransform.b,
-          currentTransform.c, Math.abs(currentTransform.d),
-          currentTransform.e, currentTransform.f
-        ]);
-        const relativeTransform = initialClip.multiply(fixedTransform);
-        this.clipPath.addPath(tempPath, relativeTransform);
-      } else {
-        // Normal case
-        const relativeTransform = initialClip.multiply(currentTransform);
-        this.clipPath.addPath(tempPath, relativeTransform);
-      }
+      const current = this.drawingContext.getTransform();
+      // Transform coordinates manually but preserve scale signs
+      const transformPoint = (x, y) => {
+        return {
+          x: current.a * x + current.c * y + current.e,
+          y: current.b * x + current.d * y + current.f
+        };
+      };
+      const transformedCenter = transformPoint(centerX, centerY);
+      // Calculate transformed radii WITHOUT Math.abs() to preserve negative scaling
+      const scaleX = Math.sqrt(current.a * current.a + current.c * current.c);
+      const scaleY = Math.sqrt(current.b * current.b + current.d * current.d);
+
+      // Preserve the sign of the scaling for mirroring effects
+      const signX = current.a < 0 ? -1 : 1;
+      const signY = current.d < 0 ? -1 : 1;
+
+      const transformedRadiusX = scaleX * radiusX * signX;
+      const transformedRadiusY = scaleY * radiusY * signY;
+      tempPath.ellipse(
+        transformedCenter.x,
+        transformedCenter.y,
+        Math.abs(transformedRadiusX),
+        Math.abs(transformedRadiusY),
+        0, 0, 2 * Math.PI
+      );
+      this.clipPath.addPath(tempPath);
     } else {
-      // Normal drawing (existing code)
       ctx.beginPath();
       ctx.ellipse(centerX, centerY, radiusX, radiusY, 0, 0, 2 * Math.PI);
       ctx.closePath();