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

MSC4247: User Pronouns #4247

Open
wants to merge 15 commits into
base: main
Choose a base branch
from
Open

Conversation

everypizza1
Copy link

@everypizza1 everypizza1 commented Dec 27, 2024

proposals/4247-user-pronouns.md Outdated Show resolved Hide resolved
proposals/4247-user-pronouns.md Outdated Show resolved Hide resolved
proposals/4247-user-pronouns.md Outdated Show resolved Hide resolved
Copy link
Member

Choose a reason for hiding this comment

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

Implementation requirements:

  • Client
  • Server

Copy link

Choose a reason for hiding this comment

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

Conduwuit already supports this by supporting arbitrary fields in #4133 so I'd assume this MSC only requires a client implementation that can read/write the field?

@turt2live turt2live added proposal A matrix spec change proposal client-server Client-Server API kind:feature MSC for not-core and not-maintenance stuff needs-implementation This MSC does not have a qualifying implementation for the SCT to review. The MSC cannot enter FCP. labels Dec 27, 2024
everypizza1 and others added 3 commits December 27, 2024 11:04
Co-authored-by: Travis Ralston <travpc@gmail.com>
Co-authored-by: Travis Ralston <travpc@gmail.com>
Copy link
Member

@uhoreg uhoreg left a comment

Choose a reason for hiding this comment

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

Seems pretty straightforward. A few mostly-nitpicky comments.

proposals/4247-user-pronouns.md Show resolved Hide resolved
proposals/4247-user-pronouns.md Outdated Show resolved Hide resolved
proposals/4247-user-pronouns.md Outdated Show resolved Hide resolved
}
```
The example uses it/its pronouns followed by she/her pronouns, both in English.
The array is ordered by preference, `language` should be a language code, and
Copy link
Member

Choose a reason for hiding this comment

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

"language code" needs to be defined more precisely, I don't think anything in the current spec uses languages. #3554 seems to reference BCP-47

Copy link
Author

Choose a reason for hiding this comment

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

Would ISO 639 language codes work?

Copy link
Member

Choose a reason for hiding this comment

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

I think BCP-47 uses ISO 639, but also allows optional subtags for specifying regions (so both en and en-US are allowed in BCP-47, while ISO 639 only has the concept of en)

Copy link
Author

@everypizza1 everypizza1 Dec 29, 2024

Choose a reason for hiding this comment

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

I'll switch to BCP-47 then, so there's more flexibility.

proposals/4247-user-pronouns.md Outdated Show resolved Hide resolved
Comment on lines 22 to 26
"subject": "it",
"object": "it",
"possessive_determiner": "its",
"possessive_pronoun": "its",
"reflexive": "itself",
Copy link
Member

Choose a reason for hiding this comment

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

I'm not sure if these fields are a good idea, since they seem to be specific to English and probably aren't compatible with any other language. Also, I'd guess most implementations will simply use summary for displaying in the profile and won't even try to apply the other fields.

It might be better to just have the freeform field. Maybe also an enum (non-freeform string) for preferred grammatical gender, although even that could get complicated.

Is there any prior art or research into user-definable pronouns that support internationalization?

If the fields are kept as-is, each of them needs to be defined separately, it's not enough to have them in the example. Keeping the fields may also require an implementation actually using them to show they're useful.

Copy link
Member

Choose a reason for hiding this comment

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

Thinking more about the enum: it should at least have values masculine, feminine and neuter. The potential complications I can think of are:

  • Some languages don't have a real neuter. Maybe that's not a problem though, because clients have to be able to fall back to gender being unknown anyway?
  • Some languages have animate and inanimate neuter forms (singular they vs it in English). Does the enum need to have those separately? (neuter_inanimate)
  • There might be other types in some weird languages. Are there any, and if there are, do they need to be options too? (and then how do other languages handle those options?)

On the implementation side: pronoun information is generally only needed for rendering state events like profile changes ("X changed his/her/their name"), but those state events don't include this profile info. Fetching the full profile for each profile change state event seems like a bad idea.

In any case, a freeform field should exist to display when viewing someones profile. If the other fields can't be used effectively, then it may be best to narrow this down to only have the freeform field and nothing else.

Copy link
Author

Choose a reason for hiding this comment

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

I think we may want to throw out subject, object, etc and do the masculine, feminine, and neuter with the animate and inanimate and for languages that don't support some of those, fall back to the closest to neutral.

array. These fields can be fetched through the
[profile API endpoints](https://spec.matrix.org/unstable/client-server-api/#profiles).
Clients should use these instead of they/them where possible. All fields
within `m.pronouns` are optional, exluding `"language"` and `"summary"`.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
within `m.pronouns` are optional, exluding `"language"` and `"summary"`.
within `m.pronouns` are optional, excluding `"language"` and `"summary"`.

(since there is only one more field, it might be better to reverse the wording of this sentence and say that all are required except grammatical gender)

"avatar_url": "", "displayname": "",
"m.pronouns": [
{
"grammatical_gender": "inanimate",
Copy link
Contributor

Choose a reason for hiding this comment

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

What grammatical genders are available? I think they need to be enumerated. I don't know what grammatical gender "they/them" would be, for example.

I also think they need to ne prefixed (m.male for example) and defined what that means in terms of how the UIs should adapt their rendering. This will also allow for future grammatical genders from other languages or linguistic innovations to be defined later (and use unstable prefixes).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
client-server Client-Server API kind:feature MSC for not-core and not-maintenance stuff needs-implementation This MSC does not have a qualifying implementation for the SCT to review. The MSC cannot enter FCP. proposal A matrix spec change proposal
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants