Skip to content

feat: add cosmetics support to per-message profiles#369

Closed
poolcritter wants to merge 1 commit into
SableClient:devfrom
poolcritter:push-ttymuoktmuut
Closed

feat: add cosmetics support to per-message profiles#369
poolcritter wants to merge 1 commit into
SableClient:devfrom
poolcritter:push-ttymuoktmuut

Conversation

@poolcritter

Copy link
Copy Markdown

This patch allows the following keys to be set on messages' com.beeper.per_message_profiles key:

  • moe.sable.app.name_color: Overrides the name color; same format as moe.sable.app.name_color in profiles.
  • io.fsky.nyx.pronouns: Overrides the per-message pronouns; same format as io.fsky.nyx.pronouns in profiles.

Considerations

  • The ability to override custom fonts has intentionally been left unimplemented. Per-message fonts could allow users to specify custom name fonts without needing room-admin approval, which could be dangerous.

This patch allows the following keys to be set on messages' `com.beeper.per_message_profiles` key:

* `moe.sable.app.name_color`: Overrides the name color; same format as `moe.sable.app.name_color` in profiles.
* `io.fsky.nyx.pronouns`: Overrides the per-message pronouns; same format as `io.fsky.nyx.pronouns` in profiles.

# Considerations

* The ability to override custom fonts has intentionally been left unimplemented. Per-message fonts could allow users to
  specify custom name fonts without needing room-admin approval, which could be dangerous.
@poolcritter poolcritter requested review from 7w1 and hazre as code owners March 18, 2026 19:34
@dozro

dozro commented Mar 18, 2026

Copy link
Copy Markdown
Member

is that related/an alternative to #309?


const sourcePronouns = pmp?.["io.fsky.nyx.pronouns"] ?? profile.pronouns ?? []
const mergedPronouns = useMemo(() => {
const existing = profile.pronouns ? [...profile.pronouns] : [];

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

#309 already handles per-message-profile's pronouns iirc

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

oh yeah, didn't notice — should I rebase onto there?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

if you wish to :3

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