Skip to content

chore: 完善 dde-appearance dconfig 描述信息 - #280

Merged
mhduiy merged 1 commit into
masterfrom
agent/developer/0ad93f61
Aug 7, 2026
Merged

chore: 完善 dde-appearance dconfig 描述信息#280
mhduiy merged 1 commit into
masterfrom
agent/developer/0ad93f61

Conversation

@mhduiy

@mhduiy mhduiy commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

概述

完善 dde-appearance 项目 dconfig schema (misc/dconfig/org.deepin.dde.appearance.json) 的描述信息:保留英文 description 并新增 description[zh_CN] 字段存放中文描述,同时将 16 个 name[zh_CN] 占位符 ***** 替换为直观的中文名称。

改动范围

  • 仓库:linuxdeepin/dde-appearance,目标分支 master
  • 改动文件:仅 misc/dconfig/org.deepin.dde.appearance.json(41 增 / 18 删)
  • 仅涉及 name[zh_CN]description(恢复为 master 原始英文值)与新增 description[zh_CN](中文)三类文案字段
  • 对 2 个边缘 key(globalThemeOverrideirregularFontOverride):原 description 本为中文、无英文原文,补拟英文 description 并将中文移入 description[zh_CN]
  • value 默认值及 JSON 结构、permissions/visibility/flags/serial/global/name 等字段保持原样,不涉及任何逻辑/接口改动

说明

  • 本次为纯 dconfig 元数据文案修改,无功能行为变化;JSON 有效性已校验,23 个 key 完整,无 ***** 占位符残留,23 个 key 均含 description[zh_CN]
  • 已通过代码审核:返工版四维度 96 分 / 0 安全漏洞 / 无阻断项(审核基于 commit 369fea4,其代码树与最终提交逐字节一致,仅 commit message 文案调整)。
  • commit message 已按团队规范调整为中英双语格式,并已绑定 PMS 任务(PMS: TASK-393847 trailer)。

关联

@sourcery-ai

sourcery-ai Bot commented Aug 6, 2026

Copy link
Copy Markdown
Reviewer's guide (collapsed on small PRs)

Reviewer's Guide

Updates the dconfig schema metadata for dde-appearance by restoring/adding proper English descriptions, adding corresponding Chinese descriptions, and replacing Chinese name placeholders, without changing any functional settings or JSON structure.

File-Level Changes

Change Details Files
Refine dconfig schema metadata for appearance settings with bilingual descriptions and meaningful Chinese names.
  • Restore or provide English description values for all 23 keys, including drafting English text for previously Chinese-only descriptions
  • Add corresponding description[zh_CN] entries with Chinese descriptions for all keys
  • Replace 16 placeholder name[zh_CN] values ("*****") with clear Chinese names that match each setting
  • Ensure all non-metadata fields (value defaults, structure, permissions, visibility, flags, serial, global, name) remain unchanged and JSON stays valid
misc/dconfig/org.deepin.dde.appearance.json

Tips and commands

Interacting with Sourcery

  • Trigger a new review: Comment @sourcery-ai review on the pull request.
  • Continue discussions: Reply directly to Sourcery's review comments.
  • Generate a GitHub issue from a review comment: Ask Sourcery to create an
    issue from a review comment by replying to it. You can also reply to a
    review comment with @sourcery-ai issue to create an issue from it.
  • Generate a pull request title: Write @sourcery-ai anywhere in the pull
    request title to generate a title at any time. You can also comment
    @sourcery-ai title on the pull request to (re-)generate the title at any time.
  • Generate a pull request summary: Write @sourcery-ai summary anywhere in
    the pull request body to generate a PR summary at any time exactly where you
    want it. You can also comment @sourcery-ai summary on the pull request to
    (re-)generate the summary at any time.
  • Generate reviewer's guide: Comment @sourcery-ai guide on the pull
    request to (re-)generate the reviewer's guide at any time.
  • Resolve all Sourcery comments: Comment @sourcery-ai resolve on the
    pull request to resolve all Sourcery comments. Useful if you've already
    addressed all the comments and don't want to see them anymore.
  • Dismiss all Sourcery reviews: Comment @sourcery-ai dismiss on the pull
    request to dismiss all existing Sourcery reviews. Especially useful if you
    want to start fresh with a new review - don't forget to comment
    @sourcery-ai review to trigger a new review!

Customizing Your Experience

Access your dashboard to:

  • Enable or disable review features such as the Sourcery-generated pull request
    summary, the reviewer's guide, and others.
  • Change the review language.
  • Add, remove or edit custom review instructions.
  • Adjust other review settings.

Getting Help

