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
Non-colored nicknames should be disabled and colors should be theme-specific #45
Comments
Strongly disagree. I don't like colored nicknames in any client, and wouldn't want to modify the theme to change it.
It's not the case, it sets colors using inline attributes. |
If we want to set the colors in the CSS, how many different colors do we want to introduce? Right now I don't think there's a limit since they're randomly generated based on the username. But adding 256 color classes to a css file is a bit overkill I guess. Could we settle somewhere around 30? |
Regarding CSS colors: It's a slightly less-than-trivial decision to make here as there is no real standard for IRC colors. We're already enumerating the 16 mIRC colors in style.css. As far as I can tell, most clients only support these 16 colors. However, HexChat supports 32 colors. Assuming this Shout issue made it to Lounge, we're probably mapping the additional 16 colors down to our original 16. I propose that we take advantage of this existing color pallet for nickname colors as well. I've implemented this locally, and I'm very happy with the result. I would much rather have theme-defined nickname colors than any random nickname colors as the chosen pallet is very much part of the theme's appearance. @xPaw raised the concern on IRC that this might not be enough colors; perhaps to that end we could lean into the 32-color pallet that HexChat defines and not only support more colors in chat but have 32 options for nicknames. |
So, this should be partially fixed by #325. We need to agree on whether we remove the non-colour setting to close out this ticket. I suggest we do not remove the setting, but we default to colours. Anyone in an objection to that? I don't think we want to remove the config to disable colours. |
Assuming that it is just changing the default option for coloured nicknames, that's very easy so I've added the up for grabs label. |
Removing the setting is not a good idea, imo. Default option was already changed to be on by default. |
Ah right. In that case, I'm going to close the issue. If someone wants us to remove the option and has a good reason for it, they can reopen and discuss, but I am strongly against us removing that option. |
Opened this almost 4 months ago, and since then I'm not strongly for removing this setting. Now that it's enabled by default and themes can control both their color palette and the nick-colors-disabled color, it seems fine to me. |
FYI, I haven't looked to see how colors for the colored nicknames are handled.
However, in terms of UX, it's very noisy and insignificant to have a checkbox to enable colored nicknames.
This checkbox should be removed, enabling colored nicknames by default. Then, colors should be set in the CSS file (if it's not already the case) so that a theme can decide they want to disable colored nicknames by setting the same color for the whole range.
This also allows theme creators to have full control over the nickname colors (again, if it's not already the case).
The text was updated successfully, but these errors were encountered: