fix(i18n): enhetlig språkkontroll på båda sidorna #2
Merged
Peter Wickenberg opened 6 months ago

Sammanfattning

Fixar i18n-inkonsekvens vid tabbyte mellan enskild domän och batch-sidan.

Problem

  • index.html använde hårdkodad svenska för nav-flikar, knapp och footer
  • batch.html använde data-i18n-attribut med dynamisk översättning
  • Engelska användare fick språkflimmer vid tabbyte (svenska nav → engelska nav)
  • batch.js hade stavfel i svensk tagline ("for" istället för "för")
  • batch.js satte inte document.documentElement.lang

Ändringar

  • Lagt till data-i18n på nav-flikar, tagline, knapp och footer i index.html
  • Ersatt manuella översättningar i app.js med data-i18n-baserad applyTranslations
  • Lagt till footer-nyckel och nav-nycklar i båda i18n-objekten
  • Fixat stavfel i batch.js
  • Sätter document.documentElement.lang i batch.js

Hur man testar

  1. docker compose build && docker compose up -d
  2. Öppna http://localhost med engelskspråkig webbläsare
  3. Verifiera att nav-flikar, tagline, knapp och footer är på engelska
  4. Klicka "Batch report" → verifiera att allt fortfarande är på engelska (ingen flimmer)
  5. Klicka "Single domain" → verifiera konsistent språk
  6. Kör batch-jobb, växla flik, växla tillbaka → verifiera att resultat bevaras

Risker

Ingen – enbart frontend, ingen API-ändring.

Commits were squashed into a single commit on target branch
  • fix(i18n): enhetlig sprakkontroll pa bade index- och batch-sidan
    Tidigare anvande index.html hardkodad svenska for nav-flikar, knapp och
    footer medan batch.html anvande data-i18n-attribut. Detta gav sprakflimmer
    vid tabbyte for engelska anvandare.
    
    Andringar:
    - Lagg till data-i18n pa nav-flikar, tagline, knapp och footer i index.html
    - Ersatt manuella oversattningar i app.js med data-i18n-baserad applyTranslations
    - Lagg till footer-nyckel i batch.js i18n
    - Fixa stavfel i batch.js (for -> for)
    - Satt document.documentElement.lang i batch.js
    
    Testat: Docker-build + Playwright-verifiering av bade index och batch-sidan,
    engelska oversattningar konsistenta vid tabbyte.
    6 months ago
  • Peter Wickenberg merged 6 months ago (merge commit: c2c34420 )
1/1
Submitter Peter Wickenberg
Target main
Source fix/i18n-batch-state
Merge Strategy
Squash Source Branch Commits
Watchers (1)
Reference
PR m365-dns-checkup#2
Please wait...
Connection lost or session expired, reload to recover
Page is in error, reload to recover