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

Should freezing still leave the features togglable? #19

Open
madig opened this issue May 13, 2020 · 4 comments
Open

Should freezing still leave the features togglable? #19

madig opened this issue May 13, 2020 · 4 comments

Comments

@madig
Copy link
Collaborator

madig commented May 13, 2020

If I freeze ss01 in a font, should toggling it do the reverse?

I think this was once the case but I might be misremembering...

@twardoch
Copy link
Owner

I think it would make sense. This would of course require reversing the sub rules. In my initial implementation I only remapped the Unicodes, so the original encoded glyphs are orphaned.

But if you do modify GSUB, make it optional. I use the freezer to build TTCs: Basically, alternative cmap & name tables are added as extra "font API" tables but the rest remains the same. This makes the TTC only negligibly larger than the original font. But with changed GSUBs, this would increase the size a lot.

@madig
Copy link
Collaborator Author

madig commented May 14, 2020

Mh. Wouldn't it be enough to swap the sub ... by ... lookups inside the feature? 🤔

@twardoch
Copy link
Owner

twardoch commented May 14, 2020

Yeah, that's what I mean. I've never done this and if you feel like doing it, please kindly make this optional (onable or offable, I don't care) for the TTC reasons ☺️

@madig
Copy link
Collaborator Author

madig commented May 14, 2020

Just wondering, would it increase TTC size because you might not want to apply it to all contained fonts?

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

2 participants