[dotCooCoo]

What and why

On main at 1e4e353, 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 Brazilian Portuguese, these messages were shown in English.

• crates/ui-egui/src/i18n/pt-br.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, automation and API parameter errors, and script, zip, line and file errors.
• The rows sit in one commented block before the Path Effects section. Placeholders, parameter names and literal 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, a linked script outside the read roots and graph design art) are left out, so no row is added twice when those merge.
• The Portuguese row of docs/localization-parity.md has the new counts, and docs/target-app-parity.md has one line for the catalog's message rows.

This is the Brazilian Portuguese part of the missing message rows; each other language goes up as its own pull request. Checked against main at 1e4e353: git merge-tree with the heads of #7211, #7378, #7405, #7116, #7330, #7410, #7423, #7426, #7430, #7434 and #7435 merges without a conflict. It conflicts with #7307 in docs/localization-parity.md, where #7307 changes the French row next to the Portuguese one; the resolution keeps both lines.

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. After the change, its "messages, all 23 languages checked" list has 53 entries, each of them a row that #7211, #7378 or #7405 adds, and its Brazilian Portuguese message list holds none of the 114 messages.

Focused checks on the final tree, on main at 1e4e353:

• cargo test -p vectorcraft-ui-egui --lib i18n: 115 passed, 0 failed.
• cargo clippy -p vectorcraft-ui-egui --all-targets -- -D warnings; cargo check --workspace --tests; cargo xtask brands; cargo xtask layers (30 crates, no layering violations); cargo xtask assets (337 asset files attributed).
• cargo fmt --all -- --check reports one diff, in crates/svg/src/lib.rs at line 491, which this pull request does not change; it is on main at 1e4e353 too, and fmt: format crates/svg/src/lib.rs #7400 fixes it.
• 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 1 minute.

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: a47d7ee
• Layers: Repository YAML
• Review profile: CHILL
• Plan: Advanced
• Run ID: dc7971d3-f6c4-482f-950d-fb9ddc21d932

Commits

Reviewing files that changed from the base of the PR and between a47d7ee and 4ca74c0.

Files ignored due to path filters: 1

• crates/ui-egui/src/i18n/pt-br.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.