-
Notifications
You must be signed in to change notification settings - Fork 1
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
Reducing the size of can-stache-element #94
Comments
I would categorize the bloat into a few categories
I think the first 3 are the easiest to tackle, especially (1) and (3). Making things more tree-shakable may not be super hard but probably can't be done without a breaking change. |
For example, we can get rid of can-construct by removing it from use in can-simple-map. It appears that the only feature of can-construct that SimpleMap is using is the setup method. Can we replace this with a constructor? |
Are those module numbers minified/gzipped? I think that’s important to knowing what to target. Inline docs would add a lot that would be pulled out. |
Btw, I might have created an issue with these numbers. |
The module numbers are using that steal script you wrote. It would be hard to know the weight min and gzipped due to tree shaking. |
At Present i am trying to evaluate a can-stache version based on mustache it would be nice to be able to SSR stache templates as string only representation without the glue code for livebindings. i try to support as much of the syntax that can-stache supports so that the templates are compatible a realy basic implamentation to show how i handle this.property references.
|
can
contains a lot of stuff that isn't always necessary, like can-query-logic and can-connect (and related packages). StacheElement can and should be an entry point to using canjs. We should work to making StacheElement as minimal size wise as possible.I created a build in this branch: https://github.com/canjs/can-stache-element/tree/size
Here are the results:
And here are the two 20 largest modules by size:
The text was updated successfully, but these errors were encountered: