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

Show Language Names In Chat #1165

Merged
merged 3 commits into from
Nov 8, 2024

Conversation

VMSolidus
Copy link
Member

Description

Port of Lost-Paradise-Project/Lost-Paradise#288 from Lost Paradise. This feature makes it so that the language you are speaking is shown in chat, preventing any confusion for players about what it is being spoken.

Media

image

Changelog

🆑

  • add: Languages are now shown in chat alongside character names.

@github-actions github-actions bot added Changes: C# Changes any cs files Changes: Localization Changes any ftl files Changes: YML Changes any yml files labels Oct 29, 2024
@github-actions github-actions bot added the Status: Needs Review Someone please review this label Oct 29, 2024
Resources/Locale/en-US/chat/managers/chat-manager.ftl Outdated Show resolved Hide resolved
@Remuchi Remuchi added Size: 4-Small For small issues/PRs Priority: 3-Medium Needs to be resolved at some point labels Oct 29, 2024
@Eagle-0
Copy link
Contributor

Eagle-0 commented Oct 29, 2024

Might just be me, but this isn't readable in the slightest.

@FoxxoTrystan
Copy link
Contributor

Dont do this, this will bloat up chat.

@FoxxoTrystan
Copy link
Contributor

I planned to use this instead.
space-wizards/RobustToolbox#5428

Copy link
Contributor

@stellar-novas stellar-novas left a comment

Choose a reason for hiding this comment

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

I am also not in favor of this system, when you add on radio channel tags too, this will make chat messages really long and ugly.

Resources/Locale/en-US/chat/managers/chat-language.ftl Outdated Show resolved Hide resolved
Resources/Prototypes/Language/genericlanguages.yml Outdated Show resolved Hide resolved
@Aidenkrz
Copy link
Contributor

I agree with the others, I don't like the visual of it being next to the name and it will bloat chat. @FoxxoTrystan had a good idea.

@Mnemotechnician
Copy link
Contributor

Mnemotechnician commented Oct 31, 2024

If anything, this should be togglable. Could add a toggle button to the filters menu, or elsewhere, and move the related code to client-side.

@FoxxoTrystan FoxxoTrystan added the Status: Needs Discussion Must be discussed label Nov 2, 2024
@Farrellka-dev
Copy link

I did this simply because we had a similar system before switching to your build. And for example, the general galactic language can be omitted for display. As for me, it is extremely convenient if someone does not speak the general language.

@Evgencheg
Copy link
Contributor

Yay our build now being used 😆

@VMSolidus VMSolidus merged commit 3209838 into Simple-Station:master Nov 8, 2024
10 checks passed
VMSolidus pushed a commit to VMSolidus/Einstein-Engines that referenced this pull request Nov 8, 2024
sleepyyapril added a commit to sleepyyapril/Einstein-Engines that referenced this pull request Nov 14, 2024
VMSolidus pushed a commit that referenced this pull request Nov 29, 2024
# Description

