[codex] 修复 MCP 设置列表空白间距#277
Open
kogekiplay wants to merge 1 commit into
Open
Conversation
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
1 participant
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.
变更内容
修复原因
MCP 设置页的空隙可能来自两类问题：一类是空白/零宽字符 id 被当成真实 MCP 条目渲染；另一类是本地列表滚动区域在小窗口下与底部按钮区共享布局空间，导致视觉上出现异常间隔或挤压。本次修复同时在数据层和布局层收紧边界。
影响
用户在设置页查看本地 MCP 列表时，不会再看到空白 MCP 条目；列表内容也会被限制在独立滚动区域内，减少窗口尺寸变化时的错位和异常留白。
验证
./node_modules/.bin/prettier --check src/components/settings/mcp-settings.tsx src/components/settings/mcp-settings.test.tsx./node_modules/.bin/eslint src/components/settings/mcp-settings.tsx src/components/settings/mcp-settings.test.tsx./node_modules/.bin/vitest run src/components/settings/mcp-settings.test.tsxcargo test --no-default-features --lib commands::mcp::testsrustup run stable rustfmt --edition 2021 --check src/commands/mcp.rsgit diff --check./node_modules/.bin/next build