From 021045e285030fb8fd9c941b5294de4e05c1ac4a Mon Sep 17 00:00:00 2001
From: duanshuwen <duanshuwen@gogpay.cn>
Date: Thu, 27 Aug 2026 09:47:39 +0800
Subject: [PATCH] fix: improve lead detail dialog hierarchy

---
 .../src/pages/LeadsPage.test.ts               | 11 +++++++
 WonderQ-Admin-UI-Vue/src/pages/LeadsPage.vue  | 29 ++++++++++++++++---
 WonderQ-Admin-UI-Vue/src/styles.css           | 24 +++++++++------
 3 files changed, 51 insertions(+), 13 deletions(-)

diff --git a/WonderQ-Admin-UI-Vue/src/pages/LeadsPage.test.ts b/WonderQ-Admin-UI-Vue/src/pages/LeadsPage.test.ts
index 39bb79a..1e646df 100644
--- a/WonderQ-Admin-UI-Vue/src/pages/LeadsPage.test.ts
+++ b/WonderQ-Admin-UI-Vue/src/pages/LeadsPage.test.ts
@@ -8,4 +8,15 @@ describe("LeadsPage initial loading", () => {
 
     expect(source).toMatch(/onMounted\(load\)/);
   });
+
+  it("keeps the grouped lead detail layout hooks", async () => {
+    const source = await readFile(fileURLToPath(new URL("./LeadsPage.vue", import.meta.url)), "utf8");
+
+    expect(source).toContain('class="lead-detail-card"');
+    expect(source).toContain('class="lead-detail-item"');
+    expect(source).toContain('class="lead-detail-section-title"');
+    expect(source).toContain('class="lead-detail-section lead-detail-note"');
+    expect(source.match(/class="lead-detail-item"/g)).toHaveLength(6);
+    expect(source).toContain("v-permission=\"'admin:leads:update'\"");
+  });
 });
diff --git a/WonderQ-Admin-UI-Vue/src/pages/LeadsPage.vue b/WonderQ-Admin-UI-Vue/src/pages/LeadsPage.vue
index 80e0b65..6696840 100644
--- a/WonderQ-Admin-UI-Vue/src/pages/LeadsPage.vue
+++ b/WonderQ-Admin-UI-Vue/src/pages/LeadsPage.vue
@@ -117,10 +117,31 @@ onMounted(load);
 
     <el-dialog v-model="detailOpen" title="线索详情" width="min(720px, 94vw)" destroy-on-close>
       <template v-if="selectedLead">
-        <div class="lead-detail-toolbar"><el-tag :type="statusTagType(selectedLead.status)" effect="plain">{{ leadStatusLabel(selectedLead.status) }}</el-tag><el-select v-permission="'admin:leads:update'" :model-value="selectedLead.status" :loading="updatingId === selectedLead.id" size="small" @update:model-value="changeStatus(selectedLead, $event)"><el-option v-for="option in leadStatusOptions" :key="option.value" :label="option.label" :value="option.value" /></el-select></div>
-        <dl class="lead-detail-grid"><div><dt>联系人</dt><dd>{{ selectedLead.contactName || "未填写" }}</dd></div><div><dt>联系方式</dt><dd>{{ selectedLead.phone || "未填写" }}</dd></div><div><dt>需求类型</dt><dd>{{ selectedLead.leadType === "vehicle" ? "用车需求" : "一般需求" }}</dd></div><div><dt>提交时间</dt><dd>{{ formatDate(selectedLead.createdAt) }}</dd></div><div><dt>目的地</dt><dd>{{ selectedLead.destination || "未填写" }}</dd></div><div><dt>出行日期</dt><dd>{{ formatDate(selectedLead.vehicleDemand?.travelDate || selectedLead.travelDate) }}</dd></div></dl>
-        <div v-if="selectedLead.vehicleDemand" class="lead-detail-section"><strong>用车行程</strong><p>{{ selectedLead.vehicleDemand.pickupLocation }} → {{ selectedLead.vehicleDemand.dropoffLocation }}</p><p>{{ selectedLead.vehicleDemand.peopleCount }} 人 · 行李 {{ selectedLead.vehicleDemand.luggageCount ?? "未填写" }} 件 · {{ selectedLead.vehicleDemand.vehicleOptionTitle || "未指定车型" }}</p></div>
-        <div class="lead-detail-section"><strong>备注</strong><p>{{ selectedLead.vehicleDemand?.specialRequirements || selectedLead.note || "暂无备注" }}</p></div>
+        <div class="lead-detail-toolbar">
+          <el-tag :type="statusTagType(selectedLead.status)" effect="plain">{{ leadStatusLabel(selectedLead.status) }}</el-tag>
+          <el-select v-permission="'admin:leads:update'" :model-value="selectedLead.status" :loading="updatingId === selectedLead.id" size="small" @update:model-value="changeStatus(selectedLead, $event)">
+            <el-option v-for="option in leadStatusOptions" :key="option.value" :label="option.label" :value="option.value" />
+          </el-select>
+        </div>
+        <div class="lead-detail-card">
+          <dl class="lead-detail-grid">
+            <div class="lead-detail-item"><dt>联系人</dt><dd>{{ selectedLead.contactName || "未填写" }}</dd></div>
+            <div class="lead-detail-item"><dt>联系方式</dt><dd>{{ selectedLead.phone || "未填写" }}</dd></div>
+            <div class="lead-detail-item"><dt>需求类型</dt><dd>{{ selectedLead.leadType === "vehicle" ? "用车需求" : "一般需求" }}</dd></div>
+            <div class="lead-detail-item"><dt>提交时间</dt><dd>{{ formatDate(selectedLead.createdAt) }}</dd></div>
+            <div class="lead-detail-item"><dt>目的地</dt><dd>{{ selectedLead.destination || "未填写" }}</dd></div>
+            <div class="lead-detail-item"><dt>出行日期</dt><dd>{{ formatDate(selectedLead.vehicleDemand?.travelDate || selectedLead.travelDate) }}</dd></div>
+          </dl>
+        </div>
+        <section v-if="selectedLead.vehicleDemand" class="lead-detail-section lead-detail-trip">
+          <div class="lead-detail-section-title"><strong>用车行程</strong></div>
+          <p>{{ selectedLead.vehicleDemand.pickupLocation }} → {{ selectedLead.vehicleDemand.dropoffLocation }}</p>
+          <p>{{ selectedLead.vehicleDemand.peopleCount }} 人 · 行李 {{ selectedLead.vehicleDemand.luggageCount ?? "未填写" }} 件 · {{ selectedLead.vehicleDemand.vehicleOptionTitle || "未指定车型" }}</p>
+        </section>
+        <section class="lead-detail-section lead-detail-note">
+          <div class="lead-detail-section-title"><strong>备注</strong></div>
+          <p>{{ selectedLead.vehicleDemand?.specialRequirements || selectedLead.note || "暂无备注" }}</p>
+        </section>
       </template>
     </el-dialog>
   </div>
diff --git a/WonderQ-Admin-UI-Vue/src/styles.css b/WonderQ-Admin-UI-Vue/src/styles.css
index 38ee583..f07dc44 100644
--- a/WonderQ-Admin-UI-Vue/src/styles.css
+++ b/WonderQ-Admin-UI-Vue/src/styles.css
@@ -61,15 +61,21 @@ button, input { font: inherit; }
 .lead-filter-card { margin-bottom: 16px; }
 .lead-filters { display: grid; grid-template-columns: 170px 170px minmax(180px, 1fr) auto; gap: 10px; }
 .lead-pagination { display: flex; justify-content: flex-end; align-items: center; gap: 14px; padding-top: 16px; color: var(--muted); font-size: 12px; }
