Skip to content

feat(windows): 支持运行配置列表左右拖拽调整宽度 - #475

Merged
1lck merged 5 commits into
1lck:previewfrom
Rangsh:fix/462-windows-run-config-list-resize
Sep 4, 2026
Merged

feat(windows): 支持运行配置列表左右拖拽调整宽度#475
1lck merged 5 commits into
1lck:previewfrom
Rangsh:fix/462-windows-run-config-list-resize

Conversation

@Rangsh

@Rangsh Rangsh commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

Summary

  • 为 Windows 运行工具窗口的配置列表增加左右拖拽调整宽度,解决长应用名被截断的问题
  • 尺寸约束对齐 macOS(默认 230px,最小 180,最大 420),松手后持久化本地偏好
  • 补充 desktop:dev 所需的 default-run = "lithe-windows",避免多二进制导致无法启动

Fixes #462

Test plan

  • 打开底部 Run / 运行工具窗口
  • 拖拽配置列表与输出区之间的分隔条,确认可左右拉伸
  • 确认长应用名在拉宽后可完整显示
  • 确认拖拽过程中输出区布局正常、无异常跳动
  • 重启应用后,确认列表宽度被正确恢复
  • 窗口较窄时,确认列表宽度被合理钳制且内容区仍可用

Fixes 1lck#462

Co-authored-by: Cursor <cursoragent@cursor.com>
@Rangsh
Rangsh requested a review from 1lck as a code owner September 4, 2026 13:40
Comment thread windows/tauri/src/features/run/components/run-configuration-list-split.tsx Outdated
@1lck

1lck commented Sep 4, 2026

Copy link
Copy Markdown
Owner

@Rangsh 感谢这次改动,整体方向是对的,Issue #462 里要求的鼠标拖拽、宽度钳制和重启恢复都已经覆盖;把拖拽宽度放进独立的本地布局组件,并用 requestAnimationFrame 避免每个鼠标事件触发完整 RunPane 重建,这个设计值得保留。\n\n这次 Review 有 1 个需要合并前处理的问题:拖拽监听器和 document.body 样式只在 mouseup 清理,组件卸载或窗口失焦时可能泄漏,导致光标/文本选择状态残留。另有 1 个不阻塞合并的问题:分隔条虽然声明为可聚焦的 role="separator",但暂不支持 ArrowLeft/ArrowRight 键盘调整。\n\n因此当前存在需要阻塞合并的问题,请先处理 P1 行级评论。建议补充组件级测试,覆盖拖拽完成、失焦/卸载清理、键盘调整和持久化恢复;同时确认 PR 上仍处于 pending 的 Windows 构建检查最终通过。

Rangsh and others added 4 commits September 4, 2026 22:01
响应 PR 1lck#475 审查:提取可复用拖拽会话,在 blur/pointercancel/卸载时清理监听与 RAF,并支持方向键调整宽度。

Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
@1lck
1lck merged commit 2bc8de4 into 1lck:preview Sep 4, 2026
14 checks passed
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.

2 participants