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

Minor: AutoMod term management messages are now hidden in Streamer Mode. #4758

Merged
merged 8 commits into from
Aug 12, 2023

Conversation

Mm2PL
Copy link
Collaborator

@Mm2PL Mm2PL commented Aug 7, 2023

Description

If you add or remove a blocked term, it no longer gets shown in streamer mode.

@Mm2PL Mm2PL requested a review from Felanbird August 7, 2023 21:27
@Mm2PL Mm2PL changed the title Block automod add/remove blocked/permitted term messages in streamer mode Minor: AutoMod term management messages are now hidden in Streamer Mode. Aug 7, 2023
false, "Hide bans & timeouts from appearing in chat.");
layout.addCheckbox(
"Hide moderation actions", s.streamerModeHideModActions, false,
"Hide bans, timeouts and automod messages from appearing in chat.");
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is not technically a good description of what this PR did, BUT this setting already did hide automod messages so now the tooltip is more correct than it was before, and automod messages I believe is blanket enough to cover what we changed here.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
"Hide bans, timeouts and automod messages from appearing in chat.");
"Hide bans, timeouts, and automod messages from appearing in chat.");

actually oxford would be mad at us if we didn't

Copy link
Collaborator

@Felanbird Felanbird left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

works 👍

@Felanbird Felanbird added this to the 2.4.5 milestone Aug 9, 2023
Copy link
Member

@pajlada pajlada left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The change itself isn't major, but it may be a very visible change for streamers so I've elaborated on the changelog entry and moved it to be a Major change.
Alternatively, it could be a Minor change but left at the top instead.

I'm happy to have this merged in whenever someone else confirms my change

@Felanbird Felanbird merged commit 1e35391 into master Aug 12, 2023
15 of 16 checks passed
@Felanbird Felanbird deleted the chore/hide_automod_term_messages branch August 12, 2023 16:16
Nerixyz pushed a commit to Nerixyz/chatterino2 that referenced this pull request Aug 12, 2023
Nerixyz pushed a commit to Nerixyz/chatterino2 that referenced this pull request Aug 19, 2023
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.

3 participants