Skip to content

Conversation

ebraminio
Copy link
Collaborator

Works on https://github.com/symbolset/ss-emoji-beta/blob/master/ss-emoji-adobe-mozilla.ttf and I was able to dump the SVGs inside the font with it!

@ebraminio ebraminio force-pushed the svg branch 3 times, most recently from 566e97e to 9e68399 Compare March 9, 2018 20:37
@ebraminio
Copy link
Collaborator Author

ebraminio commented Mar 9, 2018

Works on https://github.com/adobe-fonts/emojione-color also but needs a gz extractor in order to work.

@ebraminio ebraminio merged commit 218fa71 into harfbuzz:master Mar 10, 2018
@ebraminio ebraminio deleted the svg branch March 10, 2018 07:43
@ebraminio ebraminio changed the title [SVG ] Implementation [ot-color] SVG table implementation Mar 10, 2018
@KrasnayaPloshchad
Copy link

KrasnayaPloshchad commented Mar 10, 2018

So there is a necessary to add hb-gz.cc in order to allow HarfBuzz build with gz library, isn’t it?

@ebraminio
Copy link
Collaborator Author

ebraminio commented Mar 10, 2018

I think the short/medium plan is to provide the very gzipped blob to users so they can decode the blob for themselves and use their own SVG renderer (as librsvg or anything available there) with the provided blobs.

@ebraminio
Copy link
Collaborator Author

ebraminio commented Mar 10, 2018

Oh I think I was unclear on what I meant, I provided a temporarily and commented dump function that saves blobs of the font on out/ directory. The functionality works great on the first font but on the Adobe fonts, it extracts the SVGs as well but they are not in correct format as the need to be extracted first, basically they are SVGZ instead of SVG.

@behdad
Copy link
Member

behdad commented Mar 11, 2018

Yeah, we don't decode it. Just return.

@ebraminio
Copy link
Collaborator Author

The added tables are starving for public API Behdad :) I believe we can start with COLR/CPAL.

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