Initial support for using range.getClientRects to measure text. by nightwing · Pull Request #5922 · ajaxorg/ace

nightwing · 2026-02-22T17:57:41Z

xyos

lgtm

github-actions · 2026-05-02T08:37:38Z

One of the public type files has been updated, plase make sure there are no backwards incompatible changes done in the PR.

codecov · 2026-05-02T08:39:05Z

Codecov Report

❌ Patch coverage is 94.21687% with 48 lines in your changes missing coverage. Please review.
✅ Project coverage is 93.95%. Comparing base (f82010e) to head (bd38fd5).
⚠️ Report is 3 commits behind head on master.

Files with missing lines	Patch %	Lines
src/layer/font_metrics.js	93.62%	34 Missing ⚠️
src/virtual_renderer.js	89.01%	10 Missing ⚠️
src/ext/diff/inline_diff_view.js	77.77%	2 Missing ⚠️
src/ace_test.js	96.29%	1 Missing ⚠️
src/layer/marker.js	93.75%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #5922      +/-   ##
==========================================
+ Coverage   93.73%   93.95%   +0.21%     
==========================================
  Files         641      647       +6     
  Lines      138167   140305    +2138     
  Branches    14561    14706     +145     
==========================================
+ Hits       129510   131820    +2310     
+ Misses       8657     8485     -172

Flag	Coverage Δ
unittests	`93.95% <94.21%> (+0.21%)`	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

github-actions · 2026-05-02T09:05:10Z

One of the public type files has been updated, plase make sure there are no backwards incompatible changes done in the PR.

github-actions · 2026-05-03T14:06:29Z

One of the public type files has been updated, plase make sure there are no backwards incompatible changes done in the PR.

github-actions · 2026-05-04T10:03:10Z

One of the public type files has been updated, plase make sure there are no backwards incompatible changes done in the PR.

github-actions · 2026-05-05T14:08:50Z

One of the public type files has been updated, plase make sure there are no backwards incompatible changes done in the PR.

… calculations # Conflicts: # src/virtual_renderer.js

github-actions · 2026-07-06T18:04:04Z

One of the public type files has been updated, plase make sure there are no backwards incompatible changes done in the PR.

github-actions · 2026-07-08T17:24:01Z

One of the public type files has been updated, plase make sure there are no backwards incompatible changes done in the PR.

github-actions · 2026-07-09T15:19:52Z

One of the public type files has been updated, plase make sure there are no backwards incompatible changes done in the PR.

github-actions · 2026-07-10T06:24:12Z

One of the public type files has been updated, plase make sure there are no backwards incompatible changes done in the PR.

github-actions · 2026-07-10T07:03:08Z

One of the public type files has been updated, plase make sure there are no backwards incompatible changes done in the PR.

github-actions · 2026-07-10T14:35:42Z

One of the public type files has been updated, plase make sure there are no backwards incompatible changes done in the PR.

github-actions · 2026-07-10T14:36:43Z

One of the public type files has been updated, plase make sure there are no backwards incompatible changes done in the PR.

github-actions · 2026-07-10T15:42:33Z

One of the public type files has been updated, plase make sure there are no backwards incompatible changes done in the PR.

github-actions · 2026-07-10T16:23:57Z

One of the public type files has been updated, plase make sure there are no backwards incompatible changes done in the PR.

github-actions · 2026-07-12T23:07:58Z

One of the public type files has been updated, plase make sure there are no backwards incompatible changes done in the PR.

xyos

Nice work, this is awesome!

I built #5988 on an earlier snapshot of this branch, so below are a few things I hit while testing grapheme/bidi edge cases against real browser measurements. Most are small and cherry-pickable from #5988. Details inline.

Once this merges I'd like us to review and maybe do a follow-up PR picking up the rest of #5988: the grapheme-cluster model (Intl.Segmenter-based helpers in lib/lang, cluster-snapping moveCursorTo, cluster-aware $getDisplayTokens/$computeWrapSplits, $alignTokensToClusters). It touches the same code this PR rewrites, so rebasing it on top of merged master is cleaner than growing this PR's scope. With the current branch a wrap limit crossing 👨‍👩‍👧‍👦 still splits its 11 code units across screen rows, and moveCursorTo only snaps surrogate pairs, not ZWJ/combining-mark clusters; the follow-up addresses #460, #4142, #5431, #4602, #3753, #3866, #3617.

