Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Extracted the Tiberian Dawn mission objective strings #20265

Merged
merged 4 commits into from
Nov 17, 2022

Conversation

Mailaender
Copy link
Member

and enhance the Fluent Lua interface.

@Mailaender Mailaender mentioned this pull request Sep 6, 2022
16 tasks
@Mailaender Mailaender force-pushed the fluent-lua-cnc branch 2 times, most recently from fbeb7cf to b24186c Compare September 6, 2022 19:27
Copy link
Member

@abcdefg30 abcdefg30 left a comment

Choose a reason for hiding this comment

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

Is there a specific idea behind splitting some mission translations into their own files instead of having everything in cnc/languages/en.ftl? (Having only a single file seems simpler to me.)

mods/cnc/languages/en.ftl Outdated Show resolved Hide resolved
mods/cnc/languages/nod06abc/en.ftl Outdated Show resolved Hide resolved
@PunkPun
Copy link
Member

PunkPun commented Sep 7, 2022

Is there a specific idea behind splitting some mission translations into their own files instead of having everything in cnc/languages/en.ftl? (Having only a single file seems simpler to me.)

I think we should keep one file for the main campaign missions, it will be easier for translators and for us to maintain. For skirmish gamemodes or perhaps even for openra unique missions we support translations could be in the maps themselves

@PunkPun
Copy link
Member

PunkPun commented Sep 7, 2022

Not to forget that in the future once we get proper campaign UI we will need translations for those as well on the mod (not engine) level

@penev92
Copy link
Member

penev92 commented Sep 7, 2022

How does scoping for these mission-specific translation files work? Is there any? If yes, that would be the reason to keep them separate. If we combined them we'd likely need mission-specific prefixes on the string keys like

-destroy-gdi-forces = Destroy all GDI forces.
+nod03-destroy-gdi-forces = Destroy all GDI forces.

@PunkPun
Copy link
Member

PunkPun commented Sep 7, 2022

we would more likely just add comments, like it is in mods/cnc/languages/en.ftl

-nod03-destroy-gdi-forces = Destroy all GDI forces.
+#nod03
+destroy-gdi-forces = Destroy all GDI forces.

@penev92
Copy link
Member

penev92 commented Sep 7, 2022

That will only work until we start getting conflicting objectives from multiple missions.

@PunkPun
Copy link
Member

PunkPun commented Sep 7, 2022

Applying the #20207 concept would help with that. Though in theory multiple missions shouldn't collide, if they have different wording, since variable name is based on the wording, they will have different variable names

@penev92
Copy link
Member

penev92 commented Sep 7, 2022

That would likely work, but it neither foolproof nor clean. One would need to be scouring existing strings when adding a new one, finding an unused sequence of characters like trying to register an email account.

@PunkPun
Copy link
Member

PunkPun commented Sep 7, 2022

It's the approach we've taken on engine-level translations. Linting could be added to check for duplicates

@Mailaender
Copy link
Member Author

I went with the monolithic file structure similar to what we already have now.

mods/cnc/maps/nod02a/en.ftl Outdated Show resolved Hide resolved
mods/cnc/languages/nod06abc/en.ftl Outdated Show resolved Hide resolved
mods/cnc/languages/lua/en.ftl Outdated Show resolved Hide resolved
mods/cnc/languages/lua/en.ftl Show resolved Hide resolved
@Mailaender Mailaender force-pushed the fluent-lua-cnc branch 2 times, most recently from 7a14a4e to 80db408 Compare September 7, 2022 20:37
mods/cnc/languages/lua/en.ftl Outdated Show resolved Hide resolved
mods/cnc/languages/lua/en.ftl Show resolved Hide resolved
mods/cnc/languages/lua/en.ftl Outdated Show resolved Hide resolved
mods/cnc/scripts/campaign.lua Outdated Show resolved Hide resolved
@PunkPun
Copy link
Member

PunkPun commented Sep 8, 2022

Could we do the Code cleanup. in another PR?

PunkPun
PunkPun previously approved these changes Sep 9, 2022
PunkPun
PunkPun previously approved these changes Sep 12, 2022
mods/cnc/languages/lua/en.ftl Outdated Show resolved Hide resolved
mods/cnc/languages/lua/en.ftl Outdated Show resolved Hide resolved
mods/cnc/languages/lua/en.ftl Show resolved Hide resolved
mods/cnc/languages/lua/en.ftl Outdated Show resolved Hide resolved
@abcdefg30 abcdefg30 added this to the Next release milestone Oct 20, 2022
@Mailaender Mailaender changed the title Translated the Tiberian Dawn mission objectives Extracted the Tiberian Dawn mission objective strings Oct 21, 2022
@PunkPun PunkPun modified the milestones: Next release, Next + 1 Nov 6, 2022
Copy link
Member

@abcdefg30 abcdefg30 left a comment

Choose a reason for hiding this comment

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

Lgtm otherwise.

OpenRA.Mods.Common/Scripting/Global/UserInterfaceGlobal.cs Outdated Show resolved Hide resolved
@PunkPun PunkPun merged commit 621c850 into OpenRA:bleed Nov 17, 2022
@PunkPun
Copy link
Member

PunkPun commented Nov 17, 2022

Changelogs

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants