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

[app] Improve Emoji Display #187

Merged
merged 2 commits into from
Feb 3, 2022
Merged

Conversation

alexdanilowicz
Copy link
Collaborator

@alexdanilowicz alexdanilowicz commented Jan 5, 2022

Problem

Certain emojis sent/received over iMessage may not be on system yet. (iOS14 may have a new emoji, but the laptop's emoji system may not be up to date.) See #30

Solution

Improve the display of emojis by using the awesome unicode-emoji package, which is basically a massive dataset of emojis and their description (among other things). We use it in the frontend to detect if an emoji version is older than 12.1. We can change this later to detect what emoji version is on the system. And no more hard-coded emoji lists. Yay! 🐶 🎊

Resolves #30.

Screenshots

New

Screen Shot 2022-01-04 at 10 44 41 PM

Old

111062292-db895400-845c-11eb-9373-2044e3efa9ff

Improving the display of emojis by using the awesome unicode-emoji package, which is basically a massive dataset of emojis and their description. We use it in the frontend to detect if an emoji version is older than 12.1. This change be changed later to detect what is on the system. Resolves #30
@vercel
Copy link

vercel bot commented Jan 5, 2022

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/leftonread/leftonread-web/Dt8P1ks1NHGZq4BNJeGepNonEJ3v
✅ Preview: Canceled

[Deployment for 156ee56 canceled]

Copy link
Collaborator

@Teddarific Teddarific left a comment

Choose a reason for hiding this comment

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

Lgtm

@alexdanilowicz alexdanilowicz merged commit 15dbbb3 into main Feb 3, 2022
@alexdanilowicz alexdanilowicz deleted the alex/issue-30/improve-emoji branch February 3, 2022 23:19
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.

How to read emojis
2 participants