Skip to content
This repository has been archived by the owner on Jun 3, 2021. It is now read-only.

[optimize] JS bundle so big when add bindingx . #191

Closed
PerseveranceZ opened this issue Mar 21, 2018 · 8 comments
Closed

[optimize] JS bundle so big when add bindingx . #191

PerseveranceZ opened this issue Mar 21, 2018 · 8 comments

Comments

@PerseveranceZ
Copy link

1.Your development environment(weex、weex-toolkit、system and more).
  • webpack 3.2.0
  • .babelrc
//   babelrc
{
  "presets": [
      ["env", {
          "targets": {
              "modules": false,
              "loose": true,
              "browsers": ["> 1%", "last 2 versions", "not ie <= 8"]
          }
      }],
      "stage-0"
  ],
  "plugins": [
    ["component",{
        "libraryName": "weex-ui",
        "libDir": "packages",
        "style": false
    }],
    ["transform-runtime", {
      "polyfill": false,
      "helpers": false
    }]
  ],
  "env": {
      "test": {
          "presets": ["env", "stage-0"],
          "plugins": ["istanbul"]
      }
  }
}
2.Expected behavior and actual behavior.

I want to know why my js bundles so big and how to optimize. Thanks.

3.Steps to reproduce the problem.

when i use weex-ui without bindingx , compiled js bundle info:
image

when i use [email protected] with weex-bindingx , compiled js bundle info:
image

docs it work in weex-ui without weex-bindingx.

@PerseveranceZ
Copy link
Author

PerseveranceZ commented Mar 22, 2018

oh, many components has imported util.js and util.js imported weex-bindingx.

@tw93
Copy link
Member

tw93 commented Mar 22, 2018

Thank you for your feedback. We have optimized today.
Please upgrade weex-ui to 0.5.4beta-1 version, At the same time determine whether to open this exclude: /node_modules(?!(\/|\\).*(weex).*)/
Please try again. I'll think about the package optimization.

@tw93
Copy link
Member

tw93 commented Mar 22, 2018

oh, many component import util.js and util import weex-bindingx.

oh,My carelessness

Update weex-ui to @0.6.0-beta, Try it

@PerseveranceZ
Copy link
Author

PerseveranceZ commented Mar 22, 2018

@tw93 Thank you very much, It's a lot smaller.
image
I have another question, you can see js/pages/demo/other/weexui.js, this js bundle is be compiled by code, I think it's a lot bigger, do you think?

About weex-bindingx:
I think that rather than dealing with one file's compatible logic, it's better to divide the three files to process their respective logic separately and to quote individually, which will reduce the size of some components.

Sometimes, I really just need binding-parser and some simple compatible code like this in weex pure native project.

import { parse } from 'bindingx-parser'
import _cloneDeep from 'lodash/cloneDeep'

const WeexBinding = weex.requireModule('bindingx')
const BindingxFunction = WeexBinding.bind

let _WeexBinding = _cloneDeep(WeexBinding)

// 重写 bind 方法
_WeexBinding.bind = (options, callback) => {
    if (!options) {
        throw new Error('should pass options for binding')
    }

    options.exitExpression = formatExpression(options.exitExpression)

    if (options.props) {
        options.props.forEach((prop) => {
            prop.expression = formatExpression(prop.expression)
        })
    }

    return BindingxFunction(options, options && options.eventType === 'timing' ? fixCallback(callback) : callback)
}

module.export =  _WeexBinding

@huxiaoqi567
Copy link

@PerseveranceZ thank you for your advice. bindingx-parser is an easy way to parse expression that you just write the original expression. I'd build multiple entry files in [email protected], you can quote individually. Just like require('weex-bindingx/lib/index.weex.js') (about 18kb) on weex and require('weex-bindingx/lib/index.web.js') (about 80kb) on web .

@PerseveranceZ
Copy link
Author

@huxiaoqi567 @tw93 Perfect! thanks very much.

@tw93
Copy link
Member

tw93 commented Mar 23, 2018

In addition, you can upgrade weex-ui version to [email protected], currently only using weex of bindingx, Maybe a little bit smaller.

@PerseveranceZ
Copy link
Author

PerseveranceZ commented Mar 23, 2018

@tw93 Cooool~

Finally JS bundle size:

  • dev compile:
    image
  • build compile:
    image

@tw93 @huxiaoqi567 thanks for your help!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants