Skip to content

Expose factions to localisation#21484

Merged
PunkPun merged 2 commits into
OpenRA:bleedfrom
Mailaender:l10n-factions
Jul 18, 2024
Merged

Expose factions to localisation#21484
PunkPun merged 2 commits into
OpenRA:bleedfrom
Mailaender:l10n-factions

Conversation

@Mailaender

Copy link
Copy Markdown
Member

plus a regression from #21221 fixed that appeared during string extraction and testing.

@Mailaender Mailaender mentioned this pull request Jul 16, 2024
16 tasks

@PunkPun PunkPun 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.

untranslated in ingame menu

Screenshot 2024-07-17 at 19 06 13

Comment thread OpenRA.Game/Traits/World/Faction.cs
@Mailaender

Copy link
Copy Markdown
Member Author

Fixed the GameInfoStatsLogic.

@PunkPun PunkPun 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.

I suggest following code usage to catch all places where factions are displayed. And I don't think we can translate immediately as there might be issues with multiplayer or replays

(replays menu)
Screenshot 2024-07-17 at 23 41 53

@Mailaender

Copy link
Copy Markdown
Member Author

That dropdown is unavailable to me. I tried to localize it nonetheless.

@PunkPun

PunkPun commented Jul 18, 2024

Copy link
Copy Markdown
Member

you need to select player type apparently for it to become available

@PunkPun PunkPun merged commit 5ddc7b1 into OpenRA:bleed Jul 18, 2024
@PunkPun

PunkPun commented Jul 18, 2024

Copy link
Copy Markdown
Member

changelog

@Mailaender Mailaender deleted the l10n-factions branch July 18, 2024 20:10
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