update Emojione assets to latest Emojione version #132
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes #85
I have updated the Emojione related assets to the latest Emojione version, also adding all previously missing images (now all
has_img_emojione
aretrue
😄).First off, I wanted to thank and congratulate you on the magnific work done, all the build process is elegantly created and easy to use. Using all the scripts and everything it's been easy to do this (except one part that I struggled with and I detail below). Cheers!
In the process of doing the update I have done some more necessary changes that I detail here.
Emojione build changes
Starting from Emojione 3.0, they have separated all assets into a dedicated repo, so I added a reference to the repo (submodule) from build/emojione
Also, I have adapted the script that gets the images to reference the new directory and done another change (which took me a while to figure out, that's the thing I struggled with) to transform the filenames in emojine-assets to fully qualified names (ZWJ sequences).
The thing is that emojione has named the files with their short form, for example 🏋🏿♀️ is named 1f3cb-1f3ff-2640.png (missing the optional 200d and fe03 characters). BUT in the catalog we build we search only by the fully qualified name. Fortunately for us, Emojione provides a mapping indicating the fully qualified name for each file. What I have done is to use this mapping to rename the files copied to this repo in their fully qualified form.
build_image changes
When building the images I had a problem with the pipes system, I think it was causing some sort of overflow issue in my machine (Mac OS), so I changed to the "normal" form of calling
montage
by passing the full ordered list of files to the command. I think this is easier to understand and more portable, but I may be perfectly missing something here.bash script changes
Since I am using Mac,
stat -c
was not working since the syntax is different, I just extracted that into a method that executes the correct command depending on whether it's run on Mac or Linux.other things
I have already done the whole build process:
emoji.json
andemoji_pretty.json
(the only changes are thehas_img_emojione
flags).table.htm
(isn't it beautiful that now the Emojione column is full?)