Re: [Freeciv-Dev] [PATCH] new city options: disband, make new citizens scientists/taxmen

[David Pfitzner <dwp@xxxxxxxxxxxxxx>]

Massimo Campostrini wrote:
> 
> Trent Piepho <xyzzy@xxxxxxxxxxxxxxxx> writes:
> 
> > Why not have tri-state command button?  Have it cycle between worker,
> > taxman, and entertainer.
> 
> Indeed, why not?  It's what freeciv does in most instances.
> Consistency in GUIs is a Good Thing.

Attached is a modified version of Massimo's patch,
which uses a tri-state toggle.  I also removed the
NEW_WORKER option which Massimo pointed out was not
needed, and changed the new option order so that this 
is still compatible with the previous "autoattack1"
and previous savefiles.

(Actually, I still not entirely sure how this should
be in terms of a consistent interface; should I have 
made it look like a command button instead of a toggle?.  
Or perhaps it should really be a drop-down list like 
the science advisor choices for goal etc?  Anyway, this 
is one way of doing it which isn't too bad...)

-- David

cityopt2_d1.diff.gz
Description: GNU Zip compressed data