From 4c4b44df7badc28ba87d538a34c2239b93098fa5 Mon Sep 17 00:00:00 2001
From: neverland <chenjiahan@buaa.edu.cn>
Date: Tue, 15 Jan 2019 22:16:14 +0800
Subject: [PATCH] [improvement] ImagePreview: jsx (#2531)

---
 .../{ImagePreview.vue => ImagePreview.js}     | 112 +++++++++---------
 .../test/__snapshots__/index.spec.js.snap     |   8 +-
 2 files changed, 56 insertions(+), 64 deletions(-)
 rename packages/image-preview/{ImagePreview.vue => ImagePreview.js} (79%)

diff --git a/packages/image-preview/ImagePreview.vue b/packages/image-preview/ImagePreview.js
similarity index 79%
rename from packages/image-preview/ImagePreview.vue
rename to packages/image-preview/ImagePreview.js
index 5c85db952..69f3decf1 100644
--- a/packages/image-preview/ImagePreview.vue
+++ b/packages/image-preview/ImagePreview.js
@@ -1,53 +1,10 @@
-<template>
-  <transition name="van-fade">
-    <div
-      v-if="value"
-      :class="[b(), className]"
-      @touchstart="onWrapperTouchStart"
-      @touchend="onWrapperTouchEnd"
-      @touchcancel="onWrapperTouchEnd"
-    >
-      <div
-        v-if="showIndex"
-        :class="b('index')"
-      >
-        {{ active + 1 }}/{{ count }}
-      </div>
-      <swipe
-        ref="swipe"
-        :loop="loop"
-        indicator-color="white"
-        :initial-swipe="startPosition"
-        :show-indicators="showIndicators"
-        @change="onChange"
-      >
-        <swipe-item
-          v-for="(item, index) in images"
-          :key="index"
-        >
-          <img
-            :class="b('image')"
-            :src="item"
-            :style="index === active ? imageStyle : null"
-            @touchstart="onTouchStart"
-            @touchmove="onTouchMove"
-            @touchend="onTouchEnd"
-            @touchcancel="onTouchEnd"
-          >
-        </swipe-item>
-      </swipe>
-    </div>
-  </transition>
-</template>
-
-<script>
-import create from '../utils/create';
+import { use, range } from '../utils';
 import Popup from '../mixins/popup';
 import Touch from '../mixins/touch';
 import Swipe from '../swipe';
 import SwipeItem from '../swipe-item';
-import { range } from '../utils';
 
+const [sfc, bem] = use('image-preview');
 const MAX_ZOOM = 3;
 const MIN_ZOOM = 1 / 3;
 
@@ -60,16 +17,9 @@ function getDistance(touches) {
   );
 }
 
-export default create({
-  name: 'image-preview',
-
+export default sfc({
   mixins: [Popup, Touch],
 
-  components: {
-    Swipe,
-    SwipeItem
-  },
-
   props: {
     images: Array,
     asyncClose: Boolean,
@@ -110,10 +60,6 @@ export default create({
   },
 
   computed: {
-    count() {
-      return this.images.length;
-    },
-
     imageStyle() {
       const { scale } = this;
       const style = {
@@ -262,6 +208,56 @@ export default create({
       this.moveX = 0;
       this.moveY = 0;
     }
+  },
+
+  render(h) {
+    if (!this.value) {
+      return;
+    }
+
+    const { active, images } = this;
+
+    const Index = this.showIndex && (
+      <div class={bem('index')}>{`${active + 1}/${images.length}`}</div>
+    );
+
+    const Images = (
+      <Swipe
+        ref="swipe"
+        loop={this.loop}
+        indicator-color="white"
+        initial-swipe={this.startPosition}
+        show-indicators={this.showIndicators}
+        onChange={this.onChange}
+      >
+        {images.map((item, index) => (
+          <SwipeItem>
+            <img
+              class={bem('image')}
+              src={item}
+              style={index === active ? this.imageStyle : null}
+              onTouchstart={this.onTouchStart}
+              onTouchmove={this.onTouchMove}
+              onTouchend={this.onTouchEnd}
+              onTouchcancel={this.onTouchEnd}
+            />
+          </SwipeItem>
+        ))}
+      </Swipe>
+    );
+
+    return (
+      <transition name="van-fade">
+        <div
+          class={[bem(), this.className]}
+          onTouchstart={this.onWrapperTouchStart}
+          onTouchend={this.onWrapperTouchEnd}
+          onTouchcancel={this.onWrapperTouchEnd}
+        >
+          {Index}
+          {Images}
+        </div>
+      </transition>
+    );
   }
 });
-</script>
diff --git a/packages/image-preview/test/__snapshots__/index.spec.js.snap b/packages/image-preview/test/__snapshots__/index.spec.js.snap
index c1d1a3ba0..4b9780461 100644
--- a/packages/image-preview/test/__snapshots__/index.spec.js.snap
+++ b/packages/image-preview/test/__snapshots__/index.spec.js.snap
@@ -2,9 +2,7 @@
 
 exports[`render image 1`] = `
 <div class="van-image-preview" name="van-fade">
-  <div class="van-image-preview__index">
-    1/3
-  </div>
+  <div class="van-image-preview__index">1/3</div>
   <div class="van-swipe">
     <div class="van-swipe__track" style="width: 0px; transform: translateX(0px);">
       <div class="van-swipe-item" style="width: 0px; height: 100%; transform: translateX(0px);"><img src="https://img.yzcdn.cn/1.png" class="van-image-preview__image"></div>
@@ -18,9 +16,7 @@ exports[`render image 1`] = `
 
 exports[`zoom 1`] = `
 <div class="van-image-preview" name="van-fade">
-  <div class="van-image-preview__index">
-    1/3
-  </div>
+  <div class="van-image-preview__index">1/3</div>
   <div class="van-swipe">
     <div class="van-swipe__track" style="width: 300px; transform: translateX(0px);">
       <div class="van-swipe-item" style="width: 100px; height: 100%; transform: translateX(0px);"><img src="https://img.yzcdn.cn/1.png" class="van-image-preview__image" style="transform: scale3d(2, 2, 1) translate(0px, NaNpx);"></div>