Skip to content

Conversation

@kardeslik
Copy link
Contributor

@kardeslik kardeslik commented Jul 26, 2022

@novacrazy
Copy link

May as well update this to Unicode 15 sometime.

Copy link
Contributor

@mislav mislav left a comment

Choose a reason for hiding this comment

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

Thank you! Looks great

@mislav
Copy link
Contributor

mislav commented Nov 15, 2022

@novacrazy After Unicode 15 widely ships on devices like smartphones will be the right time to add them to this project. 🙇

@mislav mislav merged commit 9917299 into github:master Nov 15, 2022
@mislav mislav mentioned this pull request Nov 15, 2022
@novacrazy
Copy link

Out of curiosity, what is the source of these? Tags, aliases, etc.

@mislav
Copy link
Contributor

mislav commented Nov 15, 2022

We mostly generate initial aliases from the character descriptions in the Unicode spec. Sometimes we just straight-up invent aliases (for the most common emoji). Tags we add at our own discretion to aid searching in emoji pickers that use this library.

@kardeslik kardeslik deleted the import-emoji-14 branch December 15, 2022 14:51
@kardeslik kardeslik restored the import-emoji-14 branch December 15, 2022 14:51
mrsdizzie added a commit to mrsdizzie/gitea that referenced this pull request Jan 4, 2023
Gitea emoji dataset was out of date because it gets manually build and hasn't been rebuilt since it was added. This means Gitea doesn't recognize some newer emoji or changes to existing ones.

After changing the max unicode version to 14 I just ran: `go run build/generate-emoji.go`

This should address the initial issue seen in go-gitea#22153 where Gitea doesn't recognize a standard alias used elsewhere when importing content.

14 is the latest supported version from the upstream source as 15 is not widely supported (in their opinion) yet: github/gemoji#244 (comment)
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