-
Notifications
You must be signed in to change notification settings - Fork 493
Pass BASE table through subsetter by default. #1866
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
base: main
Are you sure you want to change the base?
Conversation
can you expand a little bit about the rationale for this change? |
I can't see any reason why the BASE table would be ignored by default. I was subsetting a font with BASE table and with no explanation the BASE table was just dropped. Even with |
ok yes, dropping by default may be too blunt.
hm that's not good, indeed. Can we just fix that? Ideally the subsetter would trim the BASE script coverage to match the subset. But I have to refresh my memory about BASE table to see how that would work. |
Yes, I guess ideally it would trim to match the subset if necessary. |
I think here we should simply remove BASE from the |
Even though it's cumbersome, this should currently work to prevent subsetter to drop BASE table:
|
I'm happy to pass it through by default, since that does produce a correct font. @punchcutter can you rebase this plese? |
317c9c9
to
3ba00d5
Compare
3ba00d5
to
b56730d
Compare
This would be very helpful for some of the projects I'm helping to maintain. I'll take a look at rebasing this next week perhaps |
#3949 was merged so this PR can be closed now. Thanks @punchcutter for the original PR, my rebase retained your authorship 🚀 |
No description provided.