Follow-up from "Fix icon spacing issues"
The following discussion from !741 (merged) should be addressed:
-
@jerasmus started a discussion In next iterations, we could look into why the
--html-keep-whitespace
solution did not work and perhaps contribute upstream to the minify project to fix the bug if it's a bug on their side.I'll open a follow-up issue to iterate further.
In !741 (merged) we added an MVC fix for the icon spacing issues. I believe there are still some room for improvement.
Possible next steps
- Enable minification on review apps and local dev environments to mirror how to site is compiled and served on production.
- Look into why the
--html-keep-whitespace
solution did not work and perhaps contribute upstream to the minify project to fix the bug if it's a bug on their side.