Repository navigation
fix(skills): keep evolution list and detail scrolling independent - #513
Merged
Merged
Conversation
12 of 28 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
摘要 / Summary
浏览十条进化建议时,左侧滚动会把右侧标题和操作按钮带出视口。现在宽屏的列表和详情各自滚动;切换建议后详情回到顶部,左侧保留位置,窄屏继续上下排列。
When browsing a long evolution inbox, scrolling the list moved the selected detail and actions off-screen. Bound the Pending/Done split on wide screens, scroll each pane independently, and reset only the detail when selecting another suggestion.
关联 Issue 或背景 / Related Issue or Context
Fixes #512
涉及区域 / Affected Areas
PR 类型 / PR Type
最新代码确认 / Latest Codebase Confirmation
main分支开发,或在提交前已 rebase 或合并最新main。 / I developed from the latestmain, or rebased or merged the latestmainbefore submitting.同步命令 / Sync command:
git fetch origin main; branch created fromfa4950616f8c97ce6b4b8285eb89b83cf996d94f(latest main at submission).AI 编码披露 / AI Coding Disclosure
使用的 AI 模型 / AI model used:
GPT-6
使用的编码 Agent 工具 / Coding Agent tool used:
Codex
仓库规范检查 / Repository Rules
@deepseek-ai/*NPM 契约。 / I did not modify DSH source or point tsconfig at a DSH source checkout, and used only published@deepseek-ai/*NPM contracts.src/host/protocol.ts为准,没有在两侧重复定义 wire DTO。 / The Client and Host boundary still uses browser-safesrc/host/protocol.tsas the source for wire DTOs.lib/文件。 / I did not commit tokens, credentials, private memory, unredacted logs, or generatedlib/files.兼容性与数据安全 / Compatibility and Data Safety
仅调整 Skills 进化 Pending/Done 布局和详情滚动状态,不修改 DSH/Mnemon 契约、配置、Provider、持久化格式或写入权限。窄屏保留原布局;回滚不涉及数据迁移。
Scoped to the Skills evolution Pending/Done layout and selected-detail scroll state. No DSH/Mnemon contract, configuration, Provider, storage-format, or authority changes. Existing narrow layouts remain; rollback requires no data migration.
本地验证 / Local Validation
执行的命令 / Commands run:
结果摘要 / Result summary:
pnpm run verifyandpnpm run release:intentpassed. All plugin suites passed; root suite: 2245 passed, 7 conditional tests skipped. The two new writable/read-only regression cases fail against the previous product layout and pass with this change. Re-ranverify:docsafter completing evidence. Real WebUI checks cover independent scrolling, long detail, selection reset and 425px narrow layout.用户可见变更证据 / Local Feature Evidence
证据 / Evidence:
中文验收记录 · English evidence. Reproduce with
node scripts/serve-e2e.mjs --evolution-scroll --without-mnemon-cli; synthetic suggestions, no model requests.