Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: minimize the amount of CSS assets #882

Merged
merged 1 commit into from
May 19, 2020
Merged

fix: minimize the amount of CSS assets #882

merged 1 commit into from
May 19, 2020

Conversation

lidel
Copy link
Member

@lidel lidel commented May 18, 2020

Closes #881

This PR:

  • remove icons as we dont use them directly anyway
  • let webpack to decide what is common and what is page-specific
  • ignore fonts in formats other than WOFF2
  • switch to uncompressed dependencies
    (leveraging ZIP compression of entire package instead)

Before (v2.11)

10M   chromium.zip
 4M   firefox.zip

Before (v2.11.0.918)

19M   chromium.zip
12M   firefox.zip

After (this PR)

8.6M  chromium.zip
2.0M  firefox.zip

- remove icons as we dont use them directly anyway
- let webpack to decide what is common and what is page-specific
- ignore fonts in formats other than WOFF2
@lidel lidel force-pushed the fix/size-reduction branch from 8cc1d71 to c7d27f3 Compare May 19, 2020 13:26
@lidel lidel marked this pull request as ready for review May 19, 2020 13:40
@lidel lidel merged commit 1e6f42e into master May 19, 2020
@lidel lidel deleted the fix/size-reduction branch May 19, 2020 13:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Trim down unused ipfs-css assets
1 participant