Skip to content

Fix broken translations on the update page - #3431

Merged
Salvialf merged 1 commit into
developfrom
feat/harmonize-update-page-buttons
Jul 17, 2026
Merged

Fix broken translations on the update page#3431
Salvialf merged 1 commit into
developfrom
feat/harmonize-update-page-buttons

Conversation

@Salvialf

Copy link
Copy Markdown
Contributor

The recent update page harmonization (#3426) introduced several translation regressions:

  • Some confirm dialogs interpolated a JS variable (logicalId, type) inside the {{ }} translation markers instead of outside, so the raw JS concatenation syntax was captured as literal text to translate instead of a clean sentence.
  • Two strings used double quotes as the JS string delimiter around a {{ }} block. The translation system only escapes single quotes in translated text, so any translation containing a literal " (as happened for the English "Information" and "Save" strings) broke the JS parsing or the surrounding HTML attribute.

This PR restores the correct {{ }} usage (fixed text only, variables outside the markers) and reverts to single-quoted JS strings where {{ }} is used, matching the convention used everywhere else in the codebase. It also drops an ambiguous parenthetical from the "do not update this plugin" tooltip that was prompting the translator to add quotes around "Save", which broke the title="..." attribute.

@Salvialf
Salvialf requested a review from Mips2648 July 17, 2026 11:03
@Salvialf Salvialf added Need approval changelog-fix Use to generate release notes / changelog To be apply on PR labels Jul 17, 2026
@Salvialf
Salvialf requested review from Sekiro-kost and zoic21 July 17, 2026 11:03
@Salvialf
Salvialf merged commit 0e51a6a into develop Jul 17, 2026
9 checks passed
@Salvialf
Salvialf deleted the feat/harmonize-update-page-buttons branch July 17, 2026 14:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog-fix Use to generate release notes / changelog To be apply on PR Need approval

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants