Italian: fill 232 empty strings and fix 31 broken ones - #13718
Merged
Conversation
Fills the 232 strings that were empty in the Italian translation, and
fixes strings that were functionally wrong rather than a matter of
wording:
- 7 strings whose {0} placeholders/format specs did not match English
(e.g. "Caratteri/sec: {0:0.00}" for "Chars/second: {0}", and the
statistics strings that had lost their {0:#,##0} formatting)
- 10 strings with a missing or misplaced _ menu accelerator
- "Cut" was "Taglio" (the noun) -> "Taglia"
- "Paste" was "Impasto" (dough) -> "Incolla"
- the "Speech to text" family was translated as "Sintesi vocale",
which is text-to-speech - the opposite - and once as "Da voce a SMS"
Wording comes from GitGianluc's alternative translation in #13525,
normalised to the typographic conventions already used in this file
(... rather than …, straight quotes, hyphen rather than en dash).
Version/translatedBy headers left untouched; no other strings changed.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Member
Author
|
@bovirus could you review this one? It only fills strings that were empty and fixes the placeholder/accelerator problems — no wording preferences, and your @GitGianluc the wording here is yours, just with the punctuation matched to the conventions already in the file. |
Contributor
|
I'm workinbg on revision of Italian language . |
Member
Author
could we merge this? |
Contributor
|
I beleiev that you can merge. This afternoon I will upload my changes. If @GitGianluc is agree I propose to add him as translator in the header for his contribute. |
pull Bot
pushed a commit
to matrixer2306/subtitleedit
that referenced
this pull request
Aug 17, 2026
The Italian translation update in SubtitleEdit#13718 takes its wording from GitGianluc's alternative translation, so add the credit to translatedBy. bovirus stays the maintainer and keeps his version stamp. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
|
@bovirus @niksedk OK, I’ve got no objections: thanks for mentioning me. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Step 1 of splitting up #13525, so the uncontroversial part can land now and the wording discussion can happen separately.
This PR touches only strings that were empty or functionally broken. No wording preferences, no reordering, no header changes.
What changed (263 lines)
232 strings that were empty in
Italian.jsonare now translated. The file had 250 empty strings; 232 of them exist in the alternative translation and are filled here.31 strings that were wrong rather than a matter of taste:
Chars/second: {0}wasCaratteri/sec: {0:0.00}, and the statistics strings (numberOfItalicTags,numberOfBoldTags,numberOfUnderlineTags,numberOfFontTags,numberOfAlignmentTags) had lost their{0:#,##0}formatting._menu accelerators —_OK,_Cambia maiuscole/minuscole,_Dividi sottotitolo,S_corciatoie da tastiera,_Sgancia controlli video, and stray_on strings that have none in English (Corsivo,Aggiorna).Cutwas "Taglio" (the noun, "a cut") → "Taglia"Pastewas "Impasto" (dough/batter) → "Incolla"speechToTextHintread "Da voce a SMS {0}". Now "Conversione da voce a testo".Attribution
The wording comes from @GitGianluc's alternative translation in #13525, normalised to the typographic conventions already used in this file (
...rather than…, straight quotes, hyphen rather than en dash) so the file stays internally consistent.version,translatedByandcultureNameare deliberately left untouched — @bovirus stays the maintainer of this file.Not in this PR
The remaining ~1,942 wording differences between the two translations (Visualizza/Mostra, tasti rapidi/scorciatoie, font/carattere, plugin/componenti aggiuntivi,
.../…, …). Those are preferences worth discussing one by one, not bundling into a 3,000-line diff.Testing
LanguageJsonFilesTests,LanguageFileFormatStringTests,SeLanguageJsonContextTests,EmbeddedAssetsTests— 71 passed.🤖 Generated with Claude Code