Page MenuHomePhabricator

Allow users to choo choo choose a room color
ClosedPublic

Authored by chad on Apr 20 2017, 9:22 PM.
Tags
None
Referenced Files
F12850562: D17747.id42686.diff
Fri, Mar 29, 5:33 AM
Unknown Object (File)
Feb 18 2024, 11:25 AM
Unknown Object (File)
Feb 12 2024, 9:34 AM
Unknown Object (File)
Jan 22 2024, 6:19 AM
Unknown Object (File)
Jan 19 2024, 2:34 AM
Unknown Object (File)
Jan 15 2024, 9:55 PM
Unknown Object (File)
Jan 6 2024, 8:45 PM
Unknown Object (File)
Dec 25 2023, 1:18 PM
Subscribers

Details

Summary

This adds some basic per user / per room theming for Conpherence, which should hopefully let users identify rooms from just the sidebar color.

Test Plan

Lots of threads with different colors.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

this was not an hour's work

Seems slightly odd that this is per-user/per-room instead of per-room or "per user, but with a per-room default". How will users fight over whether a room should be peach or indigo?

This revision is now accepted and ready to land.Apr 20 2017, 9:26 PM

I think users who want one color (like a color blind or dark theme) can set their default, or the other use case is identifying rooms visually (so you don't leak private data), which still seems up to the user.

I can't think of any reason room admins would want to set a specific color, other than to add to arguments, like you suggest.

This revision was automatically updated to reflect the committed changes.

I'm only half-trolling. Maybe I want to make "Security Red Team" red, or "Buy/Sell" green. Or we let you set custom images and "Dog Lovers" wants to set dog wallpaper.