Skip to content

Add AliDoc translations and enhance the rounded corner effect - #536

Open
zjhcx wants to merge 4 commits into
OpenListTeam:mainfrom
zjhcx:main
Open

Add AliDoc translations and enhance the rounded corner effect#536
zjhcx wants to merge 4 commits into
OpenListTeam:mainfrom
zjhcx:main

Conversation

@zjhcx

@zjhcx zjhcx commented May 30, 2026

Copy link
Copy Markdown

Summary / 摘要

  • This PR has breaking changes.
    / 此 PR 包含破坏性变更。
  • This PR changes public API, config, storage format, or migration behavior.
    / 此 PR 修改了公开 API、配置、存储格式或迁移行为。
  • This PR requires corresponding changes in related repositories.
    / 此 PR 需要关联仓库同步修改。

Related repository PRs / 关联仓库 PR:

Related Issues / 关联 Issue

Testing / 测试

  • go test ./...
  • Manual test / 手动测试:

Checklist / 检查清单

  • I have read CONTRIBUTING.
    / 我已阅读 CONTRIBUTING
  • I confirm this contribution follows the repository license, contribution policy, and code of conduct.
    / 我确认此贡献符合仓库许可证、贡献规范和行为准则。
  • I have formatted the changed code with gofmt, go fmt, or prettier where applicable.
    / 我已按适用情况使用 gofmtgo fmtprettier 格式化变更代码。
  • I have requested review from relevant maintainers or code owners where applicable.
    / 我已在适用情况下请求相关维护者或代码所有者审查。

AI Disclosure / AI 使用声明

  • This PR includes AI-assisted content.
    / 此 PR 包含 AI 辅助内容。

Tools used / 使用工具:

  • ChatGPT
  • Codex
  • GitHub Copilot
  • Claude
  • Gemini
  • Other (please specify) / 其他(请注明):

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-By attribution.
    / 我已确保所有 AI 辅助提交都包含 Co-Authored-By 归属信息。

  • I can reproduce all AI-assisted content included in this PR without any AI tools.
    / 我可以在没有任何 AI 工具的情况下重现此 PR 中包含的所有 AI 辅助内容。

zjhcx added 2 commits May 30, 2026 21:09
Generated with OpenAI Codex
Generated with OpenAI Codex
@zjhcx

zjhcx commented May 31, 2026

Copy link
Copy Markdown
Author

@xrgzs 审查一下代码

@zjhcx zjhcx changed the title Add and update AliDoc translations and alert copy Add AliDoc translations and enhanced fillets Jun 14, 2026
@zjhcx zjhcx changed the title Add AliDoc translations and enhanced fillets Add AliDoc translations and enhance the rounded corner effect Jun 14, 2026

@xrgzs xrgzs left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

圆角大小不统一

xrgzs added a commit that referenced this pull request Jun 26, 2026
Signed-off-by: MadDogOwner <xiaoran@xrgzs.top>

@pikachuren pikachuren left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

结论:请求修改 — 建议拆分 PR,并留意全局圆角 token 的影响面

这个 PR 实际包含两件互不相关的事:一是新增 AliDoc 驱动的英文文案,二是全站圆角风格调整。建议拆成两个 PR,理由在下面第二点会具体说明——它们的评审所需信息和风险级别差别很大,混在一起会让样式改动缺少应有的审视。

另外 PR 标题 Add AliDoc translations and enhance the rounded corner effect 不符合仓库要求的 Conventional Commits 格式,标题检查会拦截。拆分后可分别写成 feat(i18n): add AliDoc driver stringsstyle(theme): increase rounded corner radii

一、AliDoc 文案部分:没有问题

三处位置都正确,AliDoc 按现有的大小写敏感排序规则插在 AList V3Alias 之间,与文件既有顺序一致。alert 里注明「当前为只读」是很有必要的提示,能避免用户误以为可以写入。这部分可以直接合。

二、圆角改动:影响面比 diff 显示的更大

关键问题在这里:

radii: {
  xl: "14px",
  "2xl": "16px",
},

这不是新增 token,而是覆盖了 hope-ui 主题中已有的 $xl / $2xl 的值。也就是说,凡是当前已经在使用这两个 token 的地方,都会跟着一起变,而这些地方并不在本 PR 的 diff 里。我搜了一下,至少这三处现在就在用 $xl

  • src/pages/home/Readme.tsx
  • src/pages/home/Obj.tsx
  • src/pages/login/index.tsx

这几处的圆角会从 hope-ui 默认值静默变成 14px,包括登录页和首页主容器这类很显眼的位置。这未必是坏事,但应当是有意为之并经过确认,而不是作为副作用发生。建议在 PR 描述里说明是否也希望这些位置一起变化,并附上改动前后的截图对比。

如果只想调整本 PR 明确列出的那些组件,更稳妥的做法是新增独立 token(例如 radii: { control: "14px", surface: "16px" })而不是覆盖标准刻度,这样影响范围就完全等于 diff 范围。

三、几处细节

  • lightThemedarkTheme 里重复定义了同一份 radii。圆角与配色无关,重复容易在后续修改时漏改其中一份,建议提取到共享位置。
  • 全局样式里对右键菜单用了 !important
".solid-contextmenu, .solid-contextmenu .solid-contextmenu__submenu": {
  borderRadius: "$2xl !important",
},

这里依赖了第三方库 solid-contextmenu 的内部 class 名(含 __content__submenu 等 BEM 结构)。库升级改了 class 名就会静默失效,!important 也会给后续覆盖造成困难。建议加一行注释说明为何必须用 !important,方便后来者判断能否移除。

综合建议:先把 AliDoc 文案单独拆出来合并,圆角部分补充截图和影响面说明后再单独评审。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants