[Freeciv-Dev] Re: [Patch] New city dialog - new patch

[Christian Knoke <ChrisK@xxxxxxxx>]

Am Donnerstag, 30. August 2001 12:30 schrieb Daniel Sjölie:
>
> Patch:
> http://www.acc.umu.se/~deepone/freeciv/new_city_dialog3.diff.gz
>
> Comments still very welcome...

Ok, here is some feedback (promising more):

First, it's getting better!

It's not too big any more.

What about changing the traderoute display?
You now have:

---------------------------------
Esthablished trade routes
City names     |   Trade/Year
There are no traderoutes
. . .
. . .
---------------------------------

Display only a non-structured area (white?)
like this:

--------------------------------------
Traderoute to Berlin gives 1 trade(s)
Traderoute to Bonn gives 3 trade(s)
---------------------------------------

thus reflecting that the user can't change it
in the dialog, plus it's using less space.
simpler=clearer

exchange "close dialog" and "next city" button. Make them
different/smaller.

within the present units area, the '>' works wrong
(should be disabled with 1 unit, but isn't)

Next / Prev city: In which order are the cities presented?
I suggest alphabetical order, or island for island, and alphabetical
subordered.

How is the senty/fortify button supposed to work? For me,
it sentries a fortified musketeer, but never fortifies it.

In the worklist area, there should be an append button.
It is not obviously, that insert works different, when
no entry is marked.

I have to test the worklist thing, it seems not working
correct.

In the worklist, the right help button doesn't work.
Maybe the reason is long text in german locale (not tested).

Christian

-- 
* Christian Knoke                           +49 4852 92248 *
* D-25541 Brunsbuettel                  Wurtleutetweute 49 *
* * * * * * * * *  Ceterum censeo Microsoft esse dividendum.