matrix-org / matrix-spec

The Matrix protocol specification
Apache License 2.0
195 stars 97 forks source link

Need an m.room.* event for specifying accent colours (primary and secondary) for a room UI (SPEC-259) #118

Open matrixbot opened 9 years ago

matrixbot commented 9 years ago

Submitted by @​matthew:matrix.org

(Imported from https://matrix.org/jira/browse/SPEC-259)

matrixbot commented 9 years ago

Jira watchers: @ara4n

matrixbot commented 8 years ago

N.B. this should be private user data - just because I want my rooms to be bright pink shouldn't change it for other people!

-- @ara4n

turt2live commented 6 years ago

I'm not sure this is something that is cross-client compatible, given currently only Riot has a concept of tinting the room view.

just because I want my rooms to be bright pink shouldn't change it for other people

Granular settings in Riot ignores this part and lets rooms suggest color schemes fwiw. Users can of course override that though.

KitsuneRal commented 6 years ago

It's not much difficult for Quaternion to add rooms tinting. I actually like the idea. And as a user I would like to have two levels of the setting, one where the room suggests a tint colour and another where I can override the colour through my account_data for the room.

turt2live commented 6 years ago

It's generally not difficult for clients to add tinting. My concern is more that I feel it should be a per-client option and not standardized. Particularly if a client were to support it slightly differently, the user's color choice could end up looking really bad to them.

KitsuneRal commented 6 years ago

Can't we collaborate across clients on that a little bit? As a user, I would be frustrated to re-set up colours on Riot iOS, then Quaternion, then, say, uMatriks. I can understand not standardising it in the spec but something like "common practice" wouldn't hurt at all.

ara4n commented 6 years ago

i think that letting the accent colour for a room be defined matrixwide is a fine thing; similar to how caldav lets you specify the default colour serverside for a calendar. for now i’d steal the im.vector. event and then denamespace it once there’s a stable m. proposal folks are happy with.