Layout: optimize doc && code review

2017-08-22 13:31:36 +08:00
parent bafe9142ea
commit d76db3d74a
7 changed files with 92 additions and 108 deletions
@@ -13,27 +13,28 @@
 </template>

 <script>
-  export default {
-    name: 'van-col',
-    props: {
-      span: [Number, String],
-      offset: [Number, String],
-      prefix: {
-        type: String,
-        default: 'van'
-      }
-    },
-    computed: {
-      gutter() {
-        if (!this.$parent) return 0;
-        return Number(this.$parent.gutter) || 0;
-      },
-      style() {
-        const padding = `${this.gutter / 2}px`;
-        return this.gutter
-          ? { paddingLeft: padding, paddingRight: padding }
-          : null;
-      }
+export default {
+  name: 'van-col',
+
+  props: {
+    span: [Number, String],
+    offset: [Number, String],
+    prefix: {
+      type: String,
+      default: 'van'
    }
-  };
+  },
+
+  computed: {
+    gutter() {
+      return (this.$parent && Number(this.$parent.gutter)) || 0;
+    },
+    style() {
+      const padding = `${this.gutter / 2}px`;
+      return this.gutter
+        ? { paddingLeft: padding, paddingRight: padding }
+        : null;
+    }
+  }
+};
 </script>
@@ -1,32 +1,31 @@
 <template>
-  <div
-    :class="`${prefix}-row`"
-    :style="style">
+  <div :class="`${prefix}-row`" :style="style">
    <slot></slot>
  </div>
 </template>

 <script>
-  export default {
-    name: 'van-row',
-    props: {
-      prefix: {
-        type: String,
-        default: 'van'
-      },
-      gutter: {
-        type: [Number, String],
-        default: 0
-      }
+export default {
+  name: 'van-row',
+
+  props: {
+    prefix: {
+      type: String,
+      default: 'van'
    },
-    computed: {
-      style() {
-        const gutter = Number(this.gutter);
-        const margin = `-${gutter / 2}px`;
-        return gutter
-          ? { marginLeft: margin, marginRight: margin }
-          : {};
-      }
+    gutter: {
+      type: [Number, String],
+      default: 0
    }
-  };
+  },
+
+  computed: {
+    style() {
+      const margin = `-${Number(this.gutter) / 2}px`;
+      return this.gutter
+        ? { marginLeft: margin, marginRight: margin }
+        : null;
+    }
+  }
+};
 </script>