xyos · 2026-07-13T20:26:32Z

+                        maxDistance = d;
+                    }
+                }
+                if (blockCursor) {


if (blockCursor) { // TODO } is a no-op, so vim block-cursor mode gets round-to-nearest instead of floor semantics. Either implement or drop the branch and note the limitation.

xyos · 2026-07-13T20:26:32Z


        var cursorTop = (pos.row - (onScreen ? this.config.firstRowScreen : 0)) *
            this.config.lineHeight;
+        var cursorWidth = this.config.fontMetrics.textWidth(pos.row, pos.column + 1) - textWidth;


If textWidth(col+1) and textWidth(col) measure equal (zero-width cluster tail, or a measurement fallback), cursorWidth is 0 and the cursor becomes invisible. Suggest (... - textWidth) || this.config.characterWidth.

good point, done

xyos · 2026-07-13T20:26:32Z

-            this.updateBidiMap();
-        }
-        return this.bidiMap.bidiLevels;
+        return true;


isBidiRow now returns true for every row once the document contains any bidi character. One Hebrew word in a 10k-line file routes every marker on every line through drawBidiSingleLineMarker → getRects → DOM range measurement. A cheap per-line bidiRE.test(line) here would keep the fast path for the 99% of lines that aren't bidi.

The problem is that when calling isBidiRow we only have screenLine, so finding the line is quite expensive. drawBidiSingleLineMarker on the other hand is not very slow, so we end up with similar performance in both cases.

xyos · 2026-07-13T20:26:32Z

+            if (useFontMetrics && this.$desiredColumn == null) {
+                offsetX = fontMetrics.textWidth(screenPos.row, screenPos.column);
+                if (isFinite(offsetX))
+                    this.$desiredColumn = offsetX / fontMetrics.config.characterWidth;


$desiredColumn now stores a fractional pixel-derived value (offsetX / characterWidth). When the target row isn't rendered, $pixelToColumn returns it unchanged and a non-integer column flows into screenToDocumentPosition. It works in the cases I tested, but rounding the fallback would keep the column contract integer-valued.

I think we should allow fractional screenColumn for reasons similar to screenRow, also because fractional screenColumns were allowed in the old implementation of varchar https://github.com/ajaxorg/ace/blob/master/src/edit_session.js#L2507C19-L2507C34

xyos · 2026-07-13T20:26:32Z

        }

-        var rowCount = 10;
+        var rowCount = 5.5;


rowCount = 5.5 looks like a leftover from testing fractional row heights; intentional?

fractional row heights were intentional, without that all linewidgets are forced to have heights in multtiples of line height, and our api doesn't enforce that.

xyos · 2026-07-13T20:30:54Z

    }

+    /**
+     * Calculates the width of the text up to a specific scrrenColumn on a given screen row.


nit: typo, scrrenColumn (also ends up in the generated types/ace-modules.d.ts)

xyos · 2026-07-13T20:30:55Z

+        var m0 = 0.7, m1 = 0.1, m2 = 0.3, m3 = 0.82;
+        var t1 = 100, t2 = 20;
+        function testTransform() {
+            fontMetrics.config.$transformData = null; //FIXME


nit: remove this comment

xyos · 2026-07-13T20:31:18Z

         * @related EditSession.documentToScreenPosition
         **/
-        screenToDocumentPosition(screenRow: number, screenColumn: number, offsetX?: number): Point;
+        screenToDocumentPosition(screenRow: number, screenColumn: number): Point;


This is a public API break: screenToDocumentPosition loses its documented offsetX param, and BidiHandler loses getPosLeft/getSelections/offsetToCol/updateCharacterWidths. Embedders calling these will break at runtime, not just in types. Probably fine given the old bidi pixel model is gone, but worth a changelog note and a decision on whether this ships as a minor or major bump.

…5813)

github-actions · 2026-07-16T17:12:50Z

One of the public type files has been updated, plase make sure there are no backwards incompatible changes done in the PR.

github-actions · 2026-07-23T12:48:11Z

One of the public type files has been updated, plase make sure there are no backwards incompatible changes done in the PR.

xyos reviewed Feb 25, 2026

View reviewed changes

Comment thread src/layer/font_metrics.js Outdated

xyos reviewed Feb 25, 2026

