Skip to content

Add ligature support to terminal#120452

Closed
Tyriar wants to merge 4 commits intomainfrom
tyriar/ligatures
Closed

Add ligature support to terminal#120452
Tyriar wants to merge 4 commits intomainfrom
tyriar/ligatures

Conversation

@Tyriar
Copy link
Copy Markdown
Contributor

@Tyriar Tyriar commented Apr 2, 2021

Part of #34103

@deepak1556 see main/app

Todo:

@Tyriar Tyriar added this to the April 2021 milestone Apr 2, 2021
@Tyriar Tyriar requested a review from deepak1556 April 2, 2021 23:21
@Tyriar Tyriar self-assigned this Apr 2, 2021
Can't disable without reload right now due to xtermjs/xterm.js#3289
'force-color-profile',

// Enable experimental font access for terminal ligatures
`enable-features='FontAccess'`
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Did this work ? I would expect appendSwitch to ignore this, it should instead be

app.commandLine.appendSwitch('enable-features', 'FontAccess');

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yep it worked, can change it though

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lets keep it separate for consistency.

@Tyriar Tyriar modified the milestones: April 2021, May 2021 Apr 28, 2021
@Tyriar Tyriar modified the milestones: May 2021, On Deck Jun 4, 2021
@meganrogge meganrogge modified the milestones: On Deck, October 2021 Oct 1, 2021
@Tyriar Tyriar modified the milestones: October 2021, On Deck Oct 22, 2021
@Tyriar Tyriar modified the milestones: On Deck, December 2021 Dec 8, 2021
Tyriar added a commit that referenced this pull request Dec 20, 2021
Migrating over from stale #120452

Part of #34103
@Tyriar
Copy link
Copy Markdown
Contributor Author

Tyriar commented Dec 22, 2021

Closing in favor of #139537

@Tyriar Tyriar closed this Dec 22, 2021
@github-actions github-actions bot locked and limited conversation to collaborators Feb 5, 2022
@Tyriar Tyriar deleted the tyriar/ligatures branch December 15, 2022 18:44
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants