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

TGUI upgrades to support react migration #5435

Merged
merged 15 commits into from
Jan 18, 2024

Conversation

mullenpaul
Copy link
Contributor

@mullenpaul mullenpaul commented Jan 12, 2024

About the pull request

Round of upgrades and changes to support migration to React for tgui. By doing more smaller changes I hope it's easier to review and test.

Explain why it's good for the game

We want to migrate from infernojs to react.

Testing Photographs and Procedure

Screenshots & Videos

Put screenshots and videos here with an empty line between the screenshots and the <details> tags.

Changelog

🆑
refactor: upgraded root tgui files to typescript to closer align with tg
refactor: upgraded typescript to 4.9.4
refactor: upgraded yarn to 3.3.1
refactor: upgraded a bunch of tgui tooling
/:cl:

@github-actions github-actions bot added UI deletes nanoui/html Refactor Make the code harder to read labels Jan 12, 2024
@mullenpaul mullenpaul marked this pull request as draft January 12, 2024 21:28
@mullenpaul mullenpaul marked this pull request as ready for review January 12, 2024 23:37
@mullenpaul mullenpaul added the Testmerge Candidate we'll test this while you're asleep and the server has 10 players label Jan 13, 2024
@turbotrue
Copy link

I think this is making my chat bar resize randomly

@harryob harryob added this pull request to the merge queue Jan 18, 2024
Merged via the queue into cmss13-devs:master with commit 2572f25 Jan 18, 2024
27 checks passed
cm13-github added a commit that referenced this pull request Jan 18, 2024
github-merge-queue bot pushed a commit that referenced this pull request Jan 23, 2024
# About the pull request

<!-- Remove this text and explain what the purpose of your PR is.

Mention if you have tested your changes. If you changed a map, make sure
you used the mapmerge tool.
If this is an Issue Correction, you can type "Fixes Issue #169420" to
link the PR to the corresponding Issue number #169420.

Remember: something that is self-evident to you might not be to others.
Explain your rationale fully, even if you feel it goes without saying.
-->

Switches TGUI from Inferno to React.

Part 2 of the PR #5435

Based heavily on the work from:
https://github.com/tgstation/tgstation/pull/80044/files

# Explain why it's good for the game
React is a more suitable framework

# Testing Photographs and Procedure
<details>
<summary>Screenshots & Videos</summary>

Put screenshots and videos here with an empty line between the
screenshots and the `<details>` tags.

</details>


# Changelog
:cl:
refactor: switched from infernojs to react
/:cl:
Doubleumc pushed a commit to Doubleumc/PvE-CMSS13 that referenced this pull request Jul 24, 2024
<!-- Remove this text and explain what the purpose of your PR is.

Mention if you have tested your changes. If you changed a map, make sure
you used the mapmerge tool.
If this is an Issue Correction, you can type "Fixes Issue #169420" to
link the PR to the corresponding Issue number #169420.

Remember: something that is self-evident to you might not be to others.
Explain your rationale fully, even if you feel it goes without saying.
-->

Switches TGUI from Inferno to React.

Part 2 of the PR cmss13-devs/cmss13#5435

Based heavily on the work from:
https://github.com/tgstation/tgstation/pull/80044/files

React is a more suitable framework

<details>
<summary>Screenshots & Videos</summary>

Put screenshots and videos here with an empty line between the
screenshots and the `<details>` tags.

</details>

:cl:
refactor: switched from infernojs to react
/:cl:
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Refactor Make the code harder to read Testmerge Candidate we'll test this while you're asleep and the server has 10 players UI deletes nanoui/html
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants