Skip to content

fix(i18n): vertaal kadernamen en preview in Nieuw document (#835) - #836

Closed
oscar981-op wants to merge 3 commits into
OpenAEC-Foundation:mainfrom
oscar981-op:fix/tauri-frame-labels-835
Closed

oscar981-op wants to merge 3 commits into
OpenAEC-Foundation:mainfrom
oscar981-op:fix/tauri-frame-labels-835

Conversation

@oscar981-op

@oscar981-op oscar981-op commented Oct 29, 2026 •

Copy link
Copy Markdown
Contributor

Integrated with maintainer review fixes via #862 (upstream main cdda6a0). The original tested branch is retained as reference; do not merge it again.

Closes #835

De kaderlijst in Nieuw document leidde alle namen rechtstreeks af van de Nederlandse bestandsnamen. Ook de preview toonde de ruwe bestandsnaam. De drie ingebouwde stijlen krijgen nu vertaalde labels; bijvoorbeeld grootformaat verschijnt in het Engels als “Large format”. De preview toont stijl, papierformaat, oriëntatie en afmetingen in de gekozen taal.

  • Labels en previewtekst toegevoegd aan alle 40 talen. Een taalwissel werkt ook terwijl het dialoogvenster openstaat.
  • Eigen kaderprefixen behouden hun bestaande label. Selectiewaarden, bestandsnamen, scanvolgorde en paden blijven gelijk; titelblokken zijn ongewijzigd.
  • Versies volgens de repository-instructies gesynchroniseerd naar 2026.44.8.

Validatie:

  • Browserregressie faalt vóór de fix: Engelse interface toont Grootformaat; slaagt erna.
  • Headless Chromium met echte dialoog en in-memory Tauri-bestandssysteemfixture: EN/NL-stijlen, live taalwissel, staand/liggend, maatvoering, eigen prefixen inclusief constructor, oorspronkelijke bestandslookup en blanco-pagina-preview gecontroleerd.
  • Herhaalbaar met node scripts/frame-labels-e2e.mjs <geïsoleerde-vite-server-url>.
  • Alle 40 JSON-bestanden geparseerd en preview-interpolatie gecontroleerd; TypeScript-controle, Vite-productiebouw en 40 versie-/releaseconfiguratietests geslaagd. git diff --check schoon.
  • Geen live native WebView getest; filesystemfixture leest geen gebruikersbestanden. Vertalingen buiten EN/NL zijn niet door moedertaalsprekers beoordeeld.

Bijgewerkt op upstream main 139a3198f na merge van #841. Alleen versievelden conflicteerden; upstream-inhoud behouden en versies gesynchroniseerd. Browserregressie, TypeScript, Vite-productiebouw en 40 versie-/releasecontroles opnieuw geslaagd. De functionele PR-diff blijft beperkt tot kaderlabels en previewtekst.

Actuele head: 47407a28ed34495313d5b929187af05ee58f920f. Alle upstream-controles zijn geslaagd: Windows, Linux, macOS, render-regression en hosted-csp. De installercontrole is voltooid: Android, webbundel en MCPB geslaagd; desktopjobs geblokkeerd door de bekende fork-configuratie (Azure-endpoint, updater-privésleutel en Apple-certificaatimport). Dit is geen uitspraak over organisatiecredentials of gepubliceerde upstream-installers.

Blijft concept in afwachting van succesvolle installerondertekening en verplichte review. Het huidige account heeft geen upstream-mergerecht. Fork-release v2026.44.8 blijft concept; niets gepubliceerd.

Status na upstream release #844: main 846a2a67f veroorzaakt 23 conflicten in 16 versie-/releasebestanden. Een read-only mergecontrole bevestigt dat alle conflicthunks uitsluitend versienummers betreffen; de functionele wijzigingen mergen zonder conflict. De vermelde groene CI geldt voor de bestaande PR-head, niet voor een combinatie met deze nieuwe main. Versies worden opnieuw afgestemd zodra de installer-/reviewblokkades zijn opgelost of inhoudelijke review een nieuwe push nodig maakt.

@DutchSailor DutchSailor mentioned this pull request Oct 29, 2026
@DutchSailor

DutchSailor commented Oct 29, 2026 •

Copy link
Copy Markdown
Collaborator

Thank you @oscar981-op! This is released in 2026.44.4 as one squashed commit with you as the author (merged through #862).

Review fixes on top:

  • the label and caption logic moved into js/pdf/frame-labels.js, with unit tests for all locales, custom prefixes and the en/nl/de captions;
  • the caption uses lower-case orientation words (new newDoc.frameOrientation keys; German keeps its nouns capitalised);
  • the unused capitalisation helper was removed from the dialog.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature]: Show translated names for drawing frame styles in the New Document dialog

2 participants