Repository navigation
Remove unused @font-face #49
Description
Hi
Great tool. It works great except that it doesn't remove unused CSS items, e.g.
@font-face {
font-family: 'Lato';
font-style: normal;
font-weight: 400;
src: local('Lato Regular'), local('Lato-Regular'), url(data:font/woff;base64,d09GR... <snip/> ...AisA) format('woff');
}As you can see, the font data is encoded and embedded, which makes the spreadsheets massive (blame https://github.com/grammarly !).
Activity
peterbe commentedon Oct 16, 2017
But if your minimal CSS needs something like
body { font-family: 'Lato Regular'}how is it supposed to know what the font is?igitur commentedon Oct 17, 2017
Not sure I understand your question, but if the CSS needs the font, then it shouldn't be removed. Only unused fonts should be removed.
peterbe commentedon Oct 17, 2017
Right, so to know the answer to that you'd need to open each selector block and look for
font-family: 'Lato'right?igitur commentedon Oct 17, 2017
I think so. However, I don't know the CSS spec well enough to be aware of all uses of
@font-face.peterbe commentedon Jan 3, 2018
My new Node based tool (will soon) support this: peterbe/minimalcss#55
It's also based on a proper CSS parser whereas
mincssdepends on regular expressions.igitur commentedon Feb 8, 2018
Am I right that this should be fixed by peterbe/minimalcss@5672b7b ?
peterbe commentedon Feb 8, 2018
@igitur indeed!
igitur commentedon Feb 8, 2018
Sidenote, how do I get mincss to print its version? One would expect something like
mincss --version.peterbe commentedon Feb 8, 2018
Yeah, I would expect that too :)
Maybe next time 715a48f