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

Requirement of inflate/pako #58

Open
themesociety opened this issue Mar 17, 2021 · 5 comments
Open

Requirement of inflate/pako #58

themesociety opened this issue Mar 17, 2021 · 5 comments

Comments

@themesociety
Copy link

I was testing now with a lot of fonts and i never got any errors when just removing inflate/pako. Is there a way to trigger this or is it known for which use cases i need it?

Thanks a lot for the work on Wakamai, helped a lot to integrate lib-font in our project.

@RoelN
Copy link
Contributor

RoelN commented Mar 18, 2021

I observed this too: Pomax/lib-font#66 It's still on the to-figure-out-list :-)

If there's anything you can publicly share, @themesociety, I'd love to hear more about your project!

@themesociety
Copy link
Author

Hi @RoelN,

that is good to know, i will keep it removed for now since i wasn't getting in any trouble so far.

I will use the lib-font for semplice.com, a portfolio website builder. At the moment we are adding support for variable webfonts and the problem was that after uploading a font you first had to set it to variable and then you would need to find (which is sadly not as easy as you may think) the axis information from the vendor and add them manually one by one.

Now with lib-fonts you can just drag and drop your fonts in and you are done. Your project was a great inspiration for that and it was even better to discover that you shared on how you achieved it.

Since i only need it for the fvar table i removed all other tables.

Here you can see it in action: https://share.getcloudapp.com/llu9PJjG

@RoelN
Copy link
Contributor

RoelN commented Mar 24, 2021

@themesociety That is really cool! Nice work, looks slick. Great to see @Pomax' excellent work being put to good use.

@themesociety
Copy link
Author

Thank you @RoelN! You are right that @Pomax really did an awesome work, i hope we can use more features of the lib in the future besides getting the axis from variable webonts.

@Pomax
Copy link

Pomax commented Mar 26, 2021

Note that I have a question for both of you in Pomax/lib-font#66 though =)

WOFF/WOFF2 permits compression, but it does not require it, so are we sure that the fonts being used are actually using compression? (because if not, it will 100% look like inflate/pako aren't necessary, right up to the point where you try to load in a per-table compressed font, and the parser starts to interpret what is effectively garbage data as if it's real data ;)

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

No branches or pull requests

3 participants