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

[MIRROR] Removes grid usage + heavy refactors #2853

Merged
merged 1 commit into from
Apr 13, 2024
Merged

Conversation

Steals-The-PRs
Copy link
Collaborator

Mirrored on Nova: NovaSector/NovaSector#1940
Original PR: tgstation/tgstation#82571

About The Pull Request

Grid has been deprecated for quite some time and we still use it. I won't completely remove the component, this way downstreams won't immediately suffer, but I can remove it from usage.

Some of these UIs had issues with them and as a hobby project I've refactored them into typescript / rebuilt them. Airlock electronics, for instance, looks substantially better.

before/after as requested

current airlock electronics scrolls into oblivion
6RJ29HCPob

updated
Screenshot 2024-04-11 164321

Why It's Good For The Game

Code improvement + probably UI bug fixes

Changelog

🆑 jlsnow301
fix: Airlock electronics and other access-config type UIs should look much better.
/:cl:

* Removes grid usage + heavy refactors (#82571)

## About The Pull Request
Grid has been deprecated for quite some time and we still use it. I
won't completely remove the component, this way downstreams won't
immediately suffer, but I can remove it from usage.

Some of these UIs had issues with them and as a hobby project I've
refactored them into typescript / rebuilt them. Airlock electronics, for
instance, looks substantially better.

<details>
<summary>before/after as requested</summary>

current airlock electronics scrolls into oblivion

![6RJ29HCPob](https://github.com/tgstation/tgstation/assets/42397676/ba82bc20-40fa-4af0-b709-7c8846c25652)

updated
![Screenshot 2024-04-11
164321](https://github.com/tgstation/tgstation/assets/42397676/05507e06-6305-4175-8476-778c345f02c8)

</details>

## Why It's Good For The Game
Code improvement + probably UI bug fixes
## Changelog
:cl:
fix: Airlock electronics and other access-config type UIs should look
much better.
/:cl:

* Removes grid usage + heavy refactors

* Updates our UIs to get rid of grid usage

---------

Co-authored-by: Jeremiah <[email protected]>
Co-authored-by: Mal <[email protected]>
@ReezeBL ReezeBL merged commit e151b66 into master Apr 13, 2024
26 checks passed
@ReezeBL ReezeBL deleted the upstream-mirror-1940 branch April 13, 2024 12:52
AnywayFarus added a commit that referenced this pull request Apr 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants