From 3ea5510a21dfc63a7d26aff75b4f332d17e2dd9a Mon Sep 17 00:00:00 2001
From: YunaiV <zhijiantianya@gmail.com>
Date: Sun, 21 Dec 2025 20:50:27 +0800
Subject: [PATCH] =?UTF-8?q?review=EF=BC=9A=E3=80=90antd/ele=E3=80=91?=
 =?UTF-8?q?=E3=80=90mall=E3=80=91=E8=90=A5=E9=94=80=E6=A8=A1=E5=9D=97?=
 =?UTF-8?q?=E7=9A=84=E8=BF=81=E7=A7=BB?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .../bpm/components/bpmn-process-designer/package/index.ts   | 2 --
 .../views/mall/product/spu/components/spu-and-sku-list.vue  | 2 +-
 .../src/views/mall/promotion/rewardActivity/data.ts         | 1 +
 .../views/mall/promotion/rewardActivity/modules/form.vue    | 3 ---
 .../src/views/mall/product/spu/components/sku-list.vue      | 2 ++
 .../views/mall/product/spu/components/spu-and-sku-list.vue  | 1 +
 .../src/views/mall/product/spu/components/spu-select.vue    | 6 +++---
 .../views/mall/product/spu/components/spu-table-select.vue  | 6 +++---
 apps/web-ele/src/views/mall/product/spu/form/index.vue      | 1 +
 .../mall/promotion/combination/activity/modules/form.vue    | 6 +++---
 .../src/views/mall/promotion/coupon/components/select.vue   | 6 +++---
 .../views/mall/promotion/discountActivity/modules/form.vue  | 1 +
 .../views/mall/promotion/rewardActivity/modules/form.vue    | 1 -
 .../mall/promotion/rewardActivity/modules/reward-rule.vue   | 4 ++--
 apps/web-ele/src/views/system/loginlog/data.ts              | 4 ++--
 15 files changed, 23 insertions(+), 23 deletions(-)

diff --git a/apps/web-antd/src/views/bpm/components/bpmn-process-designer/package/index.ts b/apps/web-antd/src/views/bpm/components/bpmn-process-designer/package/index.ts
index 483d5fb4d..f27319abb 100644
--- a/apps/web-antd/src/views/bpm/components/bpmn-process-designer/package/index.ts
+++ b/apps/web-antd/src/views/bpm/components/bpmn-process-designer/package/index.ts
@@ -3,9 +3,7 @@ import 'bpmn-js/dist/assets/diagram-js.css';
 import 'bpmn-js/dist/assets/bpmn-font/css/bpmn.css';
 import 'bpmn-js/dist/assets/bpmn-font/css/bpmn-codes.css';
 import 'bpmn-js/dist/assets/bpmn-font/css/bpmn-embedded.css';
-// TODO @puhui999：样式问题：设计器那，位置不太对；
 
 export { default as MyProcessDesigner } from './designer';
-// TODO @puhui999：流程发起时，预览相关的，需要使用；
 export { default as MyProcessViewer } from './designer/index2';
 export { default as MyProcessPenal } from './penal';
diff --git a/apps/web-antd/src/views/mall/product/spu/components/spu-and-sku-list.vue b/apps/web-antd/src/views/mall/product/spu/components/spu-and-sku-list.vue
index f48a51f6a..319e538e2 100644
--- a/apps/web-antd/src/views/mall/product/spu/components/spu-and-sku-list.vue
+++ b/apps/web-antd/src/views/mall/product/spu/components/spu-and-sku-list.vue
@@ -133,7 +133,7 @@ watch(
         <Image
           v-if="row.picUrl"
           :src="row.picUrl"
-          class="h-[30px] w-[30px] cursor-pointer"
+          class="!h-[30px] !w-[30px] cursor-pointer"
           :preview="true"
         />
       </template>
diff --git a/apps/web-antd/src/views/mall/promotion/rewardActivity/data.ts b/apps/web-antd/src/views/mall/promotion/rewardActivity/data.ts
index 57ebd0acb..613a4a7e7 100644
--- a/apps/web-antd/src/views/mall/promotion/rewardActivity/data.ts
+++ b/apps/web-antd/src/views/mall/promotion/rewardActivity/data.ts
@@ -217,6 +217,7 @@ export function useFormSchema(): VbenFormSchema[] {
       },
       rules: 'required',
     },