View reviewed changes

Comment thread src/layer/font_metrics.js Outdated

xyos approved these changes Feb 26, 2026

View reviewed changes

nightwing force-pushed the varchar3 branch from 57a38ef to f9ab19a Compare March 23, 2026 23:51

nightwing mentioned this pull request Apr 27, 2026

Issues with Arabic Text Handling #5936

Closed

nightwing force-pushed the varchar3 branch 3 times, most recently from c7be8dd to 90b4eee Compare May 2, 2026 08:37

nightwing force-pushed the varchar3 branch from 5a08e66 to bed0438 Compare May 3, 2026 14:06

nightwing force-pushed the varchar3 branch from 2de6037 to 6138c2d Compare May 5, 2026 14:08

nightwing mentioned this pull request May 18, 2026

improve getBoundingClientRect in mockdom #5973

Merged

mkslanc and others added 8 commits June 9, 2026 17:09

WIP: add text width measurement and cursor positioning based on pixel…

5203c96

… calculations # Conflicts: # src/virtual_renderer.js

text width calculation based on createRange

982731e

refactor

3c0e0c0

test

30ee806

address review comments

66b07cb

fix most of failing tests

108fe73

fix rendering of cjk spaces with show invisibles

685d2af

fix textinput tests

d5684d1

xyos mentioned this pull request Jul 3, 2026

Grapheme-cluster aware cursor, hit-testing and soft wrap on top of #5922 #5988

Open

nightwing force-pushed the varchar3 branch from 6138c2d to d7b2cdd Compare July 6, 2026 18:03

css transform support in mockdom

037318f

nightwing force-pushed the varchar3 branch from d7b2cdd to a99a487 Compare July 8, 2026 17:23

nightwing force-pushed the varchar3 branch from a99a487 to 60ce0b1 Compare July 9, 2026 15:19

nightwing force-pushed the varchar3 branch from 60ce0b1 to 33d1040 Compare July 10, 2026 06:24

nightwing force-pushed the varchar3 branch from 33d1040 to 0e9e8be Compare July 10, 2026 07:03

nightwing force-pushed the varchar3 branch from 0e9e8be to fca477c Compare July 10, 2026 14:35

improve handling of bidi text

c6c9f87

nightwing force-pushed the varchar3 branch from fca477c to c6c9f87 Compare July 10, 2026 14:36

nightwing added 2 commits July 10, 2026 19:21

partial fix for horizontal scroll

7317a8f

update node in ci, since new typescript doesn't work with node 16

32ae297

fix multiline selection marker on wrapped bidi lines

40e5bdc

cleanup

8f9018c

xyos reviewed Jul 13, 2026

View reviewed changes

stop rendering U+200D as invalid invisible so ZWJ emoji stay intact (#…

8105fe2

…5813)

fix blockcursor

bd38fd5

nightwing merged commit 054a3d6 into master Jul 27, 2026
8 checks passed

nightwing deleted the varchar3 branch July 27, 2026 11:12

Uh oh!

Conversation

nightwing commented Feb 22, 2026 • edited by github-actions Bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

xyos left a comment

Choose a reason for hiding this comment

Uh oh!

github-actions Bot commented May 2, 2026

Uh oh!

codecov Bot commented May 2, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

github-actions Bot commented May 2, 2026

Uh oh!

github-actions Bot commented May 3, 2026

Uh oh!

github-actions Bot commented May 4, 2026

Uh oh!

github-actions Bot commented May 5, 2026

Uh oh!

github-actions Bot commented Jul 6, 2026

Uh oh!

github-actions Bot commented Jul 8, 2026

Uh oh!

github-actions Bot commented Jul 9, 2026

Uh oh!

github-actions Bot commented Jul 10, 2026

Uh oh!

github-actions Bot commented Jul 10, 2026

Uh oh!

github-actions Bot commented Jul 10, 2026

Uh oh!

github-actions Bot commented Jul 10, 2026

Uh oh!

github-actions Bot commented Jul 10, 2026

Uh oh!

github-actions Bot commented Jul 10, 2026

Uh oh!

github-actions Bot commented Jul 12, 2026

Uh oh!

xyos left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

nightwing commented Feb 22, 2026 •

edited by github-actions Bot

Loading

codecov Bot commented May 2, 2026 •

edited

Loading

xyos left a comment •

edited

Loading