From 6565948ab166e180b4a13916f61ca65970343e4e Mon Sep 17 00:00:00 2001
From: Joshua Newell <3116028+imjosh@users.noreply.github.com>
Date: Wed, 22 Jan 2025 18:14:44 -0500
Subject: [PATCH] Fix cell.js getTextDimensions doesn't respect
 lineHeightFactor option

---
 src/modules/cell.js     | 17 +++++++++++++----
 test/specs/cell.spec.js | 12 ++++++++++++
 2 files changed, 25 insertions(+), 4 deletions(-)

diff --git a/src/modules/cell.js b/src/modules/cell.js
index 6ddaafeb1..fb2f9f37b 100644
--- a/src/modules/cell.js
+++ b/src/modules/cell.js
@@ -171,10 +171,17 @@ import { jsPDF } from "../jspdf.js";
   };
 
   /**
+   * Gets height and width of text
    * @name getTextDimensions
    * @function
    * @param {string} txt
-   * @returns {Object} dimensions
+   * @param {object} [options]
+   * @param {string} [options.font] Font name or family. Example: "times"
+   * @param {number} [options.fontSize] Font size in points
+   * @param {number} [options.lineHeightFactor=1.15]  The lineheight of each line
+   * @param {number} [options.maxWidth=0] Split the text by given width, 0 = no split
+   * @param {number} [options.scaleFactor] Defaults to value determined by unit declared at inception of PDF  document
+   * @returns {object} `{ w: ${width}, h: ${height} }` (in units declared at inception of PDF document)
    */
   jsPDFAPI.getTextDimensions = function(text, options) {
     _initialize.call(this);
@@ -182,6 +189,9 @@ import { jsPDF } from "../jspdf.js";
     var fontSize = options.fontSize || this.getFontSize();
     var font = options.font || this.getFont();
     var scaleFactor = options.scaleFactor || this.internal.scaleFactor;
+    var lineHeightFactor =
+      options.lineHeightFactor || this.internal.getLineHeightFactor();
+    const maxWidth = options.maxWidth || 0;
     var width = 0;
     var amountOfLines = 0;
     var height = 0;
@@ -198,7 +208,6 @@ import { jsPDF } from "../jspdf.js";
       }
     }
 
-    const maxWidth = options.maxWidth;
     if (maxWidth > 0) {
       if (typeof text === "string") {
         text = this.splitTextToSize(text, maxWidth);
@@ -225,8 +234,8 @@ import { jsPDF } from "../jspdf.js";
 
     width = width / scaleFactor;
     height = Math.max(
-      (amountOfLines * fontSize * this.getLineHeightFactor() -
-        fontSize * (this.getLineHeightFactor() - 1)) /
+      (amountOfLines * fontSize * lineHeightFactor -
+        fontSize * (lineHeightFactor - 1)) /
         scaleFactor,
       0
     );
diff --git a/test/specs/cell.spec.js b/test/specs/cell.spec.js
index 19b1a85c1..9316361b9 100644
--- a/test/specs/cell.spec.js
+++ b/test/specs/cell.spec.js
@@ -20,12 +20,24 @@ describe("Module: Cell", () => {
     expect(
       doc.getTextDimensions("Octocat loves jsPDF", { maxWidth: 150 }).h
     ).toEqual(16);
+    expect(
+      doc.getTextDimensions("Octocat loves jsPDF", { maxWidth: 150, lineHeightFactor: 2 }).h
+    ).toEqual(16);
     expect(
       doc.getTextDimensions("Octocat loves jsPDF", { maxWidth: 100 }).h
     ).toEqual(34.4);
+    expect(
+      doc.getTextDimensions("Octocat loves jsPDF", { maxWidth: 100, lineHeightFactor: undefined }).h
+    ).toEqual(34.4);
+    expect(
+      doc.getTextDimensions("Octocat loves jsPDF", { maxWidth: 100, lineHeightFactor: 2 }).h
+    ).toEqual(48);
     expect(
       doc.getTextDimensions("Octocat loves jsPDF", { maxWidth: 100 }).w
     ).toEqual(96.64000000000001);
+    expect(
+      doc.getTextDimensions("Octocat loves jsPDF", { maxWidth: 100, lineHeightFactor: 2 }).w
+    ).toEqual(96.64000000000001);
     expect(
       doc.getTextDimensions("Octocat loves jsPDF\njsPDF loves Octocat", { maxWidth: 100 }).h
     ).toEqual(71.19999999999999);