On Wed, 29 Jul 2015 18:23:02 +0200
win...@genial.ms wrote:
> - I'd like to move some bits and pieces of code between controllers
> and gui classes, to get a clearer separation of responsibilities.

With respect to the setActiveUnit untangling, I looked at git.b8c21dc
where the colony popup went away.  I think a case was missed, so I put it back,
and dropped as many setSelectedTile calls as possible (git.8394732).
Related to this I have just put in the fix for the second race in BR#2867,
which simplifies the InfoPanel in MOVE_UNITS_MODE (git.1728101).  This has
probably caused more cases of a blank InfoPanel, which will need fixing as
they are understood.  The intention now is that IGC should keep an active
unit displayed at all times, until it runs out of them or the user explicitly
goes into terrain mode (formerly we would clear the active unit in some
cases such as when abandoning a colony, however this was arbitrary).

Cheers,
Mike Pope

Attachment: pgpjRsrDT41q7.pgp
Description: OpenPGP digital signature

------------------------------------------------------------------------------
_______________________________________________
Freecol-developers mailing list
Freecol-developers@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/freecol-developers

Reply via email to