diff --git a/memory/2026-06-09.md b/memory/2026-06-09.md index a4db2d1..123e547 100644 --- a/memory/2026-06-09.md +++ b/memory/2026-06-09.md @@ -145,3 +145,31 @@ - 风险与关注点： - 当前仓库内仍没有 `tpbig.exe`、`rjtzl.exe`、旧 ATP 模型目录和 EGM 子目录等真实生产运行资产；本次交付的是代码层接入、状态检测和可测的 worker 执行协议。 - 上线前仍需在容器/部署环境提供实际 legacy 资产挂载，并用真实资产做 smoke test。 + +## Work Log - 任务监控页面样式布局优化（2026-06-09） + +- 背景： + - Issue `FL-72` 要求“参考用户管理页面的样式和布局优化任务监控页面，包括 margin、padding、border 以及自适应高度”等。 + - 现状是 `web/src/app/admin/task-monitor/page.tsx` 采用“筛选卡 + 表格卡”的分离布局，和用户管理页的单卡结构不一致；任务表格也缺少在少量数据下的最小高度占位与视口自适应滚动高度。 + +- 本次改动： + - `web/src/app/admin/task-monitor/page.tsx` + - 页面主体收敛为与用户管理页一致的单卡布局，标题统一为“任务监控”。 + - 将 Worker/队列/任务/状态筛选与自动刷新开关合并进卡片顶部 `Form inline` 区域，补充“重置筛选”按钮。 + - 卡片顶部 `extra` 区域增加统一的刷新按钮和加载中的小型 `Spin`，保持和管理页操作区的交互节奏一致。 + - 增加生成时间、Worker 总数、在线/离线数、筛选后任务数的二级信息行，替代原本散落在筛选区尾部的时间文案。 + - 任务表格新增与用户管理页一致的动态高度计算（`scroll.y` + `resize` 监听 + `ResizeObserver`），避免筛选结果较少时表格区域坍塌。 + - 加载态统一为 `min-h-[240px]` 居中 `Spin`；空态文案改为“暂无符合筛选条件的任务数据。”。 + - 保持任务监控的数据请求、轮询、权限判断和表格列定义不变，仅调整布局与展示容器。 + - `web/src/app/globals.css` + - 新增 `.admin-task-monitor-table-anchor .ant-table-body` 作用域样式，通过页面注入变量设置最小高度，避免影响其他 Ant Design 表格。 + +- 验证： + - 基线： + - `npm --workspace web exec eslint src/app/admin/task-monitor/page.tsx --max-warnings=0` -> 通过。 + - 修改后： + - `npm --workspace web exec eslint src/app/admin/task-monitor/page.tsx --max-warnings=0` -> 通过。 + - `npm run build:web` -> 通过。 + +- 风险与关注点： + - 影响范围仅 `任务监控` 页面前端展示层与局部表格样式作用域，不涉及后端接口、权限或任务监控数据口径变更。 diff --git a/web/src/app/admin/task-monitor/page.tsx b/web/src/app/admin/task-monitor/page.tsx index 6e3e239..cc0ce33 100644 --- a/web/src/app/admin/task-monitor/page.tsx +++ b/web/src/app/admin/task-monitor/page.tsx @@ -2,14 +2,15 @@ import Link from "next/link"; import dayjs from "dayjs"; -import { useMemo, useState } from "react"; import type { ComponentType } from "react"; +import { useCallback, useEffect, useMemo, useRef, useState, type CSSProperties } from "react"; import { useQuery } from "@tanstack/react-query"; import { Alert, Button, Card, Empty, + Form, Input, Select, Space, @@ -30,6 +31,9 @@ const { Text } = Typography; const AntCard = Card as unknown as ComponentType; const DEFAULT_RECENT_LIMIT = 100; +const TASK_MONITOR_TABLE_MIN_SCROLL_Y = 180; +const TASK_MONITOR_TABLE_VIEWPORT_GAP = 40; +const TASK_MONITOR_TABLE_FALLBACK_RESERVE = 220; type FlowerWorkerItem = { worker: string; @@ -156,6 +160,8 @@ export default function AdminTaskMonitorPage() { const [queueKeyword, setQueueKeyword] = useState(""); const [taskKeyword, setTaskKeyword] = useState(""); const [statusFilter, setStatusFilter] = useState<"all" | "online" | "offline">("all"); + const [tableScrollY, setTableScrollY] = useState(TASK_MONITOR_TABLE_MIN_SCROLL_Y); + const tableScrollAnchorRef = useRef(null); const workersOverviewQuery = useQuery({ queryKey: ["flower-workers-overview"], @@ -196,6 +202,8 @@ export default function AdminTaskMonitorPage() { staleTime: 15_000, }); + const workersOverview = workersOverviewQuery.data; + const taskColumns = useMemo>( () => [ { @@ -352,9 +360,93 @@ export default function AdminTaskMonitorPage() { }); }, [allTaskRows, filteredWorkers, taskKeyword]); + const handleResetFilters = () => { + setWorkerKeyword(""); + setQueueKeyword(""); + setTaskKeyword(""); + setStatusFilter("all"); + }; + + const updateTableScrollY = useCallback(() => { + if (typeof window === "undefined") { + return; + } + const anchor = tableScrollAnchorRef.current; + if (!anchor) { + return; + } + + const anchorTop = anchor.getBoundingClientRect().top; + const tableWrapper = anchor.querySelector(".ant-table-wrapper"); + const tableBody = anchor.querySelector(".ant-table-body"); + + let nextHeight = Math.floor(window.innerHeight - anchorTop - TASK_MONITOR_TABLE_FALLBACK_RESERVE); + if (tableWrapper) { + const wrapperRect = tableWrapper.getBoundingClientRect(); + const bodyHeight = tableBody?.getBoundingClientRect().height ?? TASK_MONITOR_TABLE_MIN_SCROLL_Y; + const nonBodyHeight = Math.max(0, wrapperRect.height - bodyHeight); + const topGap = Math.max(0, wrapperRect.top - anchorTop); + nextHeight = Math.floor(window.innerHeight - anchorTop - topGap - nonBodyHeight - TASK_MONITOR_TABLE_VIEWPORT_GAP); + } + + const clampedHeight = Math.max(TASK_MONITOR_TABLE_MIN_SCROLL_Y, nextHeight); + setTableScrollY((previous) => (Math.abs(previous - clampedHeight) <= 1 ? previous : clampedHeight)); + }, []); + + useEffect(() => { + if (typeof window === "undefined") { + return; + } + window.requestAnimationFrame(updateTableScrollY); + }, [ + allTasksQuery.isFetching, + filteredTaskRows.length, + statusFilter, + updateTableScrollY, + workerKeyword, + workersOverviewQuery.error, + workersOverviewQuery.isFetching, + workersOverview?.summary.total, + ]); + + useEffect(() => { + if (typeof window === "undefined") { + return; + } + + const onViewportChange = () => { + window.requestAnimationFrame(updateTableScrollY); + }; + + window.addEventListener("resize", onViewportChange); + return () => { + window.removeEventListener("resize", onViewportChange); + }; + }, [updateTableScrollY]); + + useEffect(() => { + if (typeof window === "undefined" || typeof ResizeObserver === "undefined") { + return; + } + + const anchor = tableScrollAnchorRef.current; + if (!anchor) { + return; + } + + const resizeObserver = new ResizeObserver(() => { + window.requestAnimationFrame(updateTableScrollY); + }); + resizeObserver.observe(anchor); + + return () => { + resizeObserver.disconnect(); + }; + }, [updateTableScrollY]); + if (initializing || (workersOverviewQuery.isLoading && !workersOverviewQuery.data && canRead && Boolean(user))) { return ( -
+
); @@ -388,95 +480,138 @@ export default function AdminTaskMonitorPage() { ); } - const workersOverview = workersOverviewQuery.data; - return ( - - - - setWorkerKeyword(event.target.value)} - style={{ width: 220 }} - /> - setQueueKeyword(event.target.value)} - style={{ width: 220 }} - /> - setTaskKeyword(event.target.value)} - style={{ width: 240 }} - /> - setWorkerKeyword(event.target.value)} + /> + + + + setQueueKeyword(event.target.value)} + /> + + + + setTaskKeyword(event.target.value)} + /> + + + +