@mhduiy
mhduiy marked this pull request as draft August 6, 2026 13:02

@sourcery-ai sourcery-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Hey - I've reviewed your changes and they look great!


Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

@mhduiy
mhduiy force-pushed the agent/developer/0ad93f61 branch 2 times, most recently from 369fea4 to b59d74e Compare August 6, 2026 14:04
@mhduiy
mhduiy marked this pull request as ready for review August 7, 2026 02:17

@sourcery-ai sourcery-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Hey - I've reviewed your changes and they look great!


Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

1. Replace 16 name[zh_CN] placeholder entries with readable Chinese
   names in misc/dconfig/org.deepin.dde.appearance.json
2. Restore English description to master values and add
   description[zh_CN] for 21 keys (16 placeholder + 5 secondary)
3. For 2 edge keys whose master description was Chinese with no
   English original, draft an English description and move the
   Chinese into description[zh_CN]
4. Preserve value defaults and JSON structure; only name[zh_CN],
   description, and description[zh_CN] change, with no logic or
   interface changes

Log: Refined dde-appearance dconfig descriptions, keeping English and adding Chinese zh_CN

Influence:
1. Verify the dconfig schema loads without JSON errors
2. Confirm English description and Chinese description[zh_CN] both
   display in dconfig-editor, with name[zh_CN] labels unchanged
3. Ensure all key default values and structure fields remain unchanged

chore: 完善 dde-appearance dconfig 描述信息

1. 将 misc/dconfig/org.deepin.dde.appearance.json 中 16 个 name[zh_CN] 占位符替换为直观的中文名称
2. 对 16 个原占位 key 与 5 个 §1.2 key:description 字段恢复为 master 511dc06 的原始英文值,并新增 description[zh_CN] 字段写入中文描述
3. 对 2 个边缘 key(globalThemeOverride、irregularFontOverride):原始 description 本为中文无英文原文,补拟英文 description 并将中文移入 description[zh_CN]
4. value/serial/flags/permissions/visibility/global/name 及 JSON 结构均保持原样,不涉及逻辑/接口改动

Log: 完善 dde-appearance 项目 dconfig 描述信息,保留英文并补中文 zh_CN 描述

Influence:
1. 验证 dconfig schema 可正常加载,无 JSON 错误
2. 在 dconfig-editor 中确认英文 description 与中文 description[zh_CN] 均正常显示,且 name[zh_CN] 中文标签不变
3. 确认各 key 默认值及结构字段未被改动

PMS: TASK-393847
@mhduiy
mhduiy force-pushed the agent/developer/0ad93f61 branch from b59d74e to c0ff0f2 Compare August 7, 2026 02:24
@deepin-ci-robot

Copy link
Copy Markdown

deepin pr auto review

★ 总体评分:100分

■ 【总体评价】

代码完美修复了dconfig配置文件中的国际化缺失与中英文混用问题
逻辑正确且无任何安全与性能隐患,符合满分标准

■ 【详细分析】

  • 1.语法逻辑(完全正确)✓

JSON格式规范,所有新增的键值对均正确添加了逗号分隔符,未破坏原有的数组与对象结构
潜在问题:无
建议:无

  • 2.代码质量(良好)✓

准确将占位符"*****"替换为符合UOS规范的中文术语,如"标准字体"、"光标主题"等,并成功将IrregularFontOverride等项中误写在description字段的中文移至description[zh_CN],恢复了正确的英文描述
潜在问题:无
建议:无

  • 3.代码性能(无性能问题)✓

修改仅涉及静态JSON配置文件的字符串常量替换,不增加解析开销或内存占用
建议:无

  • 4.代码安全(存在0个安全漏洞)✓

漏洞对比统计:新增漏洞 0 个,减少漏洞 0 个,持平 0 个
纯文本配置项的国际化修改,不涉及命令执行、文件读写逻辑或敏感数据暴露,无攻击面

  • 建议:保持现有的纯文本i18n修改模式

■ 【改进建议代码示例】

--- a/misc/dconfig/org.deepin.dde.appearance.json
+++ b/misc/dconfig/org.deepin.dde.appearance.json
@@ -14,7 +14,8 @@
             "serial": 0,
             "flags": [],
             "name": "Font_Standard",
-            "name[zh_CN]": "*****",
+            "name[zh_CN]": "标准字体",
             "description": "The standard font for desktop",
+            "description[zh_CN]": "桌面环境使用的标准字体",
             "permissions": "readwrite",
             "visibility": "private"

@deepin-ci-robot

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: mhduiy, robertkill

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@mhduiy
mhduiy merged commit 21387b1 into master Aug 7, 2026
27 of 28 checks passed
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