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

Fix zero-width emotes alignment with removeSpacesBetweenEmotes setting #3249

Merged
merged 3 commits into from
Oct 24, 2021

Conversation

talneoran
Copy link
Contributor

@talneoran talneoran commented Sep 21, 2021

Pull request checklist:

  • CHANGELOG.md was updated, if applicable

Description

Included a small refactor to only check zero-width flag once.

Fixes #3242

@Mm2PL Mm2PL self-requested a review September 21, 2021 18:25
@talneoran talneoran changed the title Fix zero-width emotes allignment with removeSpacesBetweenEmotes setting Fix zero-width emotes alignment with removeSpacesBetweenEmotes setting Sep 21, 2021
@pajlada pajlada enabled auto-merge (squash) October 24, 2021 12:11
@pajlada pajlada merged commit 5112ec7 into Chatterino:master Oct 24, 2021
@talneoran talneoran deleted the fix/zero-width-remove-spaces branch October 24, 2021 16:28
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
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[bug] Zero-width emotes misaligned when spaces removed.
4 participants