Handle undefined cbox in Glyph metrics calculation

ptoussai · lehni · commit 8aef05837763 · 2025-03-26T18:09:16.000+01:00
Fix issue with foliojs#306
diff --git a/src/glyph/Glyph.js b/src/glyph/Glyph.js
@@ -68,6 +68,7 @@ export default class Glyph {
     if (this._font.vmtx) {
       var {advance:advanceHeight, bearing:topBearing} = this._getTableMetrics(this._font.vmtx);
     } else {
+      if (typeof cbox === 'undefined' || cbox === null) { ({ cbox } = this); }
       var advanceHeight = Math.abs(this._font.ascent - this._font.descent);
       var topBearing = this._font.ascent - cbox.maxY;
     }

Original file line number	Diff line number	Diff line change
`@@ -68,6 +68,7 @@ export default class Glyph {`
`68`	`68`	`if (this._font.vmtx) {`
`69`	`69`	`var {advance:advanceHeight, bearing:topBearing} = this._getTableMetrics(this._font.vmtx);`
`70`	`70`	`} else {`
	`71`	`+ if (typeof cbox === 'undefined' \|\| cbox === null) { ({ cbox } = this); }`
`71`	`72`	`var advanceHeight = Math.abs(this._font.ascent - this._font.descent);`
`72`	`73`	`var topBearing = this._font.ascent - cbox.maxY;`
`73`	`74`	`}`