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

Putting additional span srt_deadchat to deadsay for chat sorting #10899

Closed

Conversation

EvilDragonfiend
Copy link
Member

@EvilDragonfiend EvilDragonfiend commented Apr 18, 2024

About The Pull Request

Putting additional span srt_deadchat to deadsay for chat sorting

Used regex: ["']{1}[a-zA-Z,\b_ ]*dead

As the previous PR described, style class being used as chat sort class is conceptually wrong.
This PR now corrects deadsay

Why It's Good For The Game

We want sort chats correctly

Testing Photographs and Procedure

image

Changelog

🆑
fix: messages will not be incorrectly sorted into deadsay category anymore
/:cl:

@github-actions github-actions bot added Mapping DMM Change Administration TGUI-Changes Contains changes to TGUI. Make sure its up to date with TGUI 4.0 labels Apr 18, 2024
@EvilDragonfiend EvilDragonfiend changed the title Converting span deadsay to srt_deadchat Putting additional span srt_deadchat to deadsay for chat sorting Apr 18, 2024
@EvilDragonfiend
Copy link
Member Author

Hmm, actually no. I think refactoring to_chat seems more correct.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Administration Fix Mapping DMM Change TGUI-Changes Contains changes to TGUI. Make sure its up to date with TGUI 4.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant