We should have a way to indicate the chosen AI level without having to open the AI-config panel which is quite cumbersome when lot of Ais.
A possibility would be to change the background of the gear icon depending on the current AI level (if somebody has nice proposition about it), but here i've used a more basic way by adding the level in the AI_config tooltip such that we can know the level by only hovering the mouse over the gear icon.
Details
Check if you agree with the changes
Diff Detail
- Repository
- rP 0 A.D. Public Repository
- Lint
Automatic diff as part of commit; lint not applicable. - Unit
Automatic diff as part of commit; unit tests not applicable.
Event Timeline
Shouldn't we also mention the ai name? (as that is choosable too) so we get something like Configure AI settings (current: {name}, {level})
Broken translation.
(Ah right, period at the end of tooltips, Polakrity wanted those)
I would also add the Petra name, even if we only have one AI currently.
(We do that in the lobby and replay menu (gamedescription.js) too for instance.)
You likely want these:
translateAIName(playerData.AI)
translateAIDifficulty(playerData.AIDiff)
Maybe "Configure AI settings (Very Hard Petra AI)."?
Successful build - Chance fights ever on the side of the prudent.
Updating workspaces... Build (release)... Build (debug)... Running release tests... Running cxxtest tests (308 tests)....................................................................................................................................................................................................................................................................................................................OK! Running debug tests... Running cxxtest tests (308 tests)....................................................................................................................................................................................................................................................................................................................OK! Checking XML files...
Agreed about the translation (i forgot it), but is the name really needed? it is already displayed just on the left of the gear icon that you have to hover. I would rather not overcrowd this tooltip and keep room to add additional info if we ever have it (like ai agressivity and so on).
Oh, you are right.
Perhaps we could also add the difficulty to the tooltip of the dropdown. (Adding it to the label won't work for 1024x768, otherwise I would have suggested that).
If there is going to be more than one AI property, then adding these with newlines shouldn't be a problem.
In fact, as we don't have much to display in that tooltip, i've put the name in it. We can always remove it later when we have more properties to display.
And translation is fixed.
We currently don't have any tooltip for it. I'd be fine with adding the same tooltip for the dropdown, but i'm afraid it must be misleading as the dropdown does not allow to change the difficulty.
If there is going to be more than one AI property, then adding these with newlines shouldn't be a problem.
The way this tooltip is displayed only allows for 2 lines i think (but could be changed when we will need it)
Successful build - Chance fights ever on the side of the prudent.
Updating workspaces... Build (release)... Build (debug)... Running release tests... Running cxxtest tests (308 tests)....................................................................................................................................................................................................................................................................................................................OK! Running debug tests... Running cxxtest tests (308 tests)....................................................................................................................................................................................................................................................................................................................OK! Checking XML files...
- Better end the period too for consistency before Polakrity notices ;-) Oh thats not committed yet D594.
- Should we use the same order used in replay menu, load game dialog and lobby? (That is Very Hard Petra Bot, all loaded from gamedescription.js)
- Not fully sure about "Current:".
- Perhaps level -> difficulty.
That'd be Configure AI: %(difficulty)s %(name)s.
Your choice.
binaries/data/mods/public/gui/gamesetup/gamesetup.js | ||
---|---|---|
961 ↗ | (On Diff #4387) | -1 tab for the three lines judging from phabriator |
I'd keep the current way (with a separation as "Petra Bot - Very Hard") otherwise it may look strange in some languages as french where "Moyen Petra Bot" is weird.
- Not fully sure about "Current:".
- Perhaps level -> difficulty.
ok
That'd be Configure AI: %(difficulty)s %(name)s.
Your choice.
For that reason we have the sprintf, in english it can be 1 2 and in french 2 1 and somewhere else foo 1 bar 2
ps/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | ||
---|---|---|
953 | We could rename playerConfig to aiConfig to better describe what that button is about. |