Update to handle mid-push updates

Author: davepagurek

src/core/p5.Renderer3D.js

@@ -160,6 +160,7 @@ export class Renderer3D extends Renderer {
 
     // clipping
     this._clipDepths = [];
+    this._textContextSavedStack = [];
     this._isClipApplied = false;
     this._stencilTestOn = false;
 
@@ -1329,9 +1330,11 @@ export class Renderer3D extends Renderer {
 
   push() {
     super.push()
-    if (this.states.textFont?.font) {
-      this.textDrawingContext()?.save()
+    const saved = !!(this.states.textFont?.font);
+    if (saved) {
+      this.textDrawingContext().save()
     }
+    this._textContextSavedStack.push(saved);
   }
 
   pop(...args) {
@@ -1341,8 +1344,8 @@ export class Renderer3D extends Renderer {
     ) {
       this._clearClip();
     }
-    if (this.states.textFont?.font) {
-      this.textDrawingContext()?.restore()
+    if (this._textContextSavedStack.pop()) {
+      this.textDrawingContext().restore()
     }
     super.pop(...args);
     this._applyStencilTestIfClipping();

test/unit/webgl/p5.RendererGL.js

@@ -3016,5 +3016,20 @@ suite('p5.RendererGL', function() {
 
       expect(widthAt20).toBeGreaterThan(widthAt12);
     });
+
+    test('fontWidth restores correctly when font is unset inside push/pop', async function() {
+      myp5.createCanvas(100, 100, myp5.WEBGL);
+      const font = await myp5.loadFont('test/unit/assets/acmesa.ttf');
+      myp5.textFont(font);
+      myp5.textSize(12);
+      myp5.push();
+      myp5.textFont('sans-serif'); // unset loaded font
+      myp5.pop();
+      // After pop, should be back to size 12 with loaded font
+      const widthAfterPop = myp5.fontWidth('X');
+      myp5.textSize(20);
+      const widthAt20 = myp5.fontWidth('X');
+      expect(widthAfterPop).toBeLessThan(widthAt20);
+    });
   });
 });