Skip to content

feat(i18n): translate the extension (en, fr, es, de, it, pt_BR) + 1.4.0 - #15

Merged
AbrahamOP merged 4 commits into
mainfrom
feat/i18n
Sep 13, 2026
Merged

AbrahamOP merged 4 commits into
mainfrom
feat/i18n

Conversation

@AbrahamOP

Copy link
Copy Markdown
Owner

Empilée sur #14, elle-même empilée sur #13. À merger dans l'ordre : #13, #14, puis celle-ci. Le passage en 1.4.0 est dans cette PR : son merge sur main déclenche la release automatique.

Traduction

  • _locales en 6 langues (en par défaut, fr, es, de, it, pt_BR), 64 clés. Chrome suit la langue du navigateur.
  • La description du Chrome Web Store est traduite via __MSG_extDescription__, sous 132 caractères dans chaque langue. Le nom reste GoaBlockAD.
  • i18n.js remplit data-i18n et data-i18n-{title,placeholder,aria-label}. Tous les textes du popup et du dashboard, toasts compris, passent par t().
  • <html lang> et le format des nombres viennent de la clé langCode de la traduction affichée. En test, getUILanguage() et @@ui_locale renvoyaient en_US alors que les messages étaient en français.
  • CI : .github/scripts/check-i18n.js échoue sur une clé manquante, inconnue, non définie ou inutilisée.

Correctif UI préexistant

  • .md-chip { display: inline-flex } écrasait l'attribut hidden, si bien que « Reprendre » s'affichait en permanence à côté de « Pause ». Constaté sur main : display: flex.
  • Barre de défilement verticale parasite sous les onglets du dashboard.

Tests

…l scrollbar

.md-chip's display overrode the hidden attribute, so "Resume" showed
next to "Pause" at all times; overflow-x:auto on the tabs also made
overflow-y scroll.
- _locales with 64 keys, English as default_locale; the store
  description comes from __MSG_extDescription__ (name stays GoaBlockAD)
- i18n.js fills data-i18n / data-i18n-{title,placeholder,aria-label};
  <html lang> and number formats come from the displayed translation
  (langCode key) rather than getUILanguage(), which can disagree
- popup and dashboard strings, toasts included, go through t()
- CI: .github/scripts/check-i18n.js fails on missing, unknown, unused or
  undefined keys
Base automatically changed from feat/youtube-adblocking to main September 13, 2026 10:27
@AbrahamOP
AbrahamOP merged commit e62498d into main Sep 13, 2026
1 check passed
@AbrahamOP
AbrahamOP deleted the feat/i18n branch September 13, 2026 10:28
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.

1 participant