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 Say #8404

Merged
merged 44 commits into from
Apr 5, 2023
Merged

TGUI Say #8404

merged 44 commits into from
Apr 5, 2023

Conversation

itsmeow
Copy link
Member

@itsmeow itsmeow commented Jan 30, 2023

About The Pull Request

Ports TGUI Say:

And some needed pre-req PRs:

Additional changes for BeeStation:

  • Added explo, uplink, :l, :r, and :h channel handling
  • Added support for .channel prefixes
  • Tweaked colors
  • The entire window is draggable, rather than just parts of it (excluding the textbox, so you can select text)
  • The border has been tweaked slightly, it is not animated but has a minor gradient on both the top and bottom
  • Added a border around the text input part of the box
  • Added LOOC channel
  • Legacy support pref, with typing indicators and all
  • Typing ; in Say mode will convert the radio prefix to ;, and removing it will go back to Say.
  • Shift+TAB and r-clicking the button will go back in channels.
  • Re-opening the window on a different channel will no longer break the button contents.
  • Added "show prefix" mode, which treats :s as text in the input box while still updating the color and button.
  • Pressing Esc on the game window with the say unfocused will exit the say.

SQL Update on merge:

UPDATE SS13_preferences SET preference_value = preference_value | (1<<14) | (1<<15) | (1<<16) | (1<<17) WHERE preference_tag = 2;

Why It's Good For The Game

The "Say" verb is the most used and looks quite awful in my opinion, plus TGUI say is just really cool, and adds some nice features like the thinking indicator and forced-speech (AUGH!!!), as well as more accessibility to new players since the syntax of speech is baked into the say-box.

Testing Photographs and Procedure

Screenshots&Videos

Newest appearance

image

image

image

2023-01-31.03-58-50.mp4

image

image

image

image

New prefs

image

Light theme pref

image

Changelog

🆑
add: Added TGUI-Say (can be disabled in prefs, light theme also in prefs) - Tab to switch channels.
add: Added new TGUI Say keybinds - Y for Radio, U for LOOC (not currently rebindable).
add: Added TGUI input preferences (mostly unused currently).
add: Added glorfing to TGUI-Say - if you get hit hard enough, your message will be cut off with "-OW!!!", "-AUGH!!!", "-GLORF!!" or similar.
admin: Added logging to typing indicators.
add: Added OOC Verb "Refresh TGUI".
tweak: Legacy say hotkey macros (Ctrl+T, Ctrl+O) have been replaced with T and O.
/:cl:

jlsnow301 and others added 6 commits January 30, 2023 01:44
Co-authored-by: Mothblocks <35135081+Mothblocks@users.noreply.github.com>
Co-authored-by: AnturK <AnturK@users.noreply.github.com>
Co-authored-by: iamgoofball <iamgoofball@gmail.com>
Co-authored-by: Aleksej Komarov <stylemistake@gmail.com>
Co-authored-by: KubeRoot <6917698+KubeRoot@users.noreply.github.com>
Co-authored-by: Kapu1178 <75460809+Kapu1178@users.noreply.github.com>
Co-authored-by: Iamgoofball <4081722+Iamgoofball@users.noreply.github.com>
Co-authored-by: DomitiusKnack <56321744+DomitiusKnack@users.noreply.github.com>
Co-authored-by: LemonInTheDark <58055496+LemonInTheDark@users.noreply.github.com>
Co-authored-by: Wallem <66052067+Wallemations@users.noreply.github.com>
adds the ability to realtime inject HTML into TGWindow instances, just as you can create TGWindow instances with inline_html at initialization.

Co-authored-by: Aleksej Komarov <stylemistake@gmail.com>
About The Pull Request

adds the Refresh TGUI verb to the debug category OOC category, but it isn't locked behind any permissions. i'd put it in OOC, but.. eeeeeeh? there's not real much use for this verb other for debuggers. i put it in the OOC category
Why It's Good For The Game

debug purposes mostly

not player facing lole
@github-actions github-actions bot added Config Update Sprites TGUI-Changes Contains changes to TGUI. Make sure its up to date with TGUI 4.0 Tools labels Jan 30, 2023
@PowerfulBacon
Copy link
Member

Legendary PR

@PowerfulBacon PowerfulBacon added the Golden ⭐ For high-effort PRs that make big and good improvements to the game. label Jan 31, 2023
@itsmeow itsmeow marked this pull request as ready for review January 31, 2023 10:22
@llol111
Copy link

llol111 commented Jan 31, 2023

Holy smokes that looks awfull.
Ive mostly gotten over my "tgui bad on principle" phase, but man that just looks terrible.
We have decent and good looking tgui windows, why's it like this?
cant we just have the normal popup, but made in TGUI?

@itsmeow
Copy link
Member Author

itsmeow commented Jan 31, 2023

We have decent and good looking tgui windows, why's it like this?

Because it's built on IE11 with no modern CSS features, and no polyfills available to regular TGUIs because it needs to not be dependent on 5000 external dependencies.

@itsmeow
Copy link
Member Author

itsmeow commented Jan 31, 2023

cant we just have the normal popup, but made in TGUI?

This is like 5x less functional and more likely to break but certainly possible

@llol111
Copy link

llol111 commented Jan 31, 2023

I dont know (or care, if im to be honest) about the technical workings in the background.
Im sure you have decent reasons for replacing what we have now, et cetera.
Chat breaking all the time like normal tgui tends to do is a verry valid concern, i suppose.

It would, however, look a lot better.
As is, this looks bad.
Im sure you can justify this being a good thing anyways because >muh functionalities and shit
🤷

@itsmeow
Copy link
Member Author

itsmeow commented Jan 31, 2023

Players will still have the option of old chat, so if you don't like the look you can use that

@itsmeow
Copy link
Member Author

itsmeow commented Mar 23, 2023

I've moved it, is this better?

image

image

@Eric-01
Copy link

Eric-01 commented Mar 25, 2023

I don't see any difference :/

Classic-vs-TGUI-say

@Absolucy
Copy link
Contributor

Absolucy commented Mar 25, 2023

Perhaps whispering (# prefix) should appear on the say popup like it does with radio prefixes and such?

@PowerfulBacon
Copy link
Member

I mean.. I could, but also that wasn't a thing before

The old one didn't need it. This is an entirely new input box though

@itsmeow
Copy link
Member Author

itsmeow commented Mar 28, 2023

I already added the esc macro

@PowerfulBacon
Copy link
Member

Default 3 talk icon looks pretty bad, I think just remove it and use default0 instead, or a static speech bubble. After that I'm fine with merging this

@Crossedfall Crossedfall added this pull request to the merge queue Apr 5, 2023
Merged via the queue into BeeStation:master with commit cc88822 Apr 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Administration Config Update Feature Golden ⭐ For high-effort PRs that make big and good improvements to the game. Needs Testmerge Sprites SQL SQL Change TGUI-Changes Contains changes to TGUI. Make sure its up to date with TGUI 4.0 Tools Tweak
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

10 participants