Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add a titlecase counterpart of a pronoun variable #10379

Open
wants to merge 1 commit into
base: content
Choose a base branch
from

Conversation

dreamscached
Copy link
Member

Currently, the pronoun variables are lowercase-only, and it's tricky to use them as subjects (rather than objects) in the script lines. This PR adds a .title() variant to the global store, e.g. he becomes He (in addition to [he], as variables are case-sensitive.)

@Booplicate
Copy link
Member

I feel like the idea of using variables like this for substitution is smart, but I don't like polluting namespace. How about we create a small function that gets one string and returns another based on some condition (sex in this case, although we should move from sex to explicit pronouns set in the future)?

@dreamscached
Copy link
Member Author

dreamscached commented Mar 9, 2025

Alternatively, we could make pronouns (and gendered words) both __str__able and callable, so that we can maybe use both [he] and [he(upper=True)]?

I don't exactly get the idea with a 'small function', what could that look like? With explicit reference-like variables like this it'll be pretty simple for submodders to work with them, rather than invoking a function.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants