Переглянути джерело

fix(verify): address review feedback on tap feedback and click handlers

- SalesHome.vue: add :active transform to .tap-feedback
- ProfileView.vue: remove redundant clickable prop from uni-list-item wrappers
- ScheduleView.vue: remove redundant clickable prop from schedule list items
- SalesTaskCard.vue: remove unused borderStyle, keep emergency border via Tailwind classes

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
xuyunhui 1 тиждень тому
батько
коміт
1bf82b1d48

+ 0 - 1
src/components/dispatch/ScheduleView.vue

@@ -106,7 +106,6 @@
             :note="(schedule.staffIds || '未分配人员') + ' | ' + (schedule.vehicleIds || '未分配车辆')"
             :right-text="getStatusText(schedule.status)"
             show-arrow
-            clickable
           >
             <template #header>
               <view class="w-10 h-10 rounded-xl flex items-center justify-center mr-3" :class="getScheduleIconBgClass(schedule.shiftType)">

+ 3 - 0
src/components/home/SalesHome.vue

@@ -251,6 +251,9 @@ onMounted(() => {
 .tap-feedback {
   transition: transform 0.15s ease;
 }
+.tap-feedback:active {
+  transform: scale(0.95);
+}
 .task-reminder-btn {
   transition: background-color 0.15s ease;
 }

+ 0 - 3
src/components/my/ProfileView.vue

@@ -50,7 +50,6 @@
               title="修改密码"
               icon="uniui-locked-filled"
               show-arrow
-              clickable
             />
           </view>
           <view hover-class="scale-down" :hover-start-time="0" :hover-stay-time="150" @click="goToNotices">
@@ -58,7 +57,6 @@
               title="通知公告"
               icon="uniui-notification-filled"
               show-arrow
-              clickable
             />
           </view>
           <view hover-class="scale-down" :hover-start-time="0" :hover-stay-time="150" @click="goToHelp">
@@ -66,7 +64,6 @@
               title="帮助中心"
               icon="uniui-help-filled"
               show-arrow
-              clickable
             />
           </view>
         </uni-list>

+ 0 - 18
src/components/task/SalesTaskCard.vue

@@ -8,7 +8,6 @@
       'border-l-4 border-red-500': task.type === 'emergency',
       'bg-green-50': task.status === 'completed'
     }"
-    :style="borderStyle"
     @click="$emit('click', task.id)"
   >
     <view class="flex justify-between items-start mb-2">
@@ -85,23 +84,6 @@ const progressPercent = computed(() => {
   }
   return map[props.task.status] ?? 0
 })
-
-const borderStyle = computed(() => {
-  if (props.task.type === 'emergency') return {}
-  const color = {
-    pending: '#f59e0b',
-    confirmed: '#2563eb',
-    scheduled: '#2563eb',
-    departed: '#8b5cf6',
-    arrived: '#10b981',
-    constructing: '#2563eb',
-    inspecting: '#f59e0b',
-    completed: '#10b981',
-    cancelled: '#ef4444',
-    rejected: '#ef4444',
-  }[props.task.status] || '#2563eb'
-  return { borderLeftColor: color }
-})
 </script>
 
 <style scoped>