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

Add pronoun tags #281

Draft
wants to merge 1 commit into
base: fabric-1.21.4
Choose a base branch
from

Conversation

celestialfault
Copy link
Contributor

What kind of PR is this?

  • Code change
    • These changes have been tested (if applicable)
  • Documentation
  • Translation
  • Other

What changes does this PR make?

Adds optional pronouns to name tags alongside the existing contributor nametags

Example

image

Anything else we should know?

This is currently in a "it functions but isn't really usable" state, as there's no way to modify this from the UI, but instead currently requires modifying the config (since I don't want to deal with touching screens long enough to change this someone please do this for me).

As such, this is currently more of a "request for comment" PR to see if this would be wanted.

Localizing this is also going to be a nightmare 🫠

no ui for this, so you have to manually edit the config to add them

someone please do the ui for me because i do not feel like touching
screens long enough to add this

localization is going to be horrible 🫠
@PinguinLars
Copy link
Contributor

Localizing this is also going to be a nightmare 🫠

I can think of some translations, but not all.

Anyways. It's subtile, but a nice feature.

@PinguinLars
Copy link
Contributor

PinguinLars commented Jan 20, 2025

Caution

I found a crash

If you put it ("pronouns": ["she"]) as lowercase and press f5 (because I had the see own nametag enabled) you crash. If this feature is going to use cloud sync it can be used to initanily crash people, because you don't crash yourself

Also this took me more then 1 hour to setup

@celestialfault
Copy link
Contributor Author

If you put it ("pronouns": ["she"]) as lowercase

yes, known issue - the config around this is somewhat brittle right now and could definitely do with a lot of extra work, but its not one of my priorities right now

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