-
Notifications
You must be signed in to change notification settings - Fork 106
refactor(ui): 移除集群管理页面中的命名空间选择功能 #159
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
为了简化集群管理页面的用户界面,移除了命名空间选择功能及其相关代码。此功能在当前使用场景中不再需要,减少了不必要的复杂性。
📝 WalkthroughSummary by CodeRabbit
Summary by CodeRabbit
Walkthrough本次变更从集群权限管理界面中的“集群管理员”权限表格移除了“限制命名空间”字段及其相关的“选择命名空间”按钮和弹窗。同时,用户个人资料页中“MCP服务使用说明”弹窗新增了ESC键和点击弹窗外部关闭的功能。还对命名空间下多个资源的环境变量编辑输入框进行了UI配置调整,新增了多行文本输入的textarea样式。仅修改了JSON UI配置文件,未影响其他角色或界面元素。 Changes
Sequence Diagram(s)sequenceDiagram
participant 用户
participant 集群权限管理UI
用户->>集群权限管理UI: 查看集群管理员权限表格
Note over 集群权限管理UI: 不再显示“限制命名空间”字段和“选择命名空间”按钮
用户-->>集群权限管理UI: 仅操作剩余权限字段
sequenceDiagram
participant 用户
participant MCP服务使用说明弹窗
用户->>MCP服务使用说明弹窗: 点击“查看使用说明”按钮
MCP服务使用说明弹窗->>用户: 显示弹窗(支持ESC键和点击外部关闭)
用户->>MCP服务使用说明弹窗: 按ESC键或点击弹窗外部
MCP服务使用说明弹窗->>用户: 关闭弹窗
Possibly related PRs
Poem
Tip ⚡️ Faster reviews with caching
Enjoy the performance boost—your workflow just got faster. Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
在用户配置页面的MCP服务使用说明对话框中,新增了按ESC键和点击外部区域关闭对话框的功能,以提升用户体验
在deploy.json、daemonset.json、replicaset.json和statefulset.json文件中,为环境变量输入框添加了valueSchema配置,使其支持textarea类型,并设置最小行数为1。这一改进提升了用户输入环境变量的体验。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
🧹 Nitpick comments (1)
ui/public/pages/ns/statefulset.json (1)
1347-1351
: 改善环境变量编辑体验的优化修改此更改为环境变量设置对话框中的"envs"输入组件添加了textarea样式支持,使用户可以更方便地输入多行环境变量值。这与其他Kubernetes工作负载资源页面(deploy.json, replicaset.json, daemonset.json)保持一致,统一了用户界面行为。
考虑在实现中增加一个提示文本,告知用户可以输入多行内容,例如添加placeholder属性:"placeholder": "每行输入一个环境变量值"
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (4)
ui/public/pages/ns/daemonset.json
(1 hunks)ui/public/pages/ns/deploy.json
(1 hunks)ui/public/pages/ns/replicaset.json
(1 hunks)ui/public/pages/ns/statefulset.json
(1 hunks)
✅ Files skipped from review due to trivial changes (1)
- ui/public/pages/ns/daemonset.json
⏰ Context from checks skipped due to timeout of 90000ms (1)
- GitHub Check: Analyze (go)
🔇 Additional comments (2)
ui/public/pages/ns/replicaset.json (1)
1181-1185
:✅ Verification successful
新增环境变量输入多行 textarea 配置
通过valueSchema
将input-kv
渲染为多行textarea
并设置最小行数,提升了环境变量编辑的可用性,与其他资源页面保持一致。请验证以下相关页面中 “envs” 的
input-kv
是否已同步新增该 schema:
ui/public/pages/ns/deploy.json
ui/public/pages/ns/statefulset.json
ui/public/pages/ns/daemonset.json
🏁 Script executed:
#!/bin/bash # 验证所有工作负载页面 envs 输入组件是否已包含 textarea valueSchema rg -n '"type": "input-kv"' -g 'ui/public/pages/ns/*.json' -A5 | rg 'envs|valueSchema'Length of output: 1199
确认已同步更新 envs 多行 textarea 配置
验证结果显示,以下页面的envs
输入组件均已包含与 replicaset 相同的valueSchema
配置:
- ui/public/pages/ns/deploy.json (1471–1474)
- ui/public/pages/ns/statefulset.json (1345–1348)
- ui/public/pages/ns/daemonset.json (1300–1303)
✅ Approve code changes。
ui/public/pages/ns/deploy.json (1)
1473-1477
: 良好的UI改进,提升了环境变量编辑体验!这个变更为环境变量编辑框添加了textarea样式,使得多行文本输入更加便捷。这种改进对于编辑较长或多行的环境变量值非常有帮助,符合用户在处理复杂环境变量时的实际需求。
为了简化集群管理页面的用户界面,移除了命名空间选择功能及其相关代码。此功能在当前使用场景中不再需要,减少了不必要的复杂性。