rescale monospace base width to 63

closes #1879
observablehq · Sep 28, 2023 · c3d047d · c3d047d
1 parent ebe85f2
commit c3d047d
Show file tree

Hide file tree

Showing 3 changed files with 30 additions and 30 deletions.
diff --git a/src/marks/text.js b/src/marks/text.js
@@ -409,7 +409,7 @@ export function defaultWidth(text, start = 0, end = text.length) {
 export function monospaceWidth(text, start = 0, end = text.length) {
   let sum = 0;
   for (let i = start; i < end; i = readCharacter(text, i)) {
-    sum += isPictographic(text, i) ? 200 : 100;
+    sum += isPictographic(text, i) ? 126 : 63;
   }
   return sum;
 }

diff --git a/test/output/textOverflow.svg b/test/output/textOverflow.svg
diff --git a/test/plots/text-overflow.ts b/test/plots/text-overflow.ts
@@ -71,7 +71,7 @@ export async function textOverflow() {
         fx: () => "monospace",
         monospace: true,
         textOverflow: "ellipsis-end",
-        lineWidth: 13,
+        lineWidth: 8,
         dx: -4,
         frameAnchor: "right"
       }),
@@ -117,7 +117,7 @@ async function textOverflowPlot(textOverflow, {monospace = false} = {}) {
     color: {domain: opinions, scheme: "rdylbu"},
     marks: [
       Plot.axisX({monospace}),
-      Plot.axisY({lineWidth: monospace ? 9 : 6, textOverflow, monospace}),
+      Plot.axisY({lineWidth: 6, textOverflow, monospace}),
       Plot.axisY({
         anchor: "right",
         tickFormat: (name) => `${dates.get(name).getUTCFullYear()}`,