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

Replace unqualified emojis with fully qualified counterparts #175

Merged
merged 1 commit into from
Nov 30, 2023

Conversation

emesterhazy
Copy link
Contributor

The picker is currently using unqualified and minimally-qualified Unicode sequences for some emojis, which may cause rendering issues on certain browsers and devices. This commit replaces all of the unqualified emojis with their fully qualified counterparts based on the list of emojis published by unicode.org.

FIXED=#173
BUG=#163

The picker is currently using unqualified and minimally-qualified Unicode
sequences for some emojis, which may cause rendering issues on certain browsers
and devices. This commit replaces all of the unqualified emojis with their
fully qualified counterparts based on the list of emojis published by
unicode.org.

- https://www.unicode.org/Public/emoji/15.1/emoji-test.txt

FIXED=Fintasys#173
BUG=Fintasys#163
@Fintasys
Copy link
Owner

I didn't know that's even a thing. Thanks for pointing that out and provide updates for all affected emojis! This alone unfortunately didn't fix the biggest display errors we face in some emoji, but I will also check your other PR!

@Fintasys Fintasys merged commit 380cd4d into Fintasys:master Nov 30, 2023
1 check passed
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.

2 participants