From 0093e19bf7021d078454ed1286f81dae52f26a3b Mon Sep 17 00:00:00 2001
From: neverland <chenjiahan@youzan.com>
Date: Thu, 4 Mar 2021 19:40:30 +0800
Subject: [PATCH] fix(ActionSheet): should not reopen when closed (#8271)

---
 src/action-sheet/index.tsx                             | 8 ++++++--
 src/action-sheet/test/__snapshots__/index.spec.js.snap | 2 +-
 src/action-sheet/test/index.spec.js                    | 4 +++-
 3 files changed, 10 insertions(+), 4 deletions(-)

diff --git a/src/action-sheet/index.tsx b/src/action-sheet/index.tsx
index 590bd90b3..d0cf40f47 100644
--- a/src/action-sheet/index.tsx
+++ b/src/action-sheet/index.tsx
@@ -1,3 +1,5 @@
+import Vue from 'vue';
+
 // Utils
 import { createNamespace } from '../utils';
 import { emit, inherit } from '../utils/functional';
@@ -89,11 +91,13 @@ function ActionSheet(
         callback(item);
       }
 
-      emit(ctx, 'select', item, index);
-
       if (props.closeOnClickAction) {
         emit(ctx, 'input', false);
       }
+
+      Vue.nextTick(() => {
+        emit(ctx, 'select', item, index);
+      });
     }
 
     function OptionContent() {
diff --git a/src/action-sheet/test/__snapshots__/index.spec.js.snap b/src/action-sheet/test/__snapshots__/index.spec.js.snap
index 932b1a226..2764b2d66 100644
--- a/src/action-sheet/test/__snapshots__/index.spec.js.snap
+++ b/src/action-sheet/test/__snapshots__/index.spec.js.snap
@@ -1,7 +1,7 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
 exports[`callback events 1`] = `
-<div class="van-popup van-popup--round van-popup--bottom van-popup--safe-area-inset-bottom van-action-sheet" name="van-popup-slide-bottom">
+<div class="van-popup van-popup--round van-popup--bottom van-popup--safe-area-inset-bottom van-action-sheet" name="van-popup-slide-bottom" style="z-index: 2002;">
   <div class="van-action-sheet__content"><button type="button" class="van-action-sheet__item"><span class="van-action-sheet__name">Option</span></button><button type="button" class="van-action-sheet__item van-action-sheet__item--disabled"><span class="van-action-sheet__name">Option</span></button><button type="button" class="van-action-sheet__item van-action-sheet__item--loading">
       <div class="van-loading van-loading--circular van-action-sheet__loading-icon"><span class="van-loading__spinner van-loading__spinner--circular"><svg viewBox="25 25 50 50" class="van-loading__circular"><circle cx="50" cy="50" r="20" fill="none"></circle></svg></span></div>
     </button><button type="button" class="van-action-sheet__item"><span class="van-action-sheet__name">Option</span>
diff --git a/src/action-sheet/test/index.spec.js b/src/action-sheet/test/index.spec.js
index f73946c2e..289a7dc5c 100644
--- a/src/action-sheet/test/index.spec.js
+++ b/src/action-sheet/test/index.spec.js
@@ -1,7 +1,7 @@
 import { mount, later } from '../../../test';
 import ActionSheet from '..';
 
-test('callback events', () => {
+test('callback events', async () => {
   const callback = jest.fn();
   const onInput = jest.fn();
   const onCancel = jest.fn();
@@ -34,6 +34,8 @@ test('callback events', () => {
   options.at(1).trigger('click');
   wrapper.find('.van-action-sheet__cancel').trigger('click');
 
+  await later();
+
   expect(callback).toHaveBeenCalled();
   expect(onCancel).toHaveBeenCalled();
   expect(onInput).toHaveBeenCalledWith(false);