Skip to content
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

Update xterm.js #162855

Merged
merged 2 commits into from
Oct 6, 2022
Merged

Update xterm.js #162855

merged 2 commits into from
Oct 6, 2022

Conversation

Tyriar
Copy link
Member

@Tyriar Tyriar commented Oct 6, 2022

Fixes #158984
Fixes #158874


- Bump copyright year xtermjs/xterm.js#4176
- Share texture atlas cache code between webgl and canvas renderers xtermjs/xterm.js#4170
- Add willReadFrequently to canvas renderer too xtermjs/xterm.js#4169
- Ensure texture atlas comparison uses rgba not object xtermjs/xterm.js#4168
- Create new event with emitter object to simplify code xtermjs/xterm.js#4166
- Define all events and emitters consistently xtermjs/xterm.js#4165
- Inline dirty row service into input handler xtermjs/xterm.js#4163
- Move w objects to $ prefix variables xtermjs/xterm.js#4162

Fixes #158984
Fixes #158874
@Tyriar Tyriar added this to the October 2022 milestone Oct 6, 2022
@Tyriar Tyriar requested a review from meganrogge October 6, 2022 16:19
@Tyriar Tyriar self-assigned this Oct 6, 2022
@Tyriar Tyriar enabled auto-merge October 6, 2022 16:27
@Tyriar Tyriar disabled auto-merge October 6, 2022 16:36
@Tyriar
Copy link
Member Author

Tyriar commented Oct 6, 2022

Need to fix an internal API access issue

meganrogge
meganrogge previously approved these changes Oct 6, 2022
@Tyriar Tyriar marked this pull request as draft October 6, 2022 16:51
@Tyriar
Copy link
Member Author

Tyriar commented Oct 6, 2022

Blocked on xtermjs/xterm.js#4177

This includes a revert to event refactor which will fix vscode's
internal access. It was going to be reverted anyway.
@Tyriar Tyriar marked this pull request as ready for review October 6, 2022 17:38
@Tyriar Tyriar requested a review from meganrogge October 6, 2022 17:38
@Tyriar Tyriar merged commit 55be96c into main Oct 6, 2022
@Tyriar Tyriar deleted the tyriar/xterm2 branch October 6, 2022 19:34
@github-actions github-actions bot locked and limited conversation to collaborators Nov 20, 2022
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.

With canvas, should the line be continuous? Terminal: Underline with canvas uses less spacing
2 participants