Conversation
xiaoyumuxi
left a comment
There was a problem hiding this comment.
有两个 P2 建议先处理:
-
命令面板窗口动作目前没有真正接通窗口 API。
window-actions.tsx里的minimize-window/maximize-window/toggle-fullscreen都只是dispatchEvent,仓库里没有对应的窗口级 listener;实际点击后只会关闭命令面板,不会调用 Tauri 的窗口操作。建议直接复用window-menu-bar.tsx已有的getCurrentWindow().minimize()/maximize()/setFullscreen()路径,或补统一的 window-event bridge。否则这次新增的window-controls功能矩阵不应把 Windows 标为implemented。 -
Maximize / Fullscreen 的图标和标签没有跟随当前窗口状态。
这次把图标改成了更明确的 maximize / fullscreen 图标,但createWindowActions()不接收isMaximized/isFullscreen。窗口已经最大化时仍显示Window: Maximize和 maximize 图标,而描述又写了 “Maximize or restore”;全屏时也仍显示进入全屏图标。PR 新增的矩阵验收要求“最大化/还原状态与按钮标签一致”,按当前结构无法满足。建议注入窗口状态动态生成 Restore / Exit Fullscreen 的 label/icon,或者把 Maximize 定义成纯 maximize 而不是 toggle。
其余标题栏 glyph、右侧 padding、共享 Spinner 和 Updates 图标改动静态看没有发现明显回归,当前 CI 也全绿。
另外 PR 当前 mergeable: false,基线 preview 已前进,冲突主要落在功能矩阵 JSON / Markdown / CSV。建议先同步最新 preview,以 JSON 为准解决后重新生成文档并再跑一轮 CI。
|
我现在正在调整这个图标,忘记给pr改成草稿了,添麻烦了,后续我会补上UI具体变更图片 |
What changed
Validation
bun run typecheck(windows/tauri)bun run lint(passes with existing warnings)bun run build(windows/tauri)scripts/verify-platform-feature-matrix.shscripts/verify-platform-feature-matrix-change.sh upstream/preview HEAD