diff --git a/src/views/system/Menu/Detail/BasicInfo.vue b/src/views/system/Menu/Detail/BasicInfo.vue
index 52111879..2ae0fd7a 100644
--- a/src/views/system/Menu/Detail/BasicInfo.vue
+++ b/src/views/system/Menu/Detail/BasicInfo.vue
@@ -406,12 +406,7 @@ type assetType = {
     value: string;
 };
 
-watch(
-    () => form.data.icon,
-    () => {
-        basicFormRef.value?.validate();
-    },
-);
+
 </script>
 
 <style lang="less" scoped>
diff --git a/src/views/system/Menu/components/ButtonAddDialog.vue b/src/views/system/Menu/components/ButtonAddDialog.vue
index ffc69edb..21861c30 100644
--- a/src/views/system/Menu/components/ButtonAddDialog.vue
+++ b/src/views/system/Menu/components/ButtonAddDialog.vue
@@ -95,13 +95,15 @@ const confirm = () => {
     loading.value = true;
     formRef.value &&
         formRef.value.validate().then(() => {
-            const buttons = toRaw(props.menuInfo?.buttons);
+            const buttons = toRaw(props.menuInfo?.buttons) || [];
             const button = buttons?.find((item) => item.id === form.data.id);
             if (button) {
                 Object.entries(form.data).forEach(([key, value]) => {
                     button[key] = value;
                 });
-            } else buttons.push({ ...form.data });
+            } else {
+                buttons.push({ ...form.data });
+            }
             const params = {
                 ...props.menuInfo,
                 buttons,