[Done] Split "ExitGame?" string

[GunChleoc]

The string ExitGame? is used in two places

• The game menu
• A feedback question (with the meaning of: are you sure... ?)

The question mark in the menu looks ugly in English, and it's creating problems for the translators. Could we have two separate strings for this please?

I suggest

• Exit Game for the menu, and
• Exit Game? or Are you sure you want to leave the game? for the feedback question.

Related: 3.7.0 beta1 translation + usability - Game menu - Exit game?

[softcoder]

Updated as requested in svn rev#: 4481

ExitGame?=Would you like to close the game?
ExitGameMenu?=Exit current game?
ExitGameServer?=Exit current game? (this will end the game for all other connected players)

[GunChleoc]

Thanks!

[GunChleoc]

I have just tested this with the latest beta from the automatic Windows builds, and it does not work. I am presented with ExitGameMenu? both for the menu entry and the question in the prompt dialog.

[tomreyn]

Hmm, let's see...

Current (r4510) English language file in the subversion repository.
(The latest revision is always available here.)

Current (r4510) Gaelic language file in the subversion repository.
(The latest revision is always available here.)

According to the CI subversion repository polling log the latest Windows build is based on r4518, was built 3 days ago, and the build succeeded.

In r4518, these keys are referenced in the following game engine files:

File	Line no.	Code
source/glest_game/game/game.cpp	1648	menuItems.push_back(lang.get("ExitGameMenu?"));
source/glest_game/game/game.cpp	3539	showMessageBox(Lang::getInstance().get("ExitGameMenu?"), "", true);
source/glest_game/game/game.cpp	5814	showMessageBox(lang.get("YouLose")+" "+lang.get("ExitGameServer?"), lang.get("BattleOver"), false);
source/glest_game/game/game.cpp	5817	showMessageBox(lang.get("YouLose")+" "+lang.get("ExitGameMenu?"), lang.get("BattleOver"), false);
source/glest_game/game/game.cpp	5825	showMessageBox(lang.get("GameOver")+" "+lang.get("ExitGameMenu?"), lang.get("BattleOver"), false);
source/glest_game/game/game.cpp	5828	showMessageBox(lang.get("YouWin")+" "+lang.get("ExitGameMenu?"), lang.get("BattleOver"), false);
source/glest_game/menu/menu_state_root.cpp	322	showMessageBox(lang.get("ExitGame?"), "", true);
source/glest_game/main/battle_end.cpp	770	showMessageBox(lang.get("ExitGameMenu?"), "", true);

Of course, to get the latest translations you also need to use the latest game data archive (to get what's in the subversion repository) or use the games' option menu to pull the latest translations from Transifex.

Now what do you think should be changed?

Update: I just submitted a minimal change (r4519, which is only comprised of two changes to the french main language file) which will trigger the data package to be rebuild, but you can safely ignore this.

[GunChleoc]

I did pull the latest translation from Transifex.  It loaded fine, because the new "Show next hint" button is there. I then changed the ExitGameMenu? string in Transifex, reloaded the translation and it promptly changed in both locations. So, it's definitely the same variable.

I thik the culprit is this one:

File	Line no.	Code
source/glest_game/game/game.cpp	1648	menuItems.push_back(lang.get("ExitGameMenu?"));

which should be ExitGame? to separate the strings correctly.

We then still have reversed semantics. i.e. replace all ExitGame? (a question) with ExitGameMenu? (a button text) and all ExitGameMenu? with ExitGame?

ExitGameMenu? where it sould be a menu entry


ExitGameMenu? where it should be a question. It also appears in the "You Won. Exit Game?" question.

[tomreyn]

With the change you suggest it would look like this:



(while the follow-up screen is unmodified).

Hmm, I don't think that's an improvement. Maybe that's not what you wanted to do?

I don't speak Gaelic, so it's difficult to understand what your goal is. If what you are discussing is not primarily an issue with Gaelic translations but one which also affects the source language (English), then it would be best to refer to this.

[GunChleoc]

After changing this, they then are all the wrong way around, in all languages.

We then still have reversed semantics. i.e. replace all ExitGame? (a question) with ExitGameMenu? (a button text) and all ExitGameMenu? with ExitGame?

So, we have 2 problems:

1. Two variables that share the same string and that shouldn't.

2. Two translations being generally assigned the wrong way around.

Will do the table in the next post

[GunChleoc]

 

File	Line no.	Code
source/glest_game/game/game.cpp	1648	menuItems.push_back(lang.get("ExitGameMenu?"));
source/glest_game/game/game.cpp	3539	showMessageBox(Lang::getInstance().get("ExitGame?"), "", true);
source/glest_game/game/game.cpp	5814	showMessageBox(lang.get("YouLose")+" "+lang.get("ExitGameServer?"), lang.get("BattleOver"), false);
source/glest_game/game/game.cpp	5817	showMessageBox(lang.get("YouLose")+" "+lang.get("ExitGame?"), lang.get("BattleOver"), false);
source/glest_game/game/game.cpp	5825	showMessageBox(lang.get("GameOver")+" "+lang.get("ExitGame?"), lang.get("BattleOver"), false);
source/glest_game/game/game.cpp	5828	showMessageBox(lang.get("YouWin")+" "+lang.get("ExitGame?"), lang.get("BattleOver"), false);
source/glest_game/menu/menu_state_root.cpp	322	showMessageBox(lang.get("ExitGameMenu?"), "", true);
source/glest_game/main/battle_end.cpp	770	showMessageBox(lang.get("ExitGame?"), "", true);