-.lead-detail-toolbar { display: flex; justify-content: flex-end; align-items: center; gap: 10px; margin-bottom: 18px; }
+.lead-detail-toolbar { display: flex; align-items: center; gap: 10px; margin-bottom: 16px; padding: 10px 12px; border: 1px solid var(--line); border-radius: 9px; background: #f8fafc; }
+.lead-detail-toolbar .el-tag { flex: 0 0 auto; }
+.lead-detail-toolbar .el-select { min-width: 0; flex: 1 1 auto; max-width: 310px; }
 .lead-summary { display: grid; gap: 4px; min-width: 0; }
 .lead-summary span { overflow: hidden; color: var(--muted); font-size: 12px; text-overflow: ellipsis; white-space: nowrap; }
-.lead-detail-card { margin-top: 16px; }
-.lead-detail-grid { display: grid; grid-template-columns: repeat(4, minmax(0, 1fr)); gap: 16px; margin: 0; }
-.lead-detail-grid dt { color: var(--muted); font-size: 12px; }
-.lead-detail-grid dd { margin: 5px 0 0; color: var(--ink); font-weight: 650; }
-.lead-detail-section { margin-top: 20px; padding: 14px; border-radius: 9px; background: #f8fafc; }
-.lead-detail-section p { margin: 7px 0 0; color: #475569; line-height: 1.6; }
+.lead-detail-card { overflow: hidden; margin-top: 0; border: 1px solid var(--line); border-radius: 10px; background: var(--surface); }
+.lead-detail-grid { display: grid; grid-template-columns: repeat(2, minmax(0, 1fr)); gap: 1px; margin: 0; background: var(--line); }
+.lead-detail-item { min-width: 0; padding: 15px 16px; background: var(--surface); }
+.lead-detail-grid dt { color: var(--muted); font-size: 12px; line-height: 1.4; }
+.lead-detail-grid dd { min-width: 0; margin: 5px 0 0; overflow-wrap: anywhere; color: var(--ink); font-weight: 650; line-height: 1.45; }
+.lead-detail-section { margin-top: 16px; padding: 14px 16px; border: 1px solid transparent; border-radius: 9px; background: #f8fafc; }
+.lead-detail-section-title { display: flex; align-items: center; gap: 8px; color: #475569; font-size: 13px; line-height: 1.4; }
+.lead-detail-section-title::before { width: 3px; height: 14px; border-radius: 3px; background: var(--blue); content: ""; }
+.lead-detail-section p { margin: 7px 0 0; overflow-wrap: anywhere; color: #475569; line-height: 1.6; }
+.lead-detail-note { border-color: #e4eaf2; background: #f5f8fc; }
 .media-toolbar { justify-content: flex-end; }
 .media-toolbar .el-input { width: 150px; }
 .media-grid { display: grid; grid-template-columns: repeat(4, minmax(0, 1fr)); gap: 14px; }
@@ -190,9 +196,9 @@ button, input { font: inherit; }
   .system-editor-form .el-col-12 { width: 100%; max-width: 100%; flex: 0 0 100%; }
 }
 @media (max-width: 880px) { .wanfa-workspace { grid-template-columns: 1fr; } }
-@media (max-width: 980px) { .media-grid { grid-template-columns: repeat(3, minmax(0, 1fr)); } .lead-detail-grid { grid-template-columns: repeat(2, minmax(0, 1fr)); } }
+@media (max-width: 980px) { .media-grid { grid-template-columns: repeat(3, minmax(0, 1fr)); } }
 @media (max-width: 760px) { .concierge-row { align-items: flex-start; flex-direction: column; } .concierge-actions { justify-content: flex-start; } .lead-filters { grid-template-columns: 1fr 1fr; } .lead-filters .el-input { grid-column: 1 / -1; } .operations-grid { grid-template-columns: 1fr; } .media-grid { grid-template-columns: repeat(2, minmax(0, 1fr)); } }
-@media (max-width: 640px) { .site-config-intro { align-items: flex-start; } .site-config-intro .el-button, .page-intro-actions { width: 100%; } .page-intro-actions { justify-content: stretch; } .page-intro-actions .el-button { flex: 1; } .editor-options { grid-template-columns: 1fr; gap: 0; } .site-config-panel .panel-heading { align-items: flex-start; flex-direction: column; } .site-config-panel .panel-heading .el-button { width: 100%; } .concierge-detail-row { grid-template-columns: 1fr; } .lead-filters { grid-template-columns: 1fr; } .lead-filters .el-input { grid-column: auto; } .lead-detail-grid { grid-template-columns: 1fr; } .media-toolbar { align-items: stretch; flex-direction: column; width: 100%; } .media-toolbar .el-input { width: 100%; } .media-grid { grid-template-columns: 1fr; } }
+@media (max-width: 640px) { .site-config-intro { align-items: flex-start; } .site-config-intro .el-button, .page-intro-actions { width: 100%; } .page-intro-actions { justify-content: stretch; } .page-intro-actions .el-button { flex: 1; } .editor-options { grid-template-columns: 1fr; gap: 0; } .site-config-panel .panel-heading { align-items: flex-start; flex-direction: column; } .site-config-panel .panel-heading .el-button { width: 100%; } .concierge-detail-row { grid-template-columns: 1fr; } .lead-filters { grid-template-columns: 1fr; } .lead-filters .el-input { grid-column: auto; } .lead-detail-grid { grid-template-columns: 1fr; } .lead-detail-toolbar { align-items: stretch; flex-direction: column; } .lead-detail-toolbar .el-select { width: 100%; max-width: none; } .media-toolbar { align-items: stretch; flex-direction: column; width: 100%; } .media-toolbar .el-input { width: 100%; } .media-grid { grid-template-columns: 1fr; } }
 
 /* RuoYi-compatible shell: fixed sidebar, compact navbar, tags view and responsive content. */
 .admin-layout {