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

Modernizes TGUI #4219

Merged
merged 77 commits into from
Jun 28, 2021
Merged

Modernizes TGUI #4219

merged 77 commits into from
Jun 28, 2021

Conversation

Sinestia
Copy link
Contributor

@Sinestia Sinestia commented May 4, 2021

About The Pull Request

ports tgstation/tgstation#54567
tgstation/tgstation#56080
tgstation/tgstation#56225

edit: additional followup PRs will go here
tgstation/tgstation#56223
tgstation/tgstation#56713
tgstation/tgstation#56727
tgstation/tgstation#56797
tgstation/tgstation#57081
tgstation/tgstation#57154
tgstation/tgstation#57251
tgstation/tgstation#56269
tgstation/tgstation#57326
tgstation/tgstation#57432
tgstation/tgstation#57465
tgstation/tgstation#57499
tgstation/tgstation#57717
tgstation/tgstation#57746
tgstation/tgstation#57825
tgstation/tgstation#57834
tgstation/tgstation#57905
tgstation/tgstation#58081
tgstation/tgstation#58215
tgstation/tgstation#58219
tgstation/tgstation#58484
tgstation/tgstation#58701
tgstation/tgstation#56190

TGUI Changes for Coders

Stack component replaces flex and generally replaces it, read more here
Storybook.js has been introduced, described in the above link.

bin/tgui ran with flags is more robust at handling issues for you, remember to run --lint and --fix when submitting something (because I sure as hell kept forgetting and it makes you look bad anyways.)

Fragment without a key is replaced with <>.

Typescript 4.2 can now be utilized for tgui components. Unit tests can be ran using Jest. Described here

Custom SVG fonts can be utilized now! Just throw them in the tgui icon's directly and then you can utilize them in your scripts. This is described here

A prettierrc exists now, for those who care. Along with multiple linter and vscode changes in general.

Resizable as a window property is dead, all windows are resizable by default now.

Some people have issues where their build differs from what the repo generates in the CIs, to fix this I believe you need to clear your untrackeds
you can do this with git clean -xdf on your tgui folder.

edit: on further investigation, it looks like some files have CRLF line endings instead of LF which they really shouldn't. Pulled straight from the repo, they're LF. This is the best lead I can find honestly. Ping me Ithiria#9193 if there's issues or to confirm that any of this works or something.

Why It's Good For The Game

~~No unnecessary merge conflicts with tgui PRs. ~~ lol use a bot

Newer TGUI

Typescript

Custom icons

Further monopolization of our lord and saviour vscode.

Changelog

🆑 Stylemistake, Cyberboss, SpaceManiac, other TG coders, Froststahr port.
code: TGUI has been upgraded to 4.3 and now includes typescript.
code: Custom SVG icons are now easy to do as well.
fix: All tgui windows will resize automatically, and can now be resized as will.
server: TGS will require adaptation.
/:cl:

@github-actions github-actions bot added GitHub TGUI-Changes Contains changes to TGUI. Make sure its up to date with TGUI 4.0 Tools labels May 4, 2021
@ike709
Copy link
Member

ike709 commented May 4, 2021

There's definitely more follow-up PRs.

@Sinestia
Copy link
Contributor Author

Sinestia commented May 4, 2021

oh. Fixing git tasks was that easy

@Sinestia
Copy link
Contributor Author

Sinestia commented May 4, 2021

There's definitely more follow-up PRs.

Yep, will get them

Only thing I think I will leave out is tgui 4.3, it's big enough that I think it should be done separately.

@ike709
Copy link
Member

ike709 commented May 4, 2021

If there's CBT changes in the 4.3 PR then get those but otherwise yes you can ignore 4.3.

Follow-up to fix the failing Docker CI on master.

- Reorganize the entire Dockerfile to be more readable
- Inline the tgstation/byond Dockerfile into our own, so we can change the base distro at will
- Also allows us to trash the dependencies.sh<->Dockerfile hack
- Use 32-bit libs on a 64-bit distro so that we can download and run recent 64-bit Node binaries
- Call tools/build/build rather than DreamMaker directly
@Sinestia
Copy link
Contributor Author

Sinestia commented May 4, 2021

If there's CBT changes in the 4.3 PR then get those but otherwise yes you can ignore 4.3.

Seems like there's some important stuff in there, will have to get them then

edit: I Just went ahead and ported the whole thing, easier to do it all while I have the PR open instead of doing half
edit2: there's another PR I see, HmMm. Ill figure it later

@ike709
Copy link
Member

ike709 commented May 6, 2021

#4220

@github-actions
Copy link

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@github-actions
Copy link

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@github-actions
Copy link

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@BrumalLake
Copy link
Contributor

tell me this includes the tall ass atmos chamber monitoring consoles

@github-actions
Copy link

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@Sinestia
Copy link
Contributor Author

tell me this includes the tall ass atmos chamber monitoring consoles

You can think of this as being solely about the tgui backend

not any specific interfaces were changed beyond updating them

@MCterra10
Copy link
Contributor

Yolomerged

@ike709
Copy link
Member

ike709 commented Jun 28, 2021

It was testmerged on sage.

@jupyterkat
Copy link
Contributor

ci checks fails on tgui regardless if the build is correct or not

@BrumalLake
Copy link
Contributor

ci checks fails on tgui regardless if the build is correct or not

#4606
no it's somehow possible, not sure how i got it

@Sinestia
Copy link
Contributor Author

Sinestia commented Jun 28, 2021

ci checks fails on tgui regardless if the build is correct or not

Your cache/untrackeds are bad I believe

run git clean -xdf on your tgui folder

happened to me too

edit: on further investigation, it looks like some files have CRLF line endings instead of LF which they really shouldn't. Pulled straight from the repo, they're LF. This is the only tangible difference between my folder and that of someones who didn't work.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Code Improvement Fix GitHub Server TGUI-Changes Contains changes to TGUI. Make sure its up to date with TGUI 4.0 Tools
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet