Skip to content

Add Brazilian Portuguese (pt-BR) localization#925

Open
sauloverissimo wants to merge 1 commit intomicrosoft:mainfrom
sauloverissimo:pt-br-localization
Open

Add Brazilian Portuguese (pt-BR) localization#925
sauloverissimo wants to merge 1 commit intomicrosoft:mainfrom
sauloverissimo:pt-br-localization

Conversation

@sauloverissimo
Copy link

This adds pt-BR translations for the Settings app, Console app, and shared string resources — the first non-English locale for the project. Relates to #515.

Files added (no existing files modified):

  • src/user-tools/midi-settings/.../Strings/pt-BR/Resources.resw (231 entries)
  • src/user-tools/midi-console/Midi/Resources/Strings.pt-BR.resx (349 entries)
  • src/shared/strings/pt-BR/MidiEntities.resw (21 entries)
  • src/shared/strings/pt-BR/MidiMessageResources.resw (32 entries)

Technical MIDI terms (UMP, SysEx, endpoint, Function Block, loopback, Panic, etc.), product names, URLs, and MIDI® symbols were kept as-is. No .csproj changes needed — MRT and .NET detect locale folders automatically.

To help with review, here's a suggested test plan:

  • Verify XML well-formedness of the 4 resource files
  • Build Settings app and Console with pt-BR resources present
  • Set Windows language to pt-BR and check translated strings appear
  • Confirm en-US fallback still works when pt-BR is not active

Add pt-BR translations for Settings app, Console app, and shared
string resources. First non-English locale for the project.

Relates to microsoft#515
@sauloverissimo
Copy link
Author

@microsoft-github-policy-service agree

@sauloverissimo
Copy link
Author

I imagine reviewing a translation in a different language must be a different experience, but I went through all the translations again and everything looks solid -- consistent terminology, proper context for MIDI-specific terms, and natural phrasing for Brazilian Portuguese speakers. If there's anything specific you'd like me to double-check or if you have a preferred review process for localization PRs, happy to help with that too.

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