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

Removed duplicate setting for toggling Channel Point Redeemed Message highlights #3296

Conversation

Felanbird
Copy link
Collaborator

Pull request checklist:

  • CHANGELOG.md was updated, if applicable

Description

While trying to figure out the issue from #3293, I added a new general setting to toggle First Message Highlights, and because of this noticed that this setting existed, despite already being handled in HighlightModel.cpp

@Felanbird Felanbird changed the title Remove duplicate setting for toggling higlights for 'Channel Point Redeemed Messages' Removed duplicate setting for toggling Channel Point Redeemed Message highlights Oct 19, 2021
@Felanbird
Copy link
Collaborator Author

relevant code that already handled it

else if (rowIndex == 3)
{
getSettings()->enableRedeemedHighlight.setValue(
value.toBool());
}

Copy link
Contributor

@xHeaveny xHeaveny left a comment

Choose a reason for hiding this comment

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

looks good

@pajlada pajlada enabled auto-merge (squash) October 23, 2021 09:50
@pajlada pajlada merged commit 15f7e65 into Chatterino:master Oct 23, 2021
zneix added a commit to SevenTV/chatterino7 that referenced this pull request Oct 25, 2021
Now we're on commit 4a90f10; Changes from upstream we've pulled:

- Minor: Added new search predicate to filter for messages matching a regex (Chatterino#3282)
- Minor: Subcategories in settings are now searchable. (Chatterino#3157)
- Minor: Highlight usernames in /mods and /vips messages (Chatterino#3187)
- Minor: Added chatter count to viewer list popout (Chatterino#3261)
- Minor: Added `flags.first_message` filter variable (Chatterino#3292)
- Minor: Removed duplicate setting for toggling `Channel Point Redeemed Message` highlights (Chatterino#3296)
- Bugfix: Fixed a bug that caused zero-width emotes to be misaligned when the "Remove spaces between emotes" setting is on. (Chatterino#3249)
- Bugfix: Fixed the "Change channel" popup showing a wrong window title (Chatterino#3273)
- Bugfix: Fixed built-in Chatterino commands not working in whispers and mentions special channels (Chatterino#3288)
- Bugfix: Fixed `QCharRef with an index pointing outside the valid range of a QString` warning that was emitted on every Tab press. (Chatterino#3234)
- Bugfix: Fixed being unable to disable `First Message` highlights (Chatterino#3293)
- Bugfix: Fixed `First Message` custom sound not persisting through restart. (Chatterino#3303)
@Felanbird Felanbird deleted the chore/remove-duplicate-redeem-highlight-setting branch January 13, 2022 00:08
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