[dotCooCoo]

What and why

On main at e56d1c7, untranslated_report (crates/ui-egui/src/i18n/tests.rs) lists 167 status and error messages under "messages, all 23 languages checked": no complete-message catalog has a row for them. With the interface in Simplified Chinese, these messages were shown in English.

• crates/ui-egui/src/i18n/zh-hans.tsv gains 114 @msg rows for the listed messages, among them 3D and Revolve limits, image export and encoding errors, color profiles, color books and swatch exchange files, binary DXF and DWG import, path operation errors, graphic style, swatch and action set library files, and script, zip and regular expression errors.
• The rows sit in one commented block at the end of the file. They use the catalog's existing terms (色板交换文件, 颜色配置文件, 色彩空间, 图形样式库, 动作组, 色标, 存储为 Web 所用格式) and the header's punctuation. Placeholders, backticked names, parameter names and listed values stay as the English writes them.
• The 53 listed messages that i18n: translate the composite font errors in the complete-message languages #7211, i18n: SVG filter refusals are translated, and side-by-side menu items differ #7378 and i18n: translate the read-root, linked JavaScript and graph design refusals #7405 translate (composite fonts, SVG filters, read roots and graph design) are left out, so no row is added twice when those merge.
• The Simplified Chinese row of docs/localization-parity.md and its line in docs/target-app-parity.md have the new counts: 3,554 @msg rows, and 55 messages the report lists for zh-hans, each translated by open i18n: the justification order error is translated in every complete catalog #7116, i18n: translate the composite font errors in the complete-message languages #7211, Blob Brush: path.blob rejects invalid fidelity, angle and roundness #7330, i18n: SVG filter refusals are translated, and side-by-side menu items differ #7378 or i18n: translate the read-root, linked JavaScript and graph design refusals #7405.

This is the Simplified Chinese part of the missing message rows, next to #7423 (French), #7463 (Japanese), #7470 (Brazilian Portuguese) and #7483 (Ukrainian). Checked against main at e56d1c7: git merge-tree with the heads of #7116, #7211, #7330, #7352, #7378, #7405, #7423, #7434, #7463, #7470, #7472 and #7483 merges without a conflict.

How it was tested

The change adds catalog rows only, so it has no new test. untranslated_report is a report that does not fail on a gap. Before the change its "messages, all 23 languages checked" list had 167 entries; after it, 53, each of them a row that #7211, #7378 or #7405 adds. Its Simplified Chinese message list holds none of the 114 messages.

Focused checks on the final tree, on main at e56d1c7:

• cargo test -p vectorcraft-ui-egui --lib i18n: 115 passed, 0 failed.
• cargo fmt --all -- --check; cargo check --workspace --tests; cargo clippy -p vectorcraft-ui-egui --all-targets -- -D warnings; cargo xtask brands; cargo xtask layers (30 crates, no layering violations); cargo xtask assets (337 asset files attributed).
• cargo xtask ci runs in this pull request's CI.

Checklist

• cargo xtask ci passes (fmt, clippy, tests, assets, brands, layers, wasm).
• No panics in shipped code: no unwrap(), expect(), panic!, unreachable!, todo! or
  unimplemented! outside tests, and no unchecked indexing on data from files, parameters or
  messages. Errors are returned as Result and shown to the user
  (why and how).
• Clean-room: nothing copied from Adobe products or GPL/AGPL code (AGENTS.md).
• Every new asset has a row in ASSETS.md.

[coderabbitai]

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 3 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used all 10 included reviews currently available.

Learn how review limits work.

Review configuration:

Run configuration

• Source mode: Auto
• Repository: storytold/vectorcraft
• Branch: main
• File: .coderabbit.yaml
• Commit: e56d1c7
• Layers: Repository YAML
• Review profile: CHILL
• Plan: Advanced
• Run ID: 2d748c87-ca7f-4ae4-a242-40c5c4aeb3a1

Commits

Reviewing files that changed from the base of the PR and between e56d1c7 and 3341061.

Files ignored due to path filters: 1

• crates/ui-egui/src/i18n/zh-hans.tsv is excluded by !**/*.tsv

Files selected for processing: 2

• docs/localization-parity.md
• docs/target-app-parity.md

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

Share

• X
• Mastodon
• Reddit
• LinkedIn

[bflatastic]

Merged, thanks @dotCooCoo! It passed its CI checks.