Page MenuHomePhabricator

Automatically inline small images using data URIs
ClosedPublic

Authored by epriestley on Feb 14 2014, 1:27 AM.
Tags
None
Referenced Files
F13815451: D8225.id19571.diff
Thu, Sep 19, 9:55 AM
Unknown Object (File)
Wed, Sep 18, 4:07 AM
Unknown Object (File)
Wed, Sep 4, 9:57 PM
Unknown Object (File)
Mon, Sep 2, 10:23 AM
Unknown Object (File)
Wed, Aug 28, 6:32 PM
Unknown Object (File)
Tue, Aug 27, 5:41 PM
Unknown Object (File)
Sun, Aug 25, 1:48 AM
Unknown Object (File)
Thu, Aug 22, 5:43 PM
Subscribers

Details

Reviewers
btrahan
chad
Maniphest Tasks
Restricted Maniphest Task
Commits
Restricted Diffusion Commit
rP005d194cd28d: Automatically inline small images using data URIs
Summary

Fixes T3300. Images under 32KB are inlined automatically into CSS using data URIs; larger images remain as normal links. I picked the 32KB threshold arbitrarily, based on it looking roughly like it got reasonable results on core.pkg.css (inlining most of the random stuff, but not inlining all the 2X sprites and such).

Test Plan

Loaded site, browsed around, looked at generated CSS.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley updated this revision to Unknown Object (????).Feb 14 2014, 1:27 AM
  • Minor fix.
epriestley updated this revision to Unknown Object (????).Feb 14 2014, 1:31 AM

Helpfully, I'd already done some research in T3300 and discovered a 32KB limit in IE8, so this limit is now "a hard limit imposed by IE8" instead of "an arbitrary limit I made up".

epriestley updated this revision to Unknown Object (????).Feb 14 2014, 1:32 AM

Slightly clearer comment.

OK, new timeline is pretty awesome.