Last modified: 2014-09-18 15:12:56 UTC
Thanks to the work of GCI students, we now have lots of SVG images in core (and lots more in extensions). However, they're usually somewhat pretty-formatted, with whitespace and comments. CSSMin::encodeImageAsDataURI() should be smart about this in two ways: * Not base64-encode plain ASCII text, just escape quotes and backslashes * Strip whitespace and comments (and possibly apply other transformations)
Whitespace and comment stripping is unlikely to produce much gain. Stripping other unneeded stuff can produce big gains as we observed in ULS images.
(In reply to comment #1) > Whitespace and comment stripping is unlikely to produce much gain. Stripping > other unneeded stuff can produce big gains as we observed in ULS images. I'm thinking of taking up this bug!! Well what could be that other unneeded stuff that could be striped ??
Change 116490 had a related patch set uploaded by Deepali: Handling svg images in CSSMin::encodeImageAsDataURI() https://gerrit.wikimedia.org/r/116490
Splitting this into two things: * Optimizing SVGs (bug 34812) * Whether to use base 64 (bug 67341)
Change 116490 abandoned by Bartosz Dziewoński: Handling svg images in CSSMin::encodeImageAsDataURI() Reason: No interest from uploader. Superseded by https://gerrit.wikimedia.org/r/161091 https://gerrit.wikimedia.org/r/116490