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

Thought: Reduce demo size by 84% #103

Closed
wants to merge 1 commit into from
Closed

Conversation

developit
Copy link

Hiya! This is a great project. I was curious if it worked with Preact, so I forked it and aliased the react dependency using aliasify.

Turns out, dropping preact in as a direct replacement for React in the demo reduces the size of app.js from 220kb to 36kb! (from 70kb to 11kb gzipped!) Both gzipped and raw sizes are 84% smaller, with no change in functionality.

Your demo with the change in this PR is published here if you want to check it out.

…ize from 220kb to 36kb (70kb to 11kb gzipped)
@ro-ka
Copy link
Contributor

ro-ka commented Mar 11, 2016

Nice idea, but as this is a React component, I would like to keep React. It’s just a demo. If someone want’s to use Preact, they can. Maybe we can add an information that it is compatible with Preact. What do you think?

@developit
Copy link
Author

Makes sense. A note in the readme or something would do, sure.

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.

None yet

2 participants