Add chatter count to viewer list popout #3261
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
|
zneix
requested changes
Oct 1, 2021
|
IMO you should also format the number to use short notation, ie 1k instead of 1000. There is already a helper function for this. |
Co-authored-by: Paweł <zneix@zneix.eu>
Using the existing helper function rounded the chatter count too much for my liking, however in the latest commit I pushed I localized the chatter count to match the /chatters command |
pajlada
approved these changes
Oct 24, 2021
1 task
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)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Pull request checklist:
CHANGELOG.mdwas updated, if applicableDescription
Closes #2993
Adds the chatter count to the viewer list popout