+    // TODO @puhui999：1）新增时：一直报：“请输入优惠设置”；2）修改老数据，出现报“请求参数类型错误:50.00”；
     {
       fieldName: 'rules',
       label: '优惠设置',
diff --git a/apps/web-antd/src/views/mall/promotion/rewardActivity/modules/form.vue b/apps/web-antd/src/views/mall/promotion/rewardActivity/modules/form.vue
index fda9fc9f9..99e323193 100644
--- a/apps/web-antd/src/views/mall/promotion/rewardActivity/modules/form.vue
+++ b/apps/web-antd/src/views/mall/promotion/rewardActivity/modules/form.vue
@@ -61,7 +61,6 @@ const [Modal, modalApi] = useVbenModal({
     // 提交表单
     try {
       const values = await formApi.getValues();
-      // 合并表单值和 formData（含 id、productSpuIds、productCategoryIds 等）
       const data = { ...formData.value, ...values };
       if (data.startAndEndTime && Array.isArray(data.startAndEndTime)) {
         data.startTime = data.startAndEndTime[0];
@@ -74,7 +73,6 @@ const [Modal, modalApi] = useVbenModal({
           item.limit = convertToInteger(item.limit || 0);
         }
       });
-      // productScopeValues 已通过 data.ts 中的 dependencies.trigger 自动同步到表单值中
       await (data.id
         ? updateRewardActivity(data as MallRewardActivityApi.RewardActivity)
         : createRewardActivity(data as MallRewardActivityApi.RewardActivity));
@@ -99,7 +97,6 @@ const [Modal, modalApi] = useVbenModal({
     modalApi.lock();
     try {
       const result = await getReward(data.id);
-      // antd RangePicker 需要 dayjs 对象
       result.startAndEndTime = [
         result.startTime ? dayjs(result.startTime) : undefined,
         result.endTime ? dayjs(result.endTime) : undefined,
diff --git a/apps/web-ele/src/views/mall/product/spu/components/sku-list.vue b/apps/web-ele/src/views/mall/product/spu/components/sku-list.vue
index 0ee803ae4..d92eeb16a 100644
--- a/apps/web-ele/src/views/mall/product/spu/components/sku-list.vue
+++ b/apps/web-ele/src/views/mall/product/spu/components/sku-list.vue
@@ -490,6 +490,7 @@ defineExpose({
       @checkbox-all="handleSelectionChange"
     >
       <VxeColumn v-if="isComponent" type="checkbox" width="45" fixed="left" />
+      <!-- TODO @puhui999：这里的宽度貌似有点问题，图片会寄出来； -->
       <VxeColumn align="center" title="图片" max-width="140" fixed="left">
         <template #default="{ row }">
           <ElImage
@@ -582,6 +583,7 @@ defineExpose({
       }"
     >
       <VxeColumn v-if="isComponent" type="checkbox" width="45" fixed="left" />
+      <!-- TODO @puhui999：这里的宽度貌似有点问题，图片会寄出来； -->
       <VxeColumn align="center" title="图片" max-width="140" fixed="left">
         <template #default="{ row }">
           <ElImage
diff --git a/apps/web-ele/src/views/mall/product/spu/components/spu-and-sku-list.vue b/apps/web-ele/src/views/mall/product/spu/components/spu-and-sku-list.vue
index 4f0369a3e..a46770be7 100644
--- a/apps/web-ele/src/views/mall/product/spu/components/spu-and-sku-list.vue
+++ b/apps/web-ele/src/views/mall/product/spu/components/spu-and-sku-list.vue
@@ -130,6 +130,7 @@ watch(
     <VxeColumn field="id" align="center" title="商品编号" min-width="30" />
     <VxeColumn title="商品图" min-width="80">
       <template #default="{ row }">
+        <!-- TODO @puhui999：它的 preview 貌似展示有点奇怪，不像 antd 是全屏的。。。 -->
         <ElImage
           v-if="row.picUrl"
           :src="row.picUrl"
diff --git a/apps/web-ele/src/views/mall/product/spu/components/spu-select.vue b/apps/web-ele/src/views/mall/product/spu/components/spu-select.vue
index 5f51bc24b..970f56506 100644
--- a/apps/web-ele/src/views/mall/product/spu/components/spu-select.vue
+++ b/apps/web-ele/src/views/mall/product/spu/components/spu-select.vue
@@ -9,7 +9,7 @@ import { computed, nextTick, onMounted, ref } from 'vue';
 
 import { handleTree } from '@vben/utils';
 
-import { ElDialog, ElMessage } from 'element-plus';
+import { ElButton, ElDialog, ElMessage } from 'element-plus';
 
 import { useVbenVxeGrid } from '#/adapter/vxe-table';
 import { getCategoryList } from '#/api/mall/product/category';
@@ -314,8 +314,8 @@ onMounted(async () => {
     </Grid>
     <template #footer>
       <span class="dialog-footer">
-        <el-button @click="closeModal">取消</el-button>
-        <el-button type="primary" @click="handleConfirm">确定</el-button>
+        <ElButton @click="closeModal">取消</ElButton>
+        <ElButton type="primary" @click="handleConfirm">确定</ElButton>
       </span>
     </template>
   </ElDialog>
diff --git a/apps/web-ele/src/views/mall/product/spu/components/spu-table-select.vue b/apps/web-ele/src/views/mall/product/spu/components/spu-table-select.vue
index f60cc6d34..e50a19523 100644
--- a/apps/web-ele/src/views/mall/product/spu/components/spu-table-select.vue
+++ b/apps/web-ele/src/views/mall/product/spu/components/spu-table-select.vue
@@ -9,7 +9,7 @@ import { computed, nextTick, onMounted, ref } from 'vue';
 
 import { handleTree } from '@vben/utils';
 
-import { ElDialog } from 'element-plus';
+import { ElButton, ElDialog } from 'element-plus';
 
 import { useVbenVxeGrid } from '#/adapter/vxe-table';
 import { getCategoryList } from '#/api/mall/product/category';
@@ -236,8 +236,8 @@ onMounted(async () => {
     <Grid />
     <template #footer>
       <span v-if="props.multiple" class="dialog-footer">
-        <el-button @click="closeModal">取消</el-button>
-        <el-button type="primary" @click="handleConfirm">确定</el-button>
+        <ElButton @click="closeModal">取消</ElButton>
+        <ElButton type="primary" @click="handleConfirm">确定</ElButton>
       </span>
     </template>
   </ElDialog>
diff --git a/apps/web-ele/src/views/mall/product/spu/form/index.vue b/apps/web-ele/src/views/mall/product/spu/form/index.vue
index 5f584c9a8..3b5bb6ea7 100644
--- a/apps/web-ele/src/views/mall/product/spu/form/index.vue
+++ b/apps/web-ele/src/views/mall/product/spu/form/index.vue
@@ -320,6 +320,7 @@ onMounted(async () => {
       <ElCard class="h-full w-full" v-loading="formLoading">
         <template #header>
           <div class="flex items-center justify-between">
+            <!-- TODO @puhui999：这里有告警，需要修复下。 -->
             <ElTabs v-model="activeTabName" @tab-change="handleTabChange">
               <ElTabPane label="基础设置" name="info" />
               <ElTabPane label="价格库存" name="sku" />
diff --git a/apps/web-ele/src/views/mall/promotion/combination/activity/modules/form.vue b/apps/web-ele/src/views/mall/promotion/combination/activity/modules/form.vue
index 6f97d797d..3b8750de1 100644
--- a/apps/web-ele/src/views/mall/promotion/combination/activity/modules/form.vue
+++ b/apps/web-ele/src/views/mall/promotion/combination/activity/modules/form.vue
@@ -141,7 +141,7 @@ async function getSpuDetails(
 
 // ================= end =================
 
-const [Dialog, modalApi] = useVbenModal({
+const [Modal, modalApi] = useVbenModal({
   async onConfirm() {
     const { valid } = await formApi.validate();
     if (!valid) {
@@ -214,7 +214,7 @@ const [Dialog, modalApi] = useVbenModal({
 
 <template>
   <div>
-    <Dialog :title="getTitle" class="w-[70%]">
+    <Modal :title="getTitle" class="w-[70%]">
       <Form class="mx-4">
         <!-- 商品选择 -->
         <template #spuId>
@@ -253,7 +253,7 @@ const [Dialog, modalApi] = useVbenModal({
           </div>
         </template>
       </Form>
-    </Dialog>
+    </Modal>
 
     <!-- 商品选择器弹窗 -->
     <SpuSkuSelect
diff --git a/apps/web-ele/src/views/mall/promotion/coupon/components/select.vue b/apps/web-ele/src/views/mall/promotion/coupon/components/select.vue
index 380a8c4a0..386ba7b7c 100644
--- a/apps/web-ele/src/views/mall/promotion/coupon/components/select.vue
+++ b/apps/web-ele/src/views/mall/promotion/coupon/components/select.vue
@@ -4,7 +4,7 @@ import type { MallCouponTemplateApi } from '#/api/mall/promotion/coupon/couponTe
 
 import { ref } from 'vue';
 
-import { ElDialog } from 'element-plus';
+import { ElButton, ElDialog } from 'element-plus';
 
 import { useVbenVxeGrid } from '#/adapter/vxe-table';
 import { getCouponTemplatePage } from '#/api/mall/promotion/coupon/couponTemplate';
@@ -90,8 +90,8 @@ defineExpose({
   >
     <Grid />
     <template #footer>
-      <el-button @click="closeModal">取消</el-button>
-      <el-button type="primary" @click="handleConfirm">确定</el-button>
+      <ElButton @click="closeModal">取消</ElButton>
+      <ElButton type="primary" @click="handleConfirm">确定</ElButton>
     </template>
   </ElDialog>
 </template>
diff --git a/apps/web-ele/src/views/mall/promotion/discountActivity/modules/form.vue b/apps/web-ele/src/views/mall/promotion/discountActivity/modules/form.vue
index 5918b0ea2..07d640864 100644
--- a/apps/web-ele/src/views/mall/promotion/discountActivity/modules/form.vue
+++ b/apps/web-ele/src/views/mall/promotion/discountActivity/modules/form.vue
@@ -44,6 +44,7 @@ const [Form, formApi] = useVbenForm({
   showDefaultActions: false,
 });
 
+// TODO @puhui999：antd 和 ele 里，修改时，商品都没展示。
 const [Modal, modalApi] = useVbenModal({
   async onConfirm() {
     const { valid } = await formApi.validate();
diff --git a/apps/web-ele/src/views/mall/promotion/rewardActivity/modules/form.vue b/apps/web-ele/src/views/mall/promotion/rewardActivity/modules/form.vue
index 4db34f085..b2f3bec94 100644
--- a/apps/web-ele/src/views/mall/promotion/rewardActivity/modules/form.vue
+++ b/apps/web-ele/src/views/mall/promotion/rewardActivity/modules/form.vue
@@ -72,7 +72,6 @@ const [Modal, modalApi] = useVbenModal({
           item.limit = convertToInteger(item.limit || 0);
         }
       });
-      // productScopeValues 已通过 data.ts 中的 dependencies.trigger 自动同步到表单值中
       await (data.id
         ? updateRewardActivity(data as MallRewardActivityApi.RewardActivity)
         : createRewardActivity(data as MallRewardActivityApi.RewardActivity));
diff --git a/apps/web-ele/src/views/mall/promotion/rewardActivity/modules/reward-rule.vue b/apps/web-ele/src/views/mall/promotion/rewardActivity/modules/reward-rule.vue
index b039d0719..afe05ed21 100644
--- a/apps/web-ele/src/views/mall/promotion/rewardActivity/modules/reward-rule.vue
+++ b/apps/web-ele/src/views/mall/promotion/rewardActivity/modules/reward-rule.vue
@@ -107,7 +107,7 @@ function handleDelete(ruleIndex: number) {
                 <span>{{ isPriceCondition ? '元' : '件' }}</span>
               </div>
             </ElFormItem>
-
+            <!-- 优惠内容 -->
             <ElFormItem label="优惠内容:" class="!mb-0">
               <div class="flex flex-col gap-3">
                 <div
@@ -134,7 +134,7 @@ function handleDelete(ruleIndex: number) {
                   <span class="w-20 shrink-0 text-sm text-gray-500">包邮</span>
                   <ElSwitch v-model="rule.freeDelivery" />
                 </div>
-
+                <!-- 送积分 -->
                 <div
                   class="flex items-center gap-2 rounded-md bg-gray-50 px-3 py-2"
                 >
diff --git a/apps/web-ele/src/views/system/loginlog/data.ts b/apps/web-ele/src/views/system/loginlog/data.ts
index 2be5cdf1c..daa67c17b 100644
--- a/apps/web-ele/src/views/system/loginlog/data.ts
+++ b/apps/web-ele/src/views/system/loginlog/data.ts
@@ -53,7 +53,7 @@ export function useGridColumns(): VxeTableGridOptions['columns'] {
     },
     {
       field: 'logType',
-      title: '操作类型',
+      title: '登录类型',
       minWidth: 120,
       cellRender: {
         name: 'CellDict',
@@ -108,7 +108,7 @@ export function useDetailSchema(): DescriptionItemSchema[] {
     },
     {
       field: 'logType',
-      label: '操作类型',
+      label: '登录类型',
       render: (val) => {
         return h(DictTag, {
           type: DICT_TYPE.SYSTEM_LOGIN_TYPE,