Page MenuHomePhabricator

Remove unused JS, optimize background images
ClosedPublic

Authored by chad on Nov 22 2013, 12:37 AM.
Tags
None
Referenced Files
F22998573: D7623.id.diff
Sun, Nov 15, 1:21 PM
F22997971: D7623.diff
Sun, Nov 15, 1:27 AM
F22997939: D7623.id17213.diff
Sat, Nov 14, 11:28 PM
F22997938: D7623.id17213.diff
Sat, Nov 14, 11:28 PM
Unknown Object (File)
Wed, Nov 11, 6:04 PM
Unknown Object (File)
Wed, Nov 11, 6:04 PM
Unknown Object (File)
Mon, Oct 26, 6:59 PM
Unknown Object (File)
Sun, Oct 25, 2:04 PM

Details

Reviewers
epriestley
Commits
Restricted Diffusion Commit
Summary

We're not using the JS (300K) and the images are chubby (300K). Makes the pages about 500K lighter.

Test Plan

Photoshop, Yslow, Reload the site in my sandbox.

Diff Detail

Branch
optims
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

This is for dotorg.

BTW is says we're not gzipping CSS.

BTW is says we're not gzipping CSS.

What says this? It looks compressed to me:

{F83930}

haha looks fine now to me. i will continue to blame everything on ghosts.

templates/phabricator-org/bits/base.html
47–48

The bootstrap JS requires jQuery, so it seems that both should be removed if it's unused?

erm, no it seems I need it on the tour. good catch.