Page MenuHomePhabricator

Automatically inline small images using data URIs
ClosedPublic

Authored by epriestley on Feb 14 2014, 1:27 AM.
Tags
None
Referenced Files
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
Unknown Object (File)
Tue, Aug 20, 1:31 AM
Unknown Object (File)
Sun, Aug 18, 9:13 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.