Merge pull request #5012 from Tyriar/tyriar/208102

Only scale when over 50% of following cell
xtermjs · Mar 19, 2024 · bee4fd0 · bee4fd0
2 parents 9ec9dca + f8ae7ed
commit bee4fd0
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/browser/renderer/shared/RendererUtils.ts b/src/browser/renderer/shared/RendererUtils.ts
@@ -48,9 +48,9 @@ export function allowRescaling(codepoint: number | undefined, width: number, gly
   return (
     // Is single cell width
     width === 1 &&
-    // Glyph exceeds cell bounds, add 25% to avoid hurting readability by rescaling glyphs that
+    // Glyph exceeds cell bounds, add 50% to avoid hurting readability by rescaling glyphs that
     // barely overlap
-    glyphSizeX > deviceCellWidth * 1.25 &&
+    glyphSizeX > Math.ceil(deviceCellWidth * 1.5) &&
     // Never rescale emoji
     codepoint !== undefined && !isEmoji(codepoint) &&
     // Never rescale powerline or nerd fonts