refactor(action-sheet): simplify initialization logic and enhance obs…

…erver functionality
Tencent · Jan 16, 2025 · 4464026 · 4464026
1 parent 6bb91a8
commit 4464026
Showing 1 changed file with 23 additions and 18 deletions.
diff --git a/src/action-sheet/action-sheet.ts b/src/action-sheet/action-sheet.ts
@@ -36,19 +36,24 @@ export default class ActionSheet extends SuperComponent {
     },
   ];
 
-  ready() {
-    this.memoInitialData();
-    this.splitGridThemeActions();
-  }
+  observers = {
+    'visible, items'(visible: boolean) {
+      if (!visible) return;
+      this.init();
+    },
+  };
 
   methods = {
-    onSwiperChange(e: WechatMiniprogram.TouchEvent) {
-      const {
-        detail: { current },
-      } = e;
-      this.setData({
-        currentSwiperIndex: current,
-      });
+    init() {
+      this.memoInitialData();
+      this.splitGridThemeActions();
+    },
+
+    memoInitialData() {
+      this.initialData = {
+        ...this.properties,
+        ...this.data,
+      };
     },
 
     splitGridThemeActions() {
@@ -70,13 +75,6 @@ export default class ActionSheet extends SuperComponent {
       this._trigger('visible-change', { visible: true });
     },
 
-    memoInitialData() {
-      this.initialData = {
-        ...this.properties,
-        ...this.data,
-      };
-    },
-
     /** 指令调用隐藏 */
     close() {
       this.triggerEvent('close', { trigger: 'command' });
@@ -95,6 +93,13 @@ export default class ActionSheet extends SuperComponent {
       }
     },
 
+    onSwiperChange(e: WechatMiniprogram.TouchEvent) {
+      const { current } = e.detail;
+      this.setData({
+        currentSwiperIndex: current,
+      });
+    },
+
     onSelect(event: WechatMiniprogram.TouchEvent) {
       const { currentSwiperIndex, items, gridThemeItems, count, theme } = this.data;
       const { index } = event.currentTarget.dataset;