Re-enables language font and colors showing up in chat messages. The
language font/colors were bugged after adding language names in chat
(#1165)

I also overhauled the format for chat messages to be more aesthetically
pleasing, and made sure chat bubbles use the language color/font.

## Media

**Nekomimetic**


![image](https://github.com/user-attachments/assets/4cb88d7b-7466-42ed-bf2d-b68c68edb62c)

![image](https://github.com/user-attachments/assets/adec7fd7-0a84-4e21-9eff-c1a387e2d529)

**Bubblish**


![image](https://github.com/user-attachments/assets/c49f5573-ac96-4375-a584-e8b2cc6f5fee)

![image](https://github.com/user-attachments/assets/c739c249-9d1e-453e-9e92-3f7ef24a4d61)

**Tau-Ceti Basic (Default Language)**


![image](https://github.com/user-attachments/assets/e6660039-ac2f-449b-96e0-44e0142d3715)

![image](https://github.com/user-attachments/assets/2baec2ca-975d-4111-85b0-3b2056fdd7f7)

**Rootspeak**


![image](https://github.com/user-attachments/assets/99d94f8d-9a1c-4be9-8c98-b42c9c7cffd7)

![image](https://github.com/user-attachments/assets/c20b4621-1ca7-42f5-a22a-82c9a4d1dddf)

## Changelog

:cl: Skubman
- fix: Language colors and fonts will show up in text messages again,
both on in-person messages and on the radio.
- tweak: The text on chat bubbles now uses the color and font of the
language being spoken.
- tweak: The language prefix before the name on chat messages now uses
the language's color.
Fansana pushed a commit to Fansana/floofstation1 that referenced this pull request Dec 15, 2024
# Description

Re-enables language font and colors showing up in chat messages. The
language font/colors were bugged after adding language names in chat
(Simple-Station/Einstein-Engines#1165)

I also overhauled the format for chat messages to be more aesthetically
pleasing, and made sure chat bubbles use the language color/font.

## Media

**Nekomimetic**

![image](https://github.com/user-attachments/assets/4cb88d7b-7466-42ed-bf2d-b68c68edb62c)

![image](https://github.com/user-attachments/assets/adec7fd7-0a84-4e21-9eff-c1a387e2d529)

**Bubblish**

![image](https://github.com/user-attachments/assets/c49f5573-ac96-4375-a584-e8b2cc6f5fee)

![image](https://github.com/user-attachments/assets/c739c249-9d1e-453e-9e92-3f7ef24a4d61)

**Tau-Ceti Basic (Default Language)**

![image](https://github.com/user-attachments/assets/e6660039-ac2f-449b-96e0-44e0142d3715)

![image](https://github.com/user-attachments/assets/2baec2ca-975d-4111-85b0-3b2056fdd7f7)

**Rootspeak**

![image](https://github.com/user-attachments/assets/99d94f8d-9a1c-4be9-8c98-b42c9c7cffd7)

![image](https://github.com/user-attachments/assets/c20b4621-1ca7-42f5-a22a-82c9a4d1dddf)

## Changelog

:cl: Skubman
- fix: Language colors and fonts will show up in text messages again,
both on in-person messages and on the radio.
- tweak: The text on chat bubbles now uses the color and font of the
language being spoken.
- tweak: The language prefix before the name on chat messages now uses
the language's color.

(cherry picked from commit 1a9763528a9b15f11f474a2fa7ecfdde98aa9a15)
Pspritechologist pushed a commit to Simple-Station/Parkstation-Friendly-Chainsaw that referenced this pull request Dec 21, 2024
# Description

Re-enables language font and colors showing up in chat messages. The
language font/colors were bugged after adding language names in chat
(Simple-Station/Einstein-Engines#1165)

I also overhauled the format for chat messages to be more aesthetically
pleasing, and made sure chat bubbles use the language color/font.

## Media

**Nekomimetic**


![image](https://github.com/user-attachments/assets/4cb88d7b-7466-42ed-bf2d-b68c68edb62c)

![image](https://github.com/user-attachments/assets/adec7fd7-0a84-4e21-9eff-c1a387e2d529)

**Bubblish**


![image](https://github.com/user-attachments/assets/c49f5573-ac96-4375-a584-e8b2cc6f5fee)

![image](https://github.com/user-attachments/assets/c739c249-9d1e-453e-9e92-3f7ef24a4d61)

**Tau-Ceti Basic (Default Language)**


![image](https://github.com/user-attachments/assets/e6660039-ac2f-449b-96e0-44e0142d3715)

![image](https://github.com/user-attachments/assets/2baec2ca-975d-4111-85b0-3b2056fdd7f7)

**Rootspeak**


![image](https://github.com/user-attachments/assets/99d94f8d-9a1c-4be9-8c98-b42c9c7cffd7)

![image](https://github.com/user-attachments/assets/c20b4621-1ca7-42f5-a22a-82c9a4d1dddf)

## Changelog

:cl: Skubman
- fix: Language colors and fonts will show up in text messages again,
both on in-person messages and on the radio.
- tweak: The text on chat bubbles now uses the color and font of the
language being spoken.
- tweak: The language prefix before the name on chat messages now uses
the language's color.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: C# Changes any cs files Changes: Localization Changes any ftl files Changes: YML Changes any yml files Priority: 3-Medium Needs to be resolved at some point Size: 4-Small For small issues/PRs Status: Needs Discussion Must be discussed Status: Needs Review Someone please review this
Projects
None yet
Development

Successfully merging this pull request may close these issues.