Add AliDoc translations and enhance the rounded corner effect - #536
Add AliDoc translations and enhance the rounded corner effect#536zjhcx wants to merge 4 commits into
Conversation
Generated with OpenAI Codex
Generated with OpenAI Codex
|
@xrgzs 审查一下代码 |
Signed-off-by: MadDogOwner <xiaoran@xrgzs.top>
pikachuren
left a comment
There was a problem hiding this comment.
结论:请求修改 — 建议拆分 PR,并留意全局圆角 token 的影响面
这个 PR 实际包含两件互不相关的事:一是新增 AliDoc 驱动的英文文案,二是全站圆角风格调整。建议拆成两个 PR,理由在下面第二点会具体说明——它们的评审所需信息和风险级别差别很大,混在一起会让样式改动缺少应有的审视。
另外 PR 标题 Add AliDoc translations and enhance the rounded corner effect 不符合仓库要求的 Conventional Commits 格式,标题检查会拦截。拆分后可分别写成 feat(i18n): add AliDoc driver strings 和 style(theme): increase rounded corner radii。
一、AliDoc 文案部分:没有问题
三处位置都正确,AliDoc 按现有的大小写敏感排序规则插在 AList V3 与 Alias 之间,与文件既有顺序一致。alert 里注明「当前为只读」是很有必要的提示,能避免用户误以为可以写入。这部分可以直接合。
二、圆角改动:影响面比 diff 显示的更大
关键问题在这里:
radii: {
xl: "14px",
"2xl": "16px",
},这不是新增 token,而是覆盖了 hope-ui 主题中已有的 $xl / $2xl 的值。也就是说,凡是当前已经在使用这两个 token 的地方,都会跟着一起变,而这些地方并不在本 PR 的 diff 里。我搜了一下,至少这三处现在就在用 $xl:
src/pages/home/Readme.tsxsrc/pages/home/Obj.tsxsrc/pages/login/index.tsx
这几处的圆角会从 hope-ui 默认值静默变成 14px,包括登录页和首页主容器这类很显眼的位置。这未必是坏事,但应当是有意为之并经过确认,而不是作为副作用发生。建议在 PR 描述里说明是否也希望这些位置一起变化,并附上改动前后的截图对比。
如果只想调整本 PR 明确列出的那些组件,更稳妥的做法是新增独立 token(例如 radii: { control: "14px", surface: "16px" })而不是覆盖标准刻度,这样影响范围就完全等于 diff 范围。
三、几处细节
lightTheme和darkTheme里重复定义了同一份radii。圆角与配色无关,重复容易在后续修改时漏改其中一份,建议提取到共享位置。- 全局样式里对右键菜单用了
!important:
".solid-contextmenu, .solid-contextmenu .solid-contextmenu__submenu": {
borderRadius: "$2xl !important",
},这里依赖了第三方库 solid-contextmenu 的内部 class 名(含 __content、__submenu 等 BEM 结构)。库升级改了 class 名就会静默失效,!important 也会给后续覆盖造成困难。建议加一行注释说明为何必须用 !important,方便后来者判断能否移除。
综合建议:先把 AliDoc 文案单独拆出来合并,圆角部分补充截图和影响面说明后再单独评审。
Summary / 摘要
/ 此 PR 包含破坏性变更。
/ 此 PR 修改了公开 API、配置、存储格式或迁移行为。
/ 此 PR 需要关联仓库同步修改。
Related repository PRs / 关联仓库 PR:
Related Issues / 关联 Issue
Testing / 测试
go test ./...Checklist / 检查清单
/ 我已阅读 CONTRIBUTING。
/ 我确认此贡献符合仓库许可证、贡献规范和行为准则。
gofmt,go fmt, orprettierwhere applicable./ 我已按适用情况使用
gofmt、go fmt或prettier格式化变更代码。/ 我已在适用情况下请求相关维护者或代码所有者审查。
AI Disclosure / AI 使用声明
/ 此 PR 包含 AI 辅助内容。
Tools used / 使用工具:
Usage scope / 使用范围:
Code generation / 代码生成
Refactoring / 重构
Documentation / 文档
Tests / 测试
Translation / 翻译
Review assistance / 审查辅助
I have reviewed and validated all AI-assisted content included in this PR.
/ 我已审核并验证此 PR 中的所有 AI 辅助内容。
I have ensured that all AI-assisted commits include
Co-Authored-Byattribution./ 我已确保所有 AI 辅助提交都包含
Co-Authored-By归属信息。I can reproduce all AI-assisted content included in this PR without any AI tools.
/ 我可以在没有任何 AI 工具的情况下重现此 PR 中包含的所有 AI 辅助内容。