Skip to content

feat(i18n): add 76 missing pt-BR translation keys#2401

Open
obrenoalvim wants to merge 1 commit into
hydralauncher:mainfrom
obrenoalvim:feat/pt-BR-missing-translations
Open

feat(i18n): add 76 missing pt-BR translation keys#2401
obrenoalvim wants to merge 1 commit into
hydralauncher:mainfrom
obrenoalvim:feat/pt-BR-missing-translations

Conversation

@obrenoalvim

Copy link
Copy Markdown

When submitting this pull request, I confirm the following (please check the boxes):

  • I have read the Hydra documentation.
  • I have checked that there are no duplicate pull requests related to this request.
  • I have considered, and confirm that this submission is valuable to others.
  • I accept that this submission may not be used and the pull request may be closed at the discretion of the maintainers.

Fill in the PR content:

Adds 76 keys missing from the Brazilian Portuguese translation. All keys exist in en/translation.json but were absent from pt-BR/translation.json, causing the app to fall back to English for these strings.

Sections covered:

  • header — scan-games modal (progress hint, hide button, mode labels, folder management, detection warning)
  • catalogue — platforms filter label
  • game_details — collection management, controller support labels, region labels, torrent error messages, drive picker, transfer progress
  • downloads — extraction failure title and description
  • settings — emulation setup labels (wrong platform, run in background, scanning system)
  • user_profile — platform filter (All / PC)
  • library — new folder button, full context menu labels
  • friends_window — minimize and close window controls

Add translations for header scan-games modal, catalogue platforms,
game_details (collection, controller support, regions, torrent errors,
drive picker), downloads extraction error, settings emulation labels,
user_profile platform filters, library context menu, and
friends_window window controls.
@sonarqubecloud

Copy link
Copy Markdown

@greptile-apps

greptile-apps Bot commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR adds 76 missing Brazilian Portuguese (pt-BR) translation keys across 8 sections of the app, preventing English fallback for those strings. All translations are accurate, idiomatic, and preserve interpolation variables (e.g. {{system}}).

  • Sections covered: header (scan-games modal), game_details (collections, regions, torrent errors, controller support, drive picker), downloads (extraction failure), settings (emulation labels), catalogue (platforms filter), user_profile (platform filter), library (context menu), friends_window (window controls).
  • JSON structure: All commas and nesting are correct; the file remains valid JSON throughout.

Confidence Score: 5/5

Safe to merge — pure translation additions with no logic changes.

All 76 new keys are accurate Brazilian Portuguese translations of their English counterparts, JSON syntax is valid, and the {{system}} interpolation placeholder is preserved. No keys were removed or renamed, so no regressions are possible.

No files require special attention.

Important Files Changed

Filename Overview
src/locales/pt-BR/translation.json Adds 76 missing pt-BR translation keys; all values are accurate translations of the English source, JSON syntax is valid, and {{system}} interpolation variable is preserved correctly.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[pt-BR translation.json] --> B[header +9 keys]
    A --> C[game_details +43 keys]
    A --> D[downloads +2 keys]
    A --> E[settings +3 keys]
    A --> F[catalogue +1 key]
    A --> G[user_profile +3 keys]
    A --> H[library +13 keys]
    A --> I[friends_window +2 keys]
    B --> J[76 total keys restored]
    C --> J
    D --> J
    E --> J
    F --> J
    G --> J
    H --> J
    I --> J
Loading
%%{init: {'theme': 'base', 'themeVariables': {"darkMode": true, "background": "#0d1117", "primaryColor": "#21262d", "primaryTextColor": "#e6edf3", "primaryBorderColor": "#8b949e", "lineColor": "#8b949e", "textColor": "#e6edf3", "edgeLabelBackground": "#161b22", "actorBkg": "#21262d", "actorBorder": "#8b949e", "actorTextColor": "#e6edf3", "actorLineColor": "#8b949e", "signalColor": "#8b949e", "signalTextColor": "#e6edf3", "noteBkgColor": "#373320", "noteBorderColor": "#d4a72c", "noteTextColor": "#f0e6c0", "labelBoxBkgColor": "#21262d", "labelBoxBorderColor": "#8b949e", "labelTextColor": "#e6edf3", "loopTextColor": "#e6edf3", "activationBkgColor": "#30363d", "activationBorderColor": "#8b949e"}}}%%
flowchart TD
    A[pt-BR translation.json] --> B[header +9 keys]
    A --> C[game_details +43 keys]
    A --> D[downloads +2 keys]
    A --> E[settings +3 keys]
    A --> F[catalogue +1 key]
    A --> G[user_profile +3 keys]
    A --> H[library +13 keys]
    A --> I[friends_window +2 keys]
    B --> J[76 total keys restored]
    C --> J
    D --> J
    E --> J
    F --> J
    G --> J
    H --> J
    I --> J
Loading

Reviews (1): Last reviewed commit: "feat(i18n): add 76 missing pt-BR transla..." | Re-trigger Greptile

@Moyasee Moyasee left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Verified against en: no duplicate keys, no orphans introduced, placeholders match, and this actually brings pt-BR to full parity (0 keys still missing). Nice work. Thanks!

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.

2 participants