-
Notifications
You must be signed in to change notification settings - Fork 29.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support ligatures in terminal #34103
Comments
Hi! is there any timeline for this feature? |
Nope. Here's the upstream issue xtermjs/xterm.js#958 |
Would be really nice if there was a setting to re-enable the old terminal rendering until all the issues with the new terminal are resolved (blurriness, ligatures, emoji). 😕 |
It seems there is now an How can we get it into vscode? |
@MPV it would have to be built into the core, the main thing that needs to happen now is to evaluate how much weight the addon would add to the VS Code distributable. |
@Phavor I don't think your issue is about ligatures but rather your powerlines setup https://code.visualstudio.com/docs/editor/integrated-terminal#_can-i-use-powerline-fonts-in-the-integrated-terminal |
@Tyriar I fixed though. Thanks. This line did the job |
any news on it? I believe this is the last thing is missing for terminal to render everything proper |
Just tried Cascadia Code font in the integrated terminal... ligatures not present 😢 |
I see that there's a way to do this via an extension that modifies VS Code's internal files. Would it be possible to add a checkbox in settings that toggles this itself? @Tyriar |
@jhpratt that's what it would do eventually, the remaining work here is slimming down the ligatures xterm.js addon to bare essentials and test/include it. I don't have time to do it right now due to other priorities though. |
Is this bug why I got this in vscode: (using this font https://github.com/adam7/delugia-code)? |
@Geobert probably caused by xtermjs/xterm.js#1709 or xtermjs/xterm.js#942, ligatures are different to emoji. |
Got a bunch of questions about ligatures when tweeting about the WebGL renderer so I'll give an update here. Here's the remaining work to do on this:
|
@Tyriar is there a repro of the bundling issue? I feel like that would be a good starting point for those that want to help here |
Giving it another shot in #232829 |
Looks like this should land as a preview feature in the November release! #232829 Enable it with |
Wow amazing! |
Finally |
Just to confirm, this setting only accepts a boolean value and does not enable explicit font-feature-settings like // settings.json
// Works
"terminal.integrated.fontLigatures": true
// Won't work as expected at this time?
"terminal.integrated.fontLigatures": "'calt', 'liga', 'ss19', 'cv69'" Compare editorOptions.ts to terminalConfiguration.ts. |
@joshlawton yes it's all or nothing right now. If someone can figure out how to pass that information on to an OffscreenCanvas it should be trivial to add the support: https://developer.mozilla.org/en-US/docs/Web/API/OffscreenCanvasRenderingContext2D#text_styles Created #235316 to track |
@Master-Hash yes that's by design in that particular case since pwsh is coloring is differently |
@sunshaoce tracked in #235988 👍 |
Regresses with performance improvements in terminal's renderer #33954
The text was updated successfully, but these errors were encountered: