Might be overlapping ranges here?

xtermjs/xterm.js#3998

Also (xtermjs/xterm.js#3303) still happens when toggling off and on:

@deepak1556 can you please review the permissions related parts in src/main.js, vs/codeandvs/platform/protocol`?

The long ===== is actually expected:

Here it is in Windows Terminal:

The width is correct there though.

We ceil the cell width, that's my guess of why the width thing doesn't happen. In order to workaround that we would need sub-pixel offset rendering and then we would hit more glyph overlap problems.

Unfortunately this is getting put on hold again this iteration due to the packaging issues I've been having. They're related to the font-ligatures package using node APIs and the way webpack is bundling it doesn't work inside sandboxed electron renderers. Here's the webpack config if anyone want to investigate https://github.com/xtermjs/xterm.js/blob/master/addons/xterm-addon-ligatures/webpack.config.js

We ceil the cell width, that's my guess of why the width thing doesn't happen. In order to workaround that we would need sub-pixel offset rendering and then we would hit more glyph overlap problems.

@Tyriar Hey I just came across this PR and saw your long-ligature problem (long ==== ligatures seemingly shrink shorter and shorter). I'm not sure whether you already solved it in this PR, but I just so happen to have worked on this exact problem just a few weeks ago when I added ligature support for the new Windows Terminal text renderer. If you have any questions or would like to collaborate on this in any way, please let me know. 🙂
This is the PR that fixes it: microsoft/terminal#13549 and this part is the relevant addition (for Direct2D however): https://github.com/microsoft/terminal/blob/dfa92559611f5c982f78cba73d123638a5e54227/src/renderer/atlas/AtlasEngine.r.cpp#L352-L365
advanceScale is simply ceil(glyphAdvance) / glyphAdvance - it's basically the approach that you already thought of and it seems to work pretty great so far.

@lhecker thanks for the info! Good to know you hit the same problem and upscale

Any updates?

oh my god it's happening

what's the current status on this?

I hope it passes asap

We got ChatGPT, and NVIDIA moved the Nasdaq by 2% on it's own, while this PR was open.

Let's hope it goes through this time 🙏

Let's hope it goes through this time 🙏

It won't. There are still conflicts.

Let's hope it goes through this time 🙏

It won't. There are still conflicts.

Rip, glad it is getting attention again though.

Sorry, but closing to clean up PRs as this is stale and would need a new PR when we retry this.