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

Missing apple emoji #153

Closed
wants to merge 1 commit into from
Closed

Missing apple emoji #153

wants to merge 1 commit into from

Conversation

jwheare
Copy link
Contributor

@jwheare jwheare commented May 18, 2019

Not sure what was different but on both master and #152 there were a bunch of the new images missing, looks like anything with a skin tone modifier, the pirate flag and the chess pawn. See the question marks in the apple sheets, _UNKNOWN.png files in img-apple-160, has_img_apple=false in the json etc. When I run the extract.pl script on the font file locally I do get these images generated.

Possibly an outdated font file, or possibly a missed re-run of build_map.php as outlined in build/apple/README.md

Anyway here are the missing apple images, new sheets, json and preview table, optimised using pngquant@2.12.2 and ImageOptim@1.8.8 on a mac.

@jwheare jwheare mentioned this pull request May 18, 2019
@rugk
Copy link
Contributor

rugk commented Jun 22, 2019

@iamcal Anything preventing this from getting merged? As far as I see this here blocks a release with Emoji 11 support. (#145)

@vanniktech
Copy link

Friendly ping since one of my libraries also depends on this. https://github.com/vanniktech/Emoji

@iamcal
Copy link
Owner

iamcal commented Jan 10, 2020

This is fixed in the v12 branch - see #165

@iamcal iamcal closed this Jan 10, 2020
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.

4 participants