element-hq / element-web

A glossy Matrix collaboration client for the web.
https://element.io
Apache License 2.0
10.96k stars 1.94k forks source link

CSS improvement: roomlist-highlights-color and roomlist-selected-color #25575

Open Kommynct opened 1 year ago

Kommynct commented 1 year ago

Your use case

Right now both when you hover over a room, and when a room is selected, the background is set to

--roomlist-highlights-color

This should be split into two variables, that would allow for better theming (I suggest --roomlist-highlights-color and --roomlist-selected-color)

Perhaps the selected one should be set to the accent color, but it'd be better to be unique for custom themes.

t3chguy commented 1 year ago

@gsouquet worth considering during your colour pass

germain-gg commented 1 year ago

Unlikely we will get to refactor this as part of the colour pass. We're doing the smallest possible changeset to allow for Compound components to be included in EW