fixing-issue

Author: perminder-17

.vscode/settings.json

@@ -9,5 +9,6 @@
 	"editor.defaultFormatter": null,
 	"editor.formatOnSave": false,
   "editor.codeActionsOnSave": {},
-	"javascript.format.enable": false
+	"javascript.format.enable": false,
+	"liveServer.settings.port": 5501
 }

lib/empty-example/sketch.js

@@ -1,7 +1,25 @@
 function setup() {
-  // put setup code here
- }
- 
- function draw() {
-   // put drawing code here
- }
+  createCanvas(300, 200);
+  background(240);
+
+  textSize(32);
+  textAlign(CENTER,CENTER);
+  textFont('Georgia');
+
+  let txt = "Hello, \nWorld!";
+  // Compute the bounding box based on the font's intrinsic metrics
+  let bounds = fontBounds(txt, 50, 50);
+
+  fill(0);
+  text(txt, 50, 50);
+
+  noFill();
+  stroke('green');
+  rect(bounds.x, bounds.y, bounds.w, bounds.h);
+
+  noStroke();
+  fill(50);
+  textSize(15);
+  text("Font Bounds: x=" + bounds.x.toFixed(1) + ", y=" + bounds.y.toFixed(1) +
+       ", w=" + bounds.w.toFixed(1) + ", h=" + bounds.h.toFixed(1), 78, 100);
+}

src/type/textCore.js

@@ -1905,12 +1905,15 @@ function textCore(p5, fn) {
 
     // adjust the bounding boxes based on horiz. text alignment
     if (lines.length > 1) {
-      // Call the 2D mode version: the WebGL mode version does additional
-      // alignment adjustments to account for how WebGL renders text.
-      boxes.forEach(bb =>
-        bb.x += p5.Renderer2D.prototype._xAlignOffset
-          .call(this, textAlign, width)
-      );
+      // When width is not provided (e.g., fontBounds path), fall back to the widest line.
+      const maxWidth = boxes.length
+      ? boxes.reduce((m, b) => Math.max(m, b.w || 0), 0)
+      : 0;
+
+      boxes.forEach((bb) => {
+          const w = (width ?? maxWidth);
+          bb.x += p5.Renderer2D.prototype._xAlignOffset.call(this, textAlign, w);
+        });
     }
 
     // adjust the bounding boxes based on vert. text alignment

test/unit/type/p5.Font.js

@@ -40,6 +40,16 @@ suite('p5.Font', function () {
     assert.property(bbox, 'h');
   });
 
+   test('fontBounds no NaN (multiline + CENTER)', async () => {
+        const pFont = await myp5.loadFont(fontFile);
+        myp5.textAlign(myp5.CENTER, myp5.CENTER);
+        const b = pFont.fontBounds('Hello,\nWorld!', 50, 50, 24);
+        expect(b.x).not.toBeNaN();
+        expect(b.y).not.toBeNaN();
+        expect(b.w).not.toBeNaN();
+        expect(b.h).not.toBeNaN();
+      });
+
   suite('textToPoints', () => {
     test('contains no NaNs', async () => {
       const pFont = await myp5.loadFont(fontFile);