Skip to content

feat(reports): apply reference UI - #426

Merged
PeterGuy326 merged 2 commits into
mainfrom
feat/425-reports-center-redesign
Sep 21, 2026
Merged

PeterGuy326 merged 2 commits into
mainfrom
feat/425-reports-center-redesign

Conversation

@Bindy-lbb

Copy link
Copy Markdown
Collaborator

Summary

  • apply the supplied reporting-center reference UI to the desktop client
  • use a focused 1080px reading column, flat five-stream navigation, and fact-first report cards
  • restyle budget, execution, escalation, audit, timeline, drawer, and narrow-window states without changing the reports contract

Validation

  • npm run build
  • npm run typecheck:renderer
  • npm run test:renderer -- apps/desktop/renderer/test/reports-center.test.tsx apps/desktop/renderer/test/budget-dashboard.test.tsx apps/desktop/renderer/test/audit-timeline.test.tsx
  • npm run build:renderer

Closes #425

@PeterGuy326 PeterGuy326 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review — APPROVE on b165b34

这是纯 CSS 的上报中心视觉改版,未触碰 reports.v1 数据契约或交互实现。完整 diff 覆盖 1080px 单列阅读区、五流扁平导航、预算/执行/升级/审计/时间线、Drawer,以及 720px 窄窗布局;状态色仍由现有事实状态驱动。

独立 worktree 验证:

  • npm run build 通过
  • 报表聚焦测试 32/32 通过
  • npm run typecheck:renderer 通过
  • npm run build:renderer 通过
  • GitHub Node 24 双平台、smoke、installers、layout、CodeQL 全绿

未发现阻塞问题。可以进入合并流程;本结论是代码/视觉实现评审,不等同于最终人工产品验收。

@PeterGuy326 PeterGuy326 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Post-base-update re-review — APPROVE on 858be0e

#418 合入后已更新到最新 main。重新核对 origin/main...HEAD:仍仅修改 workspace-polish.css,272 行 CSS,无意外删除,git diff --check 通过。新 head 的 Node 24 双平台、smoke、installers、layout parity、CodeQL、Dependency review、Scorecard 已全部通过;Windows 两个任务首次为 GitHub/Electron 下载 504,重跑后通过。可以合入。

@PeterGuy326
PeterGuy326 merged commit 0dd1303 into main Sep 21, 2026
16 of 18 checks passed
@PeterGuy326
PeterGuy326 deleted the feat/425-reports-center-redesign branch September 21, 2026 15:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat(reports): apply redesigned reporting center UI

2 participants