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

Adjust ChatLog MessageIndicator for consistency with the base game #215

Closed
wants to merge 2 commits into from

Conversation

NotRyken
Copy link

@NotRyken NotRyken commented Jan 11, 2025

This PR makes a simple change to the MessageIndicator used by the ChatLog class, to improve consistency when identifying chat messages based on the indicator.

In the base game, a translatable text object is passed as the 3rd parameter of the MessageIndicator constructor, and a hard-coded string is passed as the 4th parameter.

Currently, ChatPatches passes a translated string as the 4th parameter, preventing other mods from using that string to identify messages.

Modifying the MessageIndicator to be consistent with the base game will allow other mods to correctly identify messages added by ChatLog when processing chat.

@NotRyken NotRyken closed this Jan 15, 2025
@mrbuilder1961
Copy link
Owner

what happened?

@mrbuilder1961
Copy link
Owner

this seems like a good idea, unless you give a reason i'm going to implement this and credit you

mrbuilder1961 added a commit that referenced this pull request Jan 16, 2025
Credit: @NotRyken in #215
(note: this may be rescinded if a reason is given from the original PR, as it was closed)
mrbuilder1961 added a commit that referenced this pull request Jan 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants