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

html: upgrade to xterm 5.4.0 #1303

Merged
merged 2 commits into from
Mar 5, 2024
Merged

html: upgrade to xterm 5.4.0 #1303

merged 2 commits into from
Mar 5, 2024

Conversation

mikelorant
Copy link
Contributor

Upgrade xterm.js to the latest stable release.

As part of this upgrade, the xterm packages have been scoped as the unscoped packages were not updated.

The addons have also been upgraded to match the compatible versions.

As part of this upgrade, a compatibility issue was identified with the Gulp task to inline source. By disabling compression, this allowed the task to complete successfully. The size of the source file html.h did not change significantly which indicates compression was not being applied previously.

@mikelorant mikelorant changed the title html: Upgrade xterm.js Upgrade xterm.js Mar 5, 2024
@mikelorant
Copy link
Contributor Author

@tsl0922 Can you please trigger the workflow.

@mikelorant
Copy link
Contributor Author

This upgrade is necessary as this will be followed by a pull request to add Unicode 11 support. This will significantly improve displaying of emojis.

https://github.com/mikelorant/ttyd/commit/6ab15260575940289ca4a5365e050a39a634015f

Upgrade xterm.js to the latest stable release.

As part of this upgrade, the xterm packages have been scoped as the
unscoped packages were not updated.

The addons have also been upgraded to match the compatible versions.

As part of this upgrade, a compatibility issue was identified with the
Gulp task to inline source. By disabling compression, this allowed the
task to complete successfully. The size of the source file `html.h` did
not change significantly which indicates compression was not being
applied previously.

Signed-off-by: Michael Lorant <michael.lorant@nine.com.au>
@mikelorant
Copy link
Contributor Author

@tsl0922 Lint warnings have been resolved.

@tsl0922 tsl0922 changed the title Upgrade xterm.js html: upgrade to xterm 5.4.0 Mar 5, 2024
@tsl0922 tsl0922 merged commit 09e7fd8 into tsl0922:main Mar 5, 2024
12 checks passed
@mikelorant mikelorant deleted the build/upgrade-xterm branch March 5, 2024 02:35
@mikelorant
Copy link
Contributor Author

@tsl0922 Thanks for merging this so promptly!

I have one more pull request I will put up now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants