shadowfacts/fonttools
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

[subset] keep and don't prune CPAL if OT-SVG table is present

Browse Source 
Fixes #2814

or at least is a workaround until a proper 'fix'
This commit is contained in:
Cosimo Lupo 2022-09-14 18:59:08 +01:00 committed by Behdad Esfahbod
parent 1f8803af96
commit 276f6aaf44
1 changed files with 8 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
Lib/fontTools/subset/__init__.py
View File

@ -2207,6 +2207,14 @@ def subset_glyphs(self, s):
@_add_method(ttLib.getTableClass('CPAL')) @_add_method(ttLib.getTableClass('CPAL'))
def prune_post_subset(self, font, options): def prune_post_subset(self, font, options):
# Keep whole "CPAL" if "SVG " is present as it may be referenced by the latter
# via 'var(--color{palette_entry_index}, ...)' CSS color variables.
# For now we just assume this is the case by the mere presence of "SVG " table,
# for parsing SVG to collect all the used indices is too much work...
# TODO(anthrotype): Do The Right Thing (TM).
if "SVG " in font:
return True
colr = font.get("COLR") colr = font.get("COLR")
if not colr: # drop CPAL if COLR was subsetted to empty if not colr: # drop CPAL if COLR was subsetted to empty
return False return False