-
Notifications
You must be signed in to change notification settings - Fork 510
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
Chatsan fixes to emote entries ending with letters. #2292
Chatsan fixes to emote entries ending with letters. #2292
Conversation
Isn't there code somewhere that's explicitly supposed to strip trailing punctuation... exactly for this purpose? E: oh right that's buried in emote chatcode instead of being in a sane place |
Was this fixed on upstream? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There is a much simpler and more robust fix to these recurring issues:
_sanitizer.TrySanitizeOutSmilies(newMessage, source, out newMessage, out emoteStr); |
Move this line up above all the if statements. Check emotes first, then punctate and capitalise and stuff. Since this is inherited from upstream, we could probably even upstream that fix. It'll help everyone who uses the auto-punctuate feature.
If you'd like this done, I'd ask you to do it. I frankly hate this feature and tried to spend the minimum amount of time on it. Thank you. |
Testbench in description works without issue. |
Oh, I was just about to make the change. Well, thanks for doing it yourself, I suppose! I hate this feature too, but it is what it is. |
About the PR
Fixes Chatsan emote entries that end in letters by adding periods.
Why / Balance
Brainrot.
How to test
Enter the following text in in-game chat. Each one of them should result in an emote. Note that the
>:i
entry requires an additional > because of local chat.Media
Requirements
Breaking changes
Changelog
🆑