Skip to content

Add Portuguese (pt-PT) translations#688

Merged
lhecker merged 2 commits into
microsoft:mainfrom
OMouta:feat/i18n-pt-pt
Oct 31, 2025
Merged

Add Portuguese (pt-PT) translations#688
lhecker merged 2 commits into
microsoft:mainfrom
OMouta:feat/i18n-pt-pt

Conversation

@OMouta

@OMouta OMouta commented Oct 30, 2025

Copy link
Copy Markdown
Contributor

Added pt-PT to default, aliases for "pt-pt" and "pt-br", translations for common UI strings. Why? Support European Portuguese users (Portugal).

@OMouta

OMouta commented Oct 30, 2025

Copy link
Copy Markdown
Contributor Author

@microsoft-github-policy-service agree

@lhecker lhecker left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thank you for doing this! Two concerns...

Comment thread i18n/edit.toml Outdated
Comment thread i18n/edit.toml Outdated
@OMouta

OMouta commented Oct 30, 2025

Copy link
Copy Markdown
Contributor Author

Thanks for the review! I removed the extra pt aliases and reverted adding pt_pt to default to avoid changing the shipped language set and binary size.

@OMouta
OMouta requested a review from lhecker October 31, 2025 13:09
@lhecker lhecker changed the title Add Portuguese (Portugal) (pt-PT) translations and aliases Add Portuguese (pt-PT) translations Oct 31, 2025
@lhecker
lhecker merged commit 82cc84a into microsoft:main Oct 31, 2025
3 checks passed
@OMouta
OMouta deleted the feat/i18n-pt-pt branch October 31, 2025 20:44
Lou32Verbose pushed a commit to Lou32Verbose/edit that referenced this pull request Jan 11, 2026
jenia90 pushed a commit to jenia90/edit that referenced this pull request Jan 30, 2026
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