From 577174c7f87e47efe17676daf7673d69f6516338 Mon Sep 17 00:00:00 2001
From: neverland <chenjiahan@youzan.com>
Date: Mon, 4 Nov 2019 17:58:20 +0800
Subject: [PATCH] feat(Switch): add click event (#4915)

---
 src/switch/README.md          | 33 +++++++------------------
 src/switch/README.zh-CN.md    | 45 ++++++++++++++++-------------------
 src/switch/demo/index.vue     | 42 +++++++++++---------------------
 src/switch/index.tsx          |  4 +++-
 src/switch/test/index.spec.js | 15 ++++++++++++
 5 files changed, 62 insertions(+), 77 deletions(-)

diff --git a/src/switch/README.md b/src/switch/README.md
index 2cee7aa52..0bacb763e 100644
--- a/src/switch/README.md
+++ b/src/switch/README.md
@@ -30,47 +30,31 @@ export default {
 ### Disabled
 
 ```html
-<van-switch
-  v-model="checked"
-  disabled
-/>
+<van-switch v-model="checked" disabled />
 ```
 
 ### Loading
 
 ```html
-<van-switch
-  v-model="checked"
-  loading
-/>
+<van-switch v-model="checked" loading />
 ```
 
 ### Custom Size
 
 ```html
-<van-switch
-  v-model="checked"
-  size="24px"
-/>
+<van-switch v-model="checked" size="24px" />
 ```
 
 ### Custom Color
 
 ```html
-<van-switch
-  v-model="checked"
-  active-color="#07c160"
-  inactive-color="#ee0a24"
-/>
+<van-switch v-model="checked" active-color="#07c160" inactive-color="#ee0a24" />
 ```
 
 ### Async Control
 
 ```html
-<van-switch
-  :value="checked"
-  @input="onInput"
-/>
+<van-switch :value="checked" @input="onInput" />
 ```
 
 ```js
@@ -111,6 +95,7 @@ export default {
 
 ### Events
 
-| Event | Description | Parameters |
-|------|------|------|
-| change | Triggered when check status changed | checked: is switch checked |
+| Event | Description | Parameters | Version |
+|------|------|------|------|
+| change | Triggered when check status changed | checked: is switch checked | - |
+| click | Triggered when clicked | event: Event | 2.2.11 |
diff --git a/src/switch/README.zh-CN.md b/src/switch/README.zh-CN.md
index 25ea3aa7a..13460c7e7 100644
--- a/src/switch/README.zh-CN.md
+++ b/src/switch/README.zh-CN.md
@@ -13,6 +13,8 @@ Vue.use(Switch);
 
 ### 基础用法
 
+通过`v-model`绑定开关的选中状态，`true`表示开，`false`表示关
+
 ```html
 <van-switch v-model="checked" />
 ```
@@ -29,48 +31,42 @@ export default {
 
 ### 禁用状态
 
+通过`disabled`属性来禁用开关，禁用状态下开关不可点击
+
 ```html
-<van-switch
-  v-model="checked"
-  disabled
-/>
+<van-switch v-model="checked" disabled />
 ```
 
 ### 加载状态
 
+通过`loading`属性设置开关为加载状态，加载状态下开关不可点击
+
 ```html
-<van-switch
-  v-model="checked"
-  loading
-/>
+<van-switch v-model="checked" loading />
 ```
 
 ### 自定义大小
 
+通过`size`属性自定义开关的大小
+
 ```html
-<van-switch
-  v-model="checked"
-  size="24px"
-/>
+<van-switch v-model="checked" size="24px" />
 ```
 
 ### 自定义颜色
 
+`active-color`属性表示打开时的背景色，`inactive-color`表示关闭时的背景色
+
 ```html
-<van-switch
-  v-model="checked"
-  active-color="#07c160"
-  inactive-color="#ee0a24"
-/>
+<van-switch v-model="checked" active-color="#07c160" inactive-color="#ee0a24" />
 ```
 
 ### 异步控制
 
+需要异步控制开关时，可以使用`value`属性和`input`事件代替`v-model`，并在`input`事件回调函数中手动处理开关状态
+
 ```html
-<van-switch
-  :value="checked"
-  @input="onInput"
-/>
+<van-switch :value="checked" @input="onInput" />
 ```
 
 ```js
@@ -111,6 +107,7 @@ export default {
 
 ### Events
 
-| 事件名 | 说明 | 回调参数 |
-|------|------|------|
-| change | 开关状态切换回调 | checked: 是否选中开关 |
+| 事件名 | 说明 | 回调参数 | 版本 |
+|------|------|------|------|
+| change | 开关状态切换回调 | checked: 是否选中开关 | - |
+| click | 点击时触发 | event: Event | 2.2.11 |
diff --git a/src/switch/demo/index.vue b/src/switch/demo/index.vue
index 0f115ace9..81bab7f5f 100644
--- a/src/switch/demo/index.vue
+++ b/src/switch/demo/index.vue
@@ -5,39 +5,23 @@
     </demo-block>
 
     <demo-block :title="$t('disabled')">
-      <van-switch
-        v-model="checked"
-        disabled
-      />
+      <van-switch v-model="checked" disabled />
     </demo-block>
 
     <demo-block :title="$t('loadingStatus')">
-      <van-switch
-        v-model="checked"
-        loading
-      />
+      <van-switch v-model="checked" loading />
     </demo-block>
 
     <demo-block :title="$t('customSize')">
-      <van-switch
-        v-model="checked2"
-        size="24px"
-      />
+      <van-switch v-model="checked2" size="24px" />
     </demo-block>
 
     <demo-block :title="$t('customColor')">
-      <van-switch
-        v-model="checked3"
-        active-color="#07c160"
-        inactive-color="#ee0a24"
-      />
+      <van-switch v-model="checked3" active-color="#07c160" inactive-color="#ee0a24" />
     </demo-block>
 
     <demo-block :title="$t('asyncControl')">
-      <van-switch
-        :value="checked4"
-        @input="onInput"
-      />
+      <van-switch :value="checked4" @input="onInput" />
     </demo-block>
   </demo-section>
 </template>
@@ -72,19 +56,21 @@ export default {
 
   methods: {
     onInput(checked) {
-      this.$dialog.confirm({
-        title: this.$t('title'),
-        message: this.$t('message')
-      }).then(() => {
-        this.checked4 = checked;
-      });
+      this.$dialog
+        .confirm({
+          title: this.$t('title'),
+          message: this.$t('message')
+        })
+        .then(() => {
+          this.checked4 = checked;
+        });
     }
   }
 };
 </script>
 
 <style lang="less">
-@import "../../style/var";
+@import '../../style/var';
 
 .demo-switch {
   .van-switch {
diff --git a/src/switch/index.tsx b/src/switch/index.tsx
index cca2b820a..66c163b92 100644
--- a/src/switch/index.tsx
+++ b/src/switch/index.tsx
@@ -40,7 +40,9 @@ function Switch(
 
   const loadingColor = checked ? activeColor || BLUE : inactiveColor || GRAY_DARK;
 
-  function onClick() {
+  function onClick(event: PointerEvent) {
+    emit(ctx, 'click', event);
+
     if (!disabled && !loading) {
       const newValue = checked ? inactiveValue : activeValue;
       emit(ctx, 'input', newValue);
diff --git a/src/switch/test/index.spec.js b/src/switch/test/index.spec.js
index 0d1611e3e..ab816297b 100644
--- a/src/switch/test/index.spec.js
+++ b/src/switch/test/index.spec.js
@@ -82,3 +82,18 @@ test('size prop', () => {
 
   expect(wrapper).toMatchSnapshot();
 });
+
+test('click event', () => {
+  const click = jest.fn();
+  const wrapper = mount(Switch, {
+    context: {
+      on: {
+        click
+      }
+    }
+  });
+
+  wrapper.trigger('click');
+
+  expect(click).toHaveBeenCalledTimes(1);
+});