rootValue for options.px2rem shouldn't be changed.

Author: MrRaindrop

README.md

@@ -39,7 +39,7 @@ const precompile = require('weex-vue-precompiler')(/*optional config*/)
 
 <!-- * `preservedTags`: the preserved weex components tag list. The default value is: `['a','container','div','image','img','text','input','switch','list','scroller','waterfall','slider','indicator','loading-indicator','loading','refresh','textarea','video','web']`. If you have other components as plugins installed in weex, you should add them to this lists, add pass the whole list to this. -->
 * `autoprefixer`: options for [autoprefixer](https://github.com/postcss/autoprefixer). default is { browsers: ['> 0.1%', 'ios >= 8', 'not ie < 12'] }.
-* `px2rem`: options for [postcss-plugin-px2rem](https://github.com/ant-tool/postcss-plugin-px2rem). default is: { rootValue: 75 }.
+* `px2rem`: options for [postcss-plugin-px2rem](https://github.com/ant-tool/postcss-plugin-px2rem). default is: { rootValue: 75 }. (NOTICE: We shouldn't change the value of rootValue here. If you want to change the viewport width, you better use `<meta name="weex-viewport" content="...">` in your html entry file.)
 * `aliweex`: boolean. default is false. Tell whether aliweex components are included in your project.
 
 ## feature

package.json

@@ -1,6 +1,6 @@
 {
   "name": "weex-vue-precompiler",
-  "version": "0.1.18",
+  "version": "0.1.19",
   "description": "a precompiler for weex-vue-render.",
   "main": "src/index.js",
   "scripts": {

src/index.js

@@ -10,6 +10,27 @@ const arrayMergeOpts = [
   'aliweexComponents'
 ]
 
+const mergeStrats = {
+  px2rem: function (thisConfig, config) {
+    const { px2rem } = config
+    const { px2rem: thisPx2rem } = thisConfig
+    if (px2rem) {
+      for (k in px2rem) {
+        if (k === 'rootValue') {
+          // ignore rootValue. Always use 750. Why? We use meta[name=weex-viewport] and
+          // meta.setViewport API to set design viewport width, and they would be broken if
+          // we use other rootValue here.
+          continue
+        }
+        else if (px2rem.hasOwnProperty(k)) {
+          thisPx2rem[k] = px2rem[k]
+        }
+      }
+    }
+    return thisPx2rem
+  }
+}
+
 function mergeConfig(thisConfig, config) {
   if (config) {
     const {
@@ -22,7 +43,7 @@ function mergeConfig(thisConfig, config) {
     // merge all fields except arrays.
     for (const k in config) {
       if (arrayMergeOpts.indexOf(k) <= -1) {
-        thisConfig[k] = config[k]
+        thisConfig[k] = mergeStrats[k] ? mergeStrats[k](thisConfig, config): config[k]
       }
     }