ntk 8.12.1 → 8.12.2

package/lib/renderingcontext_2d.js

@@ -2716,7 +2716,14 @@ class RenderingContext2d {
         if (rect.w === 0 || rect.h === 0) return;
         this._setPictureClip(rect);
       }
-      drawGlyphRuns(app, op, src.id, this._dst(), positioned);
+      drawGlyphRuns(
+        app,
+        op,
+        src.id,
+        this._dst(),
+        positioned,
+        rect ?? { x: 0, y: 0, w: this.width, h: this.height },
+      );
       if (rect) this._invalidatePictureClip();
       this._markDirty();
       return;
@@ -2737,6 +2744,7 @@ class RenderingContext2d {
       this._glyphSource.id,
       this.fillMask.id,
       positioned,
+      { x: 0, y: 0, w: this.width, h: this.height },
     );
     R.Composite(
       R.PictOp.In,

package/lib/text/glyphs.js

@@ -571,8 +571,11 @@ export function routeGlyphSize(app, font, size, policy = policyOf(app), textRend
  * @param {number} srcId source picture (fill color/gradient)
  * @param {number} dstId destination picture
  * @param {Array<{run, x, y}>} positioned visual-order runs with baseline origins
+ * @param {{x: number, y: number, w: number, h: number}} [bounds] the part of
+ *   the destination that can show anything — the clip, or the surface. A
+ *   glyph wholly outside it is not sent (`visibleGlyph`).
  */
-export function drawGlyphRuns(app, op, srcId, dstId, positioned) {
+export function drawGlyphRuns(app, op, srcId, dstId, positioned, bounds = null) {
   const policy = policyOf(app);
   let bitmap = positioned;
   let vector = null;
@@ -590,11 +593,35 @@ export function drawGlyphRuns(app, op, srcId, dstId, positioned) {
       bitmap.push(positioned[i]);
     }
   }
-  if (bitmap.length) drawBitmapGlyphRuns(app, op, srcId, dstId, bitmap, policy);
-  if (vector) drawVectorGlyphRuns(app, op, srcId, dstId, vector);
+  if (bitmap.length) drawBitmapGlyphRuns(app, op, srcId, dstId, bitmap, policy, bounds);
+  if (vector) drawVectorGlyphRuns(app, op, srcId, dstId, vector, bounds);
 }
 
-function drawBitmapGlyphRuns(app, op, srcId, dstId, positioned, policy) {
+/**
+ * Whether a glyph whose origin is at (x, y) can put any ink inside `bounds`,
+ * give or take two ems — no glyph reaches further from its origin than that,
+ * swashes and all.
+ *
+ * Not only a saving: the glyphs of a line wider than 32767 device pixels —
+ * a minified file in a code editor, a long label in a narrow cell — have
+ * origins no request can carry. CompositeGlyphs places its first glyph and
+ * every element after it in 16-bit coordinates, and the vector path's trap
+ * mask would span the whole line, so drawing such a line threw a RangeError
+ * out of the paint however little of it was on screen. What is left after
+ * the cull sits inside a surface, and every coordinate in it fits.
+ */
+function visibleGlyph(bounds, x, y, size) {
+  if (bounds === null) return true;
+  const reach = size * 2;
+  return (
+    x + reach > bounds.x &&
+    x - reach < bounds.x + bounds.w &&
+    y + reach > bounds.y &&
+    y - reach < bounds.y + bounds.h
+  );
+}
+
+function drawBitmapGlyphRuns(app, op, srcId, dstId, positioned, policy, bounds) {
   const Render = app.display.Render;
   const items = [];
   let bits = 8;
@@ -608,6 +635,7 @@ function drawBitmapGlyphRuns(app, op, srcId, dstId, positioned, policy) {
     if (page.bits > bits) bits = page.bits;
   }
   for (const pos of positionGlyphs(positioned)) {
+    if (!visibleGlyph(bounds, pos.x, pos.y, pos.run.size)) continue;
     const page = pages.get(pos.run);
     const e = page.entry(pos.glyph.id);
     items.push({ gs: e.gs, lid: e.lid, adv: e.adv, x: pos.x, y: pos.y });
@@ -646,7 +674,7 @@ const MAX_TRAPS_PER_REQUEST = 6000;
  * Positions are intentionally NOT rounded to whole pixels — fractional
  * advances and origins keep zoom animations smooth.
  */
-function drawVectorGlyphRuns(app, op, srcId, dstId, positioned) {
+function drawVectorGlyphRuns(app, op, srcId, dstId, positioned, bounds = null) {
   // unrounded glyph origins + ink bounding box
   const placed = [];
   let minX = Infinity;
@@ -657,6 +685,7 @@ function drawVectorGlyphRuns(app, op, srcId, dstId, positioned) {
       const gx = cursor + g.dx;
       const gy = y - g.dy;
       cursor += g.ax;
+      if (!visibleGlyph(bounds, gx, gy, run.size)) continue;
       const e = run.font.glyphExtents(g.id, run.size);
       if (!Number.isFinite(e.minX) || e.maxX <= e.minX) continue; // blank glyph
       placed.push({ run, glyph: g, x: gx, y: gy });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ntk",
-  "version": "8.12.1",
+  "version": "8.12.2",
   "description": "Desktop UI toolkit for X11 with canvas-like 2d and OpenGL rendering",
   "author": "Andrey Sidorov <sidorares@yandex.ru>",
   "license": "MIT",