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

Make "AutoMod Caught Messages" highlight color & "show in mentions" configurable #5206

Closed
4 tasks done
djinnet opened this issue Feb 26, 2024 · 6 comments · Fixed by #5215
Closed
4 tasks done

Make "AutoMod Caught Messages" highlight color & "show in mentions" configurable #5206

djinnet opened this issue Feb 26, 2024 · 6 comments · Fixed by #5215
Labels
enhancement Feature: Highlights Issues related to highlights and ping sounds

Comments

@djinnet
Copy link

djinnet commented Feb 26, 2024

Checklist

  • I'm reporting a problem with Chatterino
  • I've verified that I'm running the most recent nightly build or stable release
  • I've looked for my problem on the wiki
  • I've searched the issues and pull requests for similar looking reports

Describe your issue

I cannot change the highlight color to any other colors in highlight page under messages tab.
Its color picker is disabled for no reasons and it makes it harder to spot monitored or restricted users when it is blended with the background.

Screenshots

image

As you can see here; A has a color picker that allowed them to have different colors.
However B is disabled and has the same color as the darkmode's background and if the user of the program did not pay attention to this behavior, they can missed the message entirely.

OS and Chatterino Version

Chatterino Nightly 2.4.6 (commit 0f42df7) built on 2024-02-26 with Qt 6.5.0, MSVC 193833135 Running on Windows 10 Version 22H2, kernel: 10.0.19045

@djinnet djinnet added the issue-report An issue reported by a user. label Feb 26, 2024
@pajlada pajlada changed the title Cannot change color on AutoMod Caught Messages in highlight page & cannot set monitored or restricted users to show in mention. Make "AutoMod Caught Messages" highlight color configurable Feb 26, 2024
@pajlada pajlada added enhancement Feature: Highlights Issues related to highlights and ping sounds and removed issue-report An issue reported by a user. labels Feb 26, 2024
@pajlada pajlada changed the title Make "AutoMod Caught Messages" highlight color configurable Make "AutoMod Caught Messages" highlight color & "show in mentions" configurable Feb 26, 2024
@pajlada
Copy link
Member

pajlada commented Feb 26, 2024

I saw 3 separate feature requests here so I split your issue up

  1. Make "AutoMod Caught Messages" configurable (highlight color & show in mentions)
  2. Make "Restricted users" similary configurable Make "Restricted users" highlight color & "show in mentions" configurable #5207
  3. Add support for "Monitored users" Add support for "Monitored users" #5208

In the interim for this issue, there's a separate split called "AutoMod" that shows auto mod messages from every channel you have open that might be helpful

@KleberPF
Copy link
Contributor

Should the allow/deny message also show up in /mentions? If so, should the added/removed term messages also appear?

@pajlada
Copy link
Member

pajlada commented Feb 28, 2024

Should the allow/deny message also show up in /mentions?

I think yes

If so, should the added/removed term messages also appear?

I don't think this is important, no

Do you have any thoughts on this @djinnet?

@djinnet
Copy link
Author

djinnet commented Feb 28, 2024

I 100% agreed on the first one. @pajlada That could make life easier for moderation purpose in a massive community.

But for second one; I honestly dont know.
That depends on if it is easy to implemented, but perhaps that could be a choice for the end-user to have that appear or not.
After all many people are using chatterino with different setups and perhaps some may need it.
But that is a "nice-to-have" from my point of view if that makes sense? I would love to hear more thoughts on that.

@KleberPF
Copy link
Contributor

@djinnet It shouldn't be too hard to include it there. My PR for now doesn't include it, but I will probably add it later if pajlada wants.

@pajlada
Copy link
Member

pajlada commented Feb 28, 2024

Let's leave it out for now & wait if people think it's missing

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Feature: Highlights Issues related to highlights and ping sounds
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants