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] Instrument editor now uses TGUI #2413

Merged
merged 1 commit into from
Mar 15, 2024
Merged

Conversation

Steals-The-PRs
Copy link
Collaborator

Mirrored on Nova: NovaSector/NovaSector#1446
Original PR: tgstation/tgstation#81923

About The Pull Request

Instruments now use TGUI as their editor which is pretty cool.
It's mostly a 1:1 remake of the HTML UI except I did make a change to make the playback options a little more compact, leaving some more space for the editor before you have to scroll, and some other minor things that were made to make the UI hopefully nicer to look at and mess with.

When there's a song to play - While playing, Repeat section can't be edited
image

Help section and UI when there's no song put in
image

Why It's Good For The Game

It is yet another step in finishing up https://hackmd.io/XLt5MoRvRxuhFbwtk4VAUA?view
Instruments especially were in a poor spot because they didn't respect things like IN_USE to not refresh if it's not the "UI" you are on, and such.

Changelog

🆑 JohnFulpWillard
refactor: Instruments now use TGUI.
/:cl:

* Instrument editor now uses TGUI (#81923)

## About The Pull Request

Instruments now use TGUI as their editor which is pretty cool.
It's mostly a 1:1 remake of the HTML UI except I did make a change to
make the playback options a little more compact, leaving some more space
for the editor before you have to scroll, and some other minor things
that were made to make the UI hopefully nicer to look at and mess with.

When there's a song to play - While playing, Repeat section can't be
edited

![image](https://github.com/tgstation/tgstation/assets/53777086/33f21ca3-98d8-4147-83e7-74e7611463e6)

Help section and UI when there's no song put in

![image](https://github.com/tgstation/tgstation/assets/53777086/babd30ab-9551-448b-9fe6-24e0b0535caf)

## Why It's Good For The Game

It is yet another step in finishing up
https://hackmd.io/XLt5MoRvRxuhFbwtk4VAUA?view
Instruments especially were in a poor spot because they didn't respect
things like ``IN_USE`` to not refresh if it's not the "UI" you are on,
and such.

## Changelog

:cl:
refactor: Instruments now use TGUI.
/:cl:

---------

Co-authored-by: Ghom <[email protected]>

* Instrument editor now uses TGUI

---------

Co-authored-by: John Willard <[email protected]>
Co-authored-by: Ghom <[email protected]>
@ReezeBL ReezeBL merged commit 7f19e34 into master Mar 15, 2024
24 checks passed
@ReezeBL ReezeBL deleted the upstream-mirror-1446 branch March 15, 2024 09:25
AnywayFarus added a commit that referenced this pull request Mar 15, 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