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

More chat style updates - new announcement colors, admin announcement changes, and more #16087

Merged
merged 15 commits into from
Jun 29, 2024

Conversation

Neocloudy
Copy link
Contributor

@Neocloudy Neocloudy commented Jun 16, 2024

About The Pull Request

Porting a bunch of stuff from other codebases

Might be good to TM when approved so that players can riot give feedback

Probably the most obvious change: new examine block style (from Nova Sector)

image
image

Replaces neon announcement colors, most of them are less saturated (from Nova Sector)

image
image
image

OOC announcements are much more obvious (from TG)

image

Helpful stuff is colored green (from TG)

image
image

Other stuff:

  • Tweezers use the standard "icons" for their do_afters, so now the friendly icon appears above the user, and the medical effect appears on the patient.
  • Special robot damage strings for examine.

Light mode:

image

Why It's Good For The Game

I think all of these are pretty replacements for the current styles. Announcement colors especially. We should at least give these a try on live through a TM to see what everyone else thinks

Changelog

🆑
add: Replaced neon announcement colors with less saturated ones
add: Helpful stuff, like using gauze or pills, has a green span in chat
add: Admin announcements play sound and look more obvious
add: Custom Information plays sound
add: New examine block style
add: Campaign missions and gamemodes use large red announcement blocks
/:cl:

@tgstation-server tgstation-server added Sound Sound change. Feature New interesting mechanics with new interesting bugs UI We make things worse, but rounder labels Jun 16, 2024
@TiviPlus TiviPlus added the Test Merge Candidate This PR has been reviewed and is ready for testing, unless something changes. label Jun 18, 2024
comfyorange added a commit that referenced this pull request Jun 19, 2024
@Lumipharon
Copy link
Contributor

It would be better if we had different sounds tbh. This pr and the previous announcement pr both use admin sounds for not admin stuff, which is quite inconvenient for being an admeme.

comfyorange added a commit that referenced this pull request Jun 20, 2024
@Barnet2
Copy link
Contributor

Barnet2 commented Jun 20, 2024

The sound used for admin announcements is the admin ping sound which is uh, not good at all for admins.

light mode examine blocks have the same margin as dark mode

game modes use ooc announcements !!!!

some stack examine fixes: no more "It is a small item.blah blah blah" all on the same line
@tgstation-server tgstation-server removed the Test Merge Candidate This PR has been reviewed and is ready for testing, unless something changes. label Jun 20, 2024
comfyorange added a commit that referenced this pull request Jun 20, 2024
comfyorange added a commit that referenced this pull request Jun 20, 2024
comfyorange added a commit that referenced this pull request Jun 21, 2024
comfyorange added a commit that referenced this pull request Jun 21, 2024
comfyorange added a commit that referenced this pull request Jun 21, 2024
… they can't be confused with admeme shenanigans
comfyorange added a commit that referenced this pull request Jun 22, 2024
comfyorange added a commit that referenced this pull request Jun 22, 2024
comfyorange added a commit that referenced this pull request Jun 22, 2024
comfyorange added a commit that referenced this pull request Jun 23, 2024
comfyorange added a commit that referenced this pull request Jun 25, 2024
Copy link
Contributor

@Lumipharon Lumipharon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not entirely happy with these in campaign. The mission end ones in particular are... not particular informative. The most important info is the actual outcome, and thats the smallest text at the end of the annoucement.

It also doesn't even SAY what faction won/lost now.

@Lumipharon Lumipharon added the Test Merge Candidate This PR has been reviewed and is ready for testing, unless something changes. label Jun 26, 2024
comfyorange added a commit that referenced this pull request Jun 26, 2024
comfyorange added a commit that referenced this pull request Jun 26, 2024
comfyorange added a commit that referenced this pull request Jun 26, 2024
comfyorange added a commit that referenced this pull request Jun 26, 2024
comfyorange added a commit that referenced this pull request Jun 27, 2024
@tgstation-server tgstation-server removed the Test Merge Candidate This PR has been reviewed and is ready for testing, unless something changes. label Jun 27, 2024
comfyorange added a commit that referenced this pull request Jun 27, 2024
comfyorange added a commit that referenced this pull request Jun 29, 2024
@TiviPlus TiviPlus merged commit 3add8e4 into tgstation:master Jun 29, 2024
35 checks passed
@Neocloudy Neocloudy deleted the chatstyle-phase3 branch June 29, 2024 09:36
github-actions bot added a commit that referenced this pull request Jun 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature New interesting mechanics with new interesting bugs Sound Sound change. UI We make things worse, but rounder
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants