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

feat(roleColorEverywhere): color blending from multiple roles #3034

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

Conversation

EnergoStalin
Copy link

@EnergoStalin EnergoStalin commented Nov 22, 2024

It was sitting in my repo long time soo let's try to get it upstream.

Hit me if some changes required.

This enable automatically blending colors from selected roles on the server.

To show highlighted roles on the server.

image

This will look like this

image

Here you can click role circle to remove from the list.

image

As well as just hit remove from coloring list button which is togglable so role adding is done via this button too.

This is how user looks like.

image

For users with no roles used in coloring list fallback to default user color.

When only one role matches blending skipped and this role color is used.

This solved problem of being forced to look at user everytime to check out their role or multiple roles.

@EnergoStalin EnergoStalin force-pushed the roleColorEverywhere branch 3 times, most recently from 8407a64 to ee3b9d1 Compare December 1, 2024 07:04
@EnergoStalin EnergoStalin force-pushed the roleColorEverywhere branch 3 times, most recently from 47a51b6 to 24d822b Compare December 19, 2024 21:26
@EnergoStalin EnergoStalin force-pushed the roleColorEverywhere branch 3 times, most recently from b1a6033 to 8b48bc0 Compare January 11, 2025 01:10
@EnergoStalin EnergoStalin force-pushed the roleColorEverywhere branch 3 times, most recently from 42a9afe to 042f37e Compare January 26, 2025 22:23
@EnergoStalin EnergoStalin force-pushed the roleColorEverywhere branch 2 times, most recently from 4e546d3 to f31484c Compare January 30, 2025 04:15
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.

1 participant