Merge pull request #4366 from arekouzounian/master

Fix for #2488. Use element offsetHeight rather than getBoundingRect
xtermjs · Aug 1, 2023 · ce3bb35 · ce3bb35
2 parents 2d81490 + 5939d72
commit ce3bb35
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/src/browser/services/CharSizeService.ts b/src/browser/services/CharSizeService.ts
@@ -77,7 +77,10 @@ class DomMeasureStrategy implements IMeasureStrategy {
     this._measureElement.style.fontSize = `${this._optionsService.rawOptions.fontSize}px`;
 
     // Note that this triggers a synchronous layout
-    const geometry = this._measureElement.getBoundingClientRect();
+    const geometry = {
+      height: Number(this._measureElement.offsetHeight),
+      width: Number(this._measureElement.offsetWidth)
+    };
 
     // If values are 0 then the element is likely currently display:none, in which case we should
     // retain the previous value.