feat(notification): add notification setting translation#1471
feat(notification): add notification setting translation#147118202781743 merged 1 commit intolinuxdeepin:masterfrom
Conversation
Add translation for Notification Setting in notification center. 为通知中心添加通知设置的翻译。 Log: 添加通知设置翻译 PMS: BUG-351769 Influence: 通知中心新增通知设置功能的翻译支持,提升多语言用户体验。
deepin pr auto reviewGit Diff 代码审查报告总体评价这是一个针对通知中心(NotificationCenter)的多语言翻译文件(.ts)的修改。主要涉及UI文本的重新排序和新增"Notification Setting"的翻译。整体修改符合Qt TS文件格式规范,但存在一些需要改进的地方。 详细审查意见1. 语法和格式问题1.1 XML格式一致性问题: 多个翻译文件中XML声明格式不统一 <!-- 不一致的格式 -->
<?xml version="1.0" ?><!DOCTYPE TS><TS version="2.1" language="bn">
<!-- 标准格式 -->
<?xml version="1.0" encoding="utf-8"?>
<!DOCTYPE TS>
<TS version="2.1" language="bn">建议: 统一所有翻译文件的XML声明格式,使用标准格式并明确指定编码为utf-8。 1.2 空标签格式问题: 部分文件中 <!-- 不一致 -->
<translation type="unfinished"/>
<translation type="unfinished"></translation>建议: 统一使用闭合标签格式 2. 代码逻辑问题2.1 文本顺序变更问题: GroupNotify和NotifyHeader中的文本顺序发生了变化 <!-- 原顺序 -->
<source>Clear All</source>
<source>Fold</source>
<source>More</source>
<!-- 新顺序 -->
<source>Fold</source>
<source>More</source>
<source>Clear All</source>建议:
2.2 上下文重复问题: "Notification Setting"出现在多个context中 <!-- NotifyHeader context -->
<message>
<source>Notification Setting</source>
<translation type="unfinished"></translation>
</message>
<!-- NotifySetting context -->
<message>
<source>Notification Setting</source>
<translation type="unfinished"></translation>
</message>建议:
3. 代码质量问题3.1 翻译完整性问题: 新增的"Notification Setting"翻译标记为unfinished <translation type="unfinished">通知设置</translation>建议:
3.2 俄语文件中的特殊变更问题: ru.ts文件中存在额外的上下文重组 <!-- 位置变更 -->
<message>
<source>Notification Center</source>
<translation type="unfinished"></translation>
</message>
<message>
<source>Clear All</source>
<translation>Очистить всё</translation>
</message>建议:
4. 性能和安全问题4.1 文件大小问题: 多语言文件数量多且内容相似
4.2 特殊字符处理问题: 某些语言(如阿拉伯语)包含从右到左(RTL)文本 <translation>إعداد الإخطارات</translation>建议:
改进建议总结
优先级建议
这些改进将有助于提高代码质量和维护性,确保多语言支持的稳定性和一致性。 |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: 18202781743, add-uos The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
Add translation for Notification Setting in notification center.
为通知中心添加通知设置的翻译。
Log: 添加通知设置翻译
PMS: BUG-351769
Influence: 通知中心新增通知设置功能的翻译支持,提升多语言用户体验。