i18n: row-problem log and README name precedence (review)

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
This commit is contained in:
Martin BallandClaude Opus 5.5 committed 2026-10-23 22:11:30 +01:00
1 parent 4ecda6fa8c
commit f7cc8a7df1
2 files changed
+3 -3

No files matched your search

+2 -2
View File
@@ -60,8 +60,8 @@ The app keeps its `tl!` macro, its language setting (`"auto"` → `I18N.system_l
last form. CLDR's `pt` is Brazilian, so `pt` uses `pt-PT`'s rules (0 is plural) and `pt-br`
`pt`'s. `intl_pluralrules` is pinned to the version craft-i18n's checker uses, and
`plural_form_counts` holds the same counts.
- **Names:** a language's name comes from its shared catalog's `@id language.name` row, else its
code; English is "English".
- **Names:** a language's name comes from the app catalog's `@id language.name` row, else the
shared catalog's, else its code; English is "English".
- **Right to left:** `rtl` is true for `ar`, `ckb`, `dv`, `fa`, `he`, `ps`, `sd`, `ug`, `ur` and
`yi`; putting text into display order is the app's job.
- **System locale:** `system_lang` matches the system's list once and caches it: on desktop Unix
+1 -1
View File
@@ -68,7 +68,7 @@ impl Locale {
rows.entry(e.context).or_default().insert(e.source, e.translation);
}
}
problems.iter().for_each(|p| log::warn!("translation catalog: {p}; that string shows in English"));
problems.iter().for_each(|p| log::warn!("translation catalog: {p}; row skipped"));
Parsed { rows, problems }
})
}