style: 统一滚动条样式与设置页面字体规范#366
Conversation
滚动条统一： - 降低默认透明度，hover 恢复原始值，添加过渡动画 - 移除 SearchResults、UpdateWindow、setting/global.css、Dropdown 的 重复滚动条样式，由 globalStyles.ts 和 style.css 统一管理 - toast.ts 内联样式同步更新 设置插件 UI 规范： - MCP 服务页面完全重写，对齐 HttpServiceSetting/SyncSetting 风格 移除发光背景、大圆角、阴影、浮起动画；添加折叠动画和 SVG 箭头 - GeneralSetting: setting-label 15px→14px, setting-desc 13px→12px - DebugSetting: section-title 16px→20px - ShortcutsSetting: section-title 重命名为 category-label 避免混淆 - PluginMarketSetting/CategoryDetail: section-title 重命名为 storefront-title Made-with: Cursor
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request focuses on unifying the application's visual style by centralizing scrollbar CSS into a global configuration and refactoring the MCP Service settings into a more consistent list-based layout. It also includes various font size adjustments, class renames for better specificity, and minor import cleanups. Review feedback identifies a missing state rollback mechanism when saving MCP configurations fails, potential concurrency issues when toggling multiple plugins rapidly, and the inclusion of unsupported CSS transition properties on scrollbar pseudo-elements.
| <span class="setting-desc">开启后外部 MCP 客户端可连接并调用插件工具</span> | ||
| </div> | ||
| <label class="toggle"> | ||
| <input v-model="enabled" type="checkbox" @change="handleServiceToggle" /> |
There was a problem hiding this comment.
此处绑定的 handleServiceToggle 函数（最终调用 saveConfig）在后端保存失败时缺少回滚 enabled 状态的逻辑。由于 enabled 是通过 v-model 双向绑定的，如果后端接口返回错误，UI 上的开关仍会保持在错误的状态。建议在 saveConfig 的错误处理逻辑中增加回滚操作。
| <label class="toggle" @click.stop> | ||
| <input | ||
| :checked="plugin.enabled" | ||
| :disabled="savingPluginPath === plugin.pluginPath" |
There was a problem hiding this comment.
使用 savingPluginPath === plugin.pluginPath 来禁用开关在处理并发操作时是不够的。如果用户快速点击多个插件，savingPluginPath 只会记录最后一个点击的路径，导致之前正在处理的插件开关被提前解除禁用。此外，并发修改 MCP_DISABLED_PLUGINS_KEY 可能会导致状态丢失。建议在脚本逻辑中使用 Set 管理加载状态，并确保数据库操作的顺序性。
|
|
||
| ::-webkit-scrollbar-thumb { | ||
| border-radius: 3px !important; | ||
| transition: background 0.2s ease !important; |
There was a problem hiding this comment.
在 WebKit/Chromium 环境下，::-webkit-scrollbar-thumb 等伪元素并不支持 transition 属性。这段代码不会产生预期的过渡动画效果，属于无效代码。建议移除以保持全局样式表的纯净，避免误导后续开发。同理，src/main/core/globalStyles.ts 和 src/main/api/plugin/toast.ts 中类似的定义也建议移除。
2 participants
Summary
Changes
src/main/core/globalStyles.ts/src/renderer/src/style.css/src/main/api/plugin/toast.ts: 滚动条透明度和过渡internal-plugins/setting/src/views/McpServiceSetting/McpServiceSetting.vue: 完全重写GeneralSetting.vue/DebugSetting.vue/ShortcutsSetting.vue/PluginMarketSetting.vue/CategoryDetail.vue: 字体和类名规范SearchResults.vue/UpdateWindow.vue/global.css/Dropdown.vue: 移除重复滚动条样式