On Fri, 7 Aug 2015 07:34:43 +0930
"Michael T. Pope" <mp...@computer.org> wrote:
> > I think we have several methods already, which could provide that
> > functionality. Maybe setFocus, setFocusImmediately, centerActiveUnit,
> > isTileVisible, onScreen or some others in MapViewer would help?
> > But the call should be routed through SwingGUI and there are some
> > helper methods already.
> 
> OK, I missed centerActiveUnit, that sounds promising. 

centerActiveUnit is too strong, but looking deeper I think it might be ok
to just get rid of the setSelectedTile() altogether.  IR#115 called for
animation-off to just skip movement display, which I think makes sense and
provides another useful work around for annoying broken graphics drivers
that cause animation to become glacial, and FreeCol to be blamed.  So we
then only need worry about whether a tile is on screen when animation is
occurring, which means the decision to refocus during movement can be
handled entirely by the animation routines and not by IGC at all.  I have
implemented IR#115, and am play testing dropping setSelectedTile from
moveMove.

Meanwhile we seem to have broken the initial unit selection after loading
a game.  I do not know why, but I know where that should be
happening.

Cheers,
Mike Pope

Attachment: pgpwLv_7hPVkL.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