[Freeciv-Dev] Re: [Patch] New city dialog (8)

[Mike Kaufman <mkaufman@xxxxxxxxxxxxxx>]

On Mon, Sep 03, 2001 at 11:38:11PM +0200, Christian Knoke wrote:
> A must have are shortcuts for the buttons.
> 
> And then something I have already written:
> 
> --------
> Keys: "Ins" and "Del" for switching the panes is not very
> intiutive. Could you change this to shift-cursor or ctrl-cursor?
> "Ins" and "del" then could be used for inserting / deleting items
> in the worklist. You'd have full keyboard control then.

I assume that this means right and left arrows? I cringe about doing this since 
a lot of window managers use those as defaults to switch desktops. Maybe that's 
not what you meant.

I too have some issues with the worklist dialog.

> On the overview page, you can't see whether you have a worklist.
> So, if you change production, you will erase any worklist accidentally.
> A worklist indicator would be nice. I'm thinking of a small thing,
> maybe "WL" behind the "Currently building: .." line.

I'm not sure how that's helpful, since the first item in the worklist is what 
you're currently building anyway, and changing production will just change that 
item, not the rest of the worklist. also, since it's far easier now to check on 
the status of a city's worklist (instead of popping up a new window)...

> --------
> 
> .. or simply: "Currently building: Legion (and more)"

this might be acceptable, anyone else have a comment about this? Currently, I 
don't think that I'll do anything about this unless anyone else speaks up, or 
you can rebut the arguments I made above.

> At last a suggestion for a redesign:
> 
> Exchange "supported units" and "established trade routes". 
> Reason: That is the way it is now. It is easier to compare
> both rows of units, looking for supported and not present e.g.
> It is the same order as on the units page. It is irritating
> now, when you switch to the units page.
> 
> As a consequence of this move, remove the last line of the Trade routes
> "No trade routes exist". Instead, write a line "Trade routes: +7"

This is my first thought as well, _but_ I do _not_ want to sacrifice the 'city 
improvement' window's screen real estate to do it. An alternative would be to 
move Trade Routes dialog off the overview page. Then we'd have plenty of room.
Any comment about that? Where to move it to? It's own page? It'll be lonely...

> (e.g.) into the topleft display. And, shouldn't there be a line "Corruption"
> in that display?

could be... ;)