- mouseover on an unit displayed its info in the sidebar but the
status icons disappeared when the mouse leaved the unit.
- stop using the invisible icon about the hex under the mouse (was not
obvious that it was not the current status, see bug #9288, and broke
tooltip use)
PS: I will commit later a new visual clue to replace this mouseover feature.
(was inconsistent with other numbers and all moves need "1 turn")
Instead, use "1" only for multiple turns moves and introduce little
icons at the same place to show ZoC and capture end-of-move.
As always, these images are working placeholders (but not too ugly, I
think) Note that icons and numbers must stay visible when mixed (but
it's rare)
* if animations are jumpy, this should be the first suspect
* slowing down on hard to cross tiles has been removed as discussed on
IRC
* i'm not satisfied with that completely yet, expect some more work
here...
This meet a frequent FR for displaying only simple tips for beginners
and later show more advanced ones. The key accept number (at least N
units) and list of id (at least one of them) and you can mix it
("Troll,30"), using a OR base.
This allow to show very specific tip (e.g. teleport) only after having
encountered units with this abilities. You can also use
campaign-specific units to detect if already played and so maybe make
reference about it.
The idea is to better show the end-of-turn rules to beginners (or special case)
- Use red color for end of turn caused by ZoC
- Use bracketed number for end of turn caused by village capture
- Remove the old parentheses (a relic from the "all in one line" times)
- Also display turn to reach > 9, we have enough space now
- Don't use "1" just because no MP left (on rough terrains, you can
have only 1 MP left, so be forced to end your turn there and a missing
"1" is misleading) PS: we can also display this by using other clues
(images, etc..), but like this it's stay simple (more info with only
one number)
No regression expected. if anything changes, please tell me.
Note that:
* Fire and forget doesn't work yet, don't try using it
* I will probably change how mvt anims work as discussed in IRC, but
that's for a future commit