Topic: Uncluttering soldier stats.
Nordfriese |
Posted at: 2017-06-01, 18:48
Which is exactly what I meant.
Agreed. Would the training levels for a partly-trained soldier show up for all categories or only for those he has trained in? Top Quote |
GunChleoc Topic Opener |
Posted at: 2017-06-01, 21:54
For all categories. We'd end up with weird tiny floating squares otherwise. Busy indexing nil values Top Quote |
kaputtnik |
Posted at: 2017-06-01, 22:16
From my point of view i believe the bug and the proposed solutions are invalid/not necessary. Showing or not showing one thing (healthbar, training status) do not solve any problems (if there are any ). The image shown in the bug report is annoying though: In this situation none of the proposed solutions will work. Mostly you will start an attack if you have promoted soldiers, so the last proposal (showing healthbar always and promotions only if a soldier has one) will not change this view. Maybe having a keybord shortcut for displaying soldier stats will be helpfull, but i don't know if this is worth the work, since no one since year 2011 has complained about that. Fight simulator for Widelands: |
GunChleoc Topic Opener |
Posted at: 2017-06-01, 22:36
Screenshots with the current state of affairs (auto-hiding of health bar and training levels). Note that I also colored the outer edge of the health bar a bit, this makes it easier to see the player color when soldiers have very little health left. The hotkeys are already implemented in the experimental branch, so if we go for them, they wouldn't create more work, apart from the code review, of course. Busy indexing nil values Top Quote |
WorldSavior |
Posted at: 2017-06-02, 17:00
I think you're right. And the picture is very funny
That's good. The yellow looks best, because the colours are identical. Wanted to save the world, then I got widetracked Top Quote |
GunChleoc Topic Opener |
Posted at: 2017-06-04, 09:06
I tried identical colors with blue as well, but it looked horrible OK, so the auto-hide is a -1 and I'll remove it. Do we want the hotkeys? Busy indexing nil values Top Quote |
kaputtnik |
Posted at: 2017-06-04, 10:31
When the promotions aren't shown, i have trouble to distinguish a soldier from other workers (except the second carrier ). E.g. in your first picture there is a soldier walking on left road (near the fox) which i didn't noticed on a quick view. Same goes for some of the soldiers on right (waiting for battle) in your second picture. I think having one hotkey for all is not bad. E.g. pressing "L" hide/shows the promotion and the health bar. Presetting should then be "show", imho. Fight simulator for Widelands: |
GunChleoc Topic Opener |
Posted at: 2017-06-07, 10:19
I have updated the branch now. So, the changes from trunk are:
I also updated the building statistics for missing workers: If all missing workers are on their way, it now says "Waiting for worker"/"Waiting for workers" in green. So, red "Worker missing/"Workers missing" now only shows up if a worker isn't available and the player needs to take action. Busy indexing nil values Top Quote |