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

1984 (Autopunctuation) #154

Merged
merged 1 commit into from
Oct 13, 2024
Merged

Conversation

Nerev4r
Copy link
Collaborator

@Nerev4r Nerev4r commented Oct 13, 2024

About The Pull Request

Code by Yooriss, I just work here.
This PR introduces basic automatic punctuation for the following circumstances on living mobs only, and only for says (/me or emotes that aren't special say-emotes are unaffected):

If a chat message does not end in any kind of punctuation (like a period (.), a comma (,), a question mark (?), an exclamation mark (!) or a hyphen (-)), a period is automatically added to the end.
Any kind of lowercase 'i' between word boundaries is automatically converted to uppercase.

To be completely clear: this does not affect PDA's, asterisk emotes (*emotes), full emotes, announcements or basically any other non-speaking form of communication. It affects speech, so says and radio chatter only. PDAspeak enjoyers can continue to sling slang at one another without The Man moderating their grammar.

I've strategically chosen these two changes to produce what I believe to be most comfortable compromise between an easy chatting experience and having dialogue look clean, while also allowing for people to continue to flavour their character's speech patterns as appropriate.

There is one caveat: if you make use of in-game bold/italics/underline formatting, the autopunct function this PR adds will not add an additional end-of-line period if your message does not contain one. For example: |I can't believe you've done this| will not have a period added - you'll need to do this yourself.

I've confirmed with our unofficial TGcode liaison (goof) that regex filters have negligible performance impacts (and that any concerns that they did was largely what he describes as 'cargo cult' behavior). See for yourself on /tg/'s Discord here, if you like.

tl;dr: easier and faster Good Chatting (tm) with likely little performance impact

Why It's Good For The Game

I swear to God.

Changelog

🆑
add: Automatic punctuation!
/:cl:

Copy link

@CliffracerX CliffracerX left a comment

Choose a reason for hiding this comment

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

you should punctuate, NOW

@CliffracerX CliffracerX merged commit 36128da into DopplerShift13:master Oct 13, 2024
22 checks passed
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