feat(settings): add configurable font preferences#124
feat(settings): add configurable font preferences#124MoozLee wants to merge 2 commits intoxintaofei:mainfrom
Conversation
|
🟥 PR 描述与实现不一致 SystemFontSettings 只有 ui_font_family + code_font_family 两个字段(models/system.rs) 🟧 Web 模式体验静默降级 🟧 性能:fontdb 没有缓存 🟧 AppearanceProvider 过度设计 🟨 模型 cfg 标注不一致 🟨 测试覆盖 建议请作者修订后再合: 修正 PR 描述:要么删掉 terminal/AI 字体的措辞,要么把那两类真正实现出来 |
Share font settings persistence across desktop and web paths, cache font enumeration, and simplify appearance state so fallback mode keeps stored choices instead of silently resetting them.
|
已根据这轮 review feedback 补上一轮修复,PR head 现在已经包含最新提交: 这次主要处理了下面几项:
另外我已经同步更新了 PR description,使其与当前实际实现一致。 本轮验证:
说明:这次 CLI 修复里没有做手动桌面端 / Web UI smoke test;如果需要,我可以再补一轮针对字体切换、刷新保留、terminal refit 的手动验证说明。 |
Summary / 摘要
中文
English
Follow-up for review feedback / 针对评审意见的后续修复
中文
fix/pr-124-font-settings-review,用于处理评审里指出的几点:get/update_system_font_settings路径补齐fontdb进程级缓存AppearanceProvider简化English
fix/pr-124-font-settings-review, contains the review-driven fixes for:get/update_system_font_settingsroutesfontdbcachingAppearanceProvidersimplificationTest plan / 测试计划
git diff --check origin/main...HEADpnpm -C /Users/lee/projects/codeg exec eslint .pnpm -C /Users/lee/projects/codeg buildcargo check --manifest-path /Users/lee/projects/codeg/src-tauri/Cargo.tomlcargo check --manifest-path /Users/lee/projects/codeg/src-tauri/Cargo.toml --bin codeg-server --no-default-featuresNotes / 说明
中文
English