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:
Version: unspecified
Severity: enhancement
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Duplicate | None | T59777 CSSMin::encodeImageAsDataURI() should be smarter about SVG images (tracking) | |||
| Duplicate | None | T65491 Embedded SVG files should be optimized in ResourceLoader/CSSMin | |||
| Resolved | matmarex | T69341 CSSMin: Embed non-binary data URIs using plain text instead of base64 | |||
| Resolved | matmarex | T73003 CSSMin: Don't generate double rules for IE < 8 when embedding SVG files |
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()
Splitting this into two things:
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