[dotCooCoo]

What and why

untranslated_report (crates/ui-egui/src/i18n/tests.rs) lists, for each complete-message language, the status and error messages its catalog does not translate. Greek's list included 115 messages that this change translates. 113 of them have a Russian row on main (from #7482), and #7423 (French) and #7499 (Traditional Chinese) add rows for them in their languages. The other two, the "index must be a whole number" error and the PDF export warning for knockout groups, have no row in any catalog. With the interface in Greek, these messages were shown in English.

• crates/ui-egui/src/i18n/el.tsv gains 115 @msg rows: image export and encoding errors, color profiles, color books and swatch exchange files, binary DXF and DWG import, 3D and Revolve limits, script, zip, line and file errors, and the two messages above.
• The rows sit in one commented block at the end of the file. Placeholders, parameter names, command ids and literal values stay as the English writes them. The translations use the catalog's existing terms (for example εικονοστοιχεία for pixels, βιβλιοθήκη δειγμάτων for a swatch library, σύνολο ενεργειών for an action set and ομάδες διάτρησης for knockout groups).
• The knockout warning's row is keyed on the text the app shows. The report lists that message with a newline and indentation in it, because its scanner does not apply Rust's \ line continuation in crates/pdf/src/export/nodes.rs:884, so the report keeps listing it after this change.
• 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. i18n: script.run answers errors are translated in every complete-message catalog #7532 adds the Greek row for `answers[{i}]` {e}, so this change leaves that message out too. No other open pull request that changes el.tsv adds one of these 115 messages.

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 the 53 messages of #7211, #7378 and #7405 and the knockout warning under its scanned key, and its Greek message list holds none of the 115 messages.

Focused checks on the final tree, one commit on main at edb228a (checked on 2026-11-14 as the current main):

• cargo fmt --all -- --check
• cargo clippy -p vectorcraft-ui-egui --all-targets -- -D warnings
• cargo check --workspace --tests
• cargo test -p vectorcraft-ui-egui --lib i18n: 118 passed, 1 failed. The failure, translated_messages_keep_the_field_names_and_values_their_english_lists, reports two rows in fr.tsv on main that translate the field names subpath and anchor of the list [subpath, anchor] (its message counts the four dropped names as "4 rows"); this change does not touch fr.tsv, and no Greek row has that list.
• 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]

Important

Review skipped

Review was skipped due to path filters

⛔ Files ignored due to path filters: 1

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

CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including **/dist/** will override the default block on the dist directory, by removing the pattern from both the lists.

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: d2d24590-8c8d-4e87-b44b-443dd512e6f8

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• Trigger review

• 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.