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

Large SVGs in Rewards should be external asset #267

Open
petemill opened this issue Nov 5, 2018 · 0 comments
Open

Large SVGs in Rewards should be external asset #267

petemill opened this issue Nov 5, 2018 · 0 comments
Assignees

Comments

@petemill
Copy link
Member

@petemill petemill commented Nov 5, 2018

There are some large ~50kb assets that are included in the bundle. These would benefit from being external assets to consuming bundle sizes (especially valid on Android).

Inline SVGs are very useful when they are component-like, or for some other reason need to be styled with CSS. These assets seem like graphics and therefore do not need to be included. If they did, then I would suggest using import() or React.lazy() syntax so that they are contained in a different JS file.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
4 participants
You can’t perform that action at this time.