[Freeciv-Dev] Re: (PR#3712) City choose production dialog

["Raimar Falke" <i-freeciv-lists@xxxxxxxxxxxxx>]

<URL: http://rt.freeciv.org/Ticket/Display.html?id=3712 >

On Thu, Apr 01, 2004 at 01:24:55PM -0800, Vasco Alexandre da Silva Costa wrote:
> 
> <URL: http://rt.freeciv.org/Ticket/Display.html?id=3712 >
> 
> > [guest - Wed Mar 12 20:55:01 2003]:
> > 
> > I was recently talkingto a comparative freeciv newbie who complained the
> > choose production dialog gave too few hints about what the things were.
> >  She suggested color coding.  Here is my code:
> > 
> > Units{
> >  Land: green
> >  Sea: blue
> >  Air/Heli: red
> > }
> > Buildings{
> >  redundand: grey
> >  current: black
> > }
> > Wonders{
> >  obsolete: grey
> >  current: black
> > }
> > 
> > The patch is appended, it's very simple.  I can't think of much to add
> > to it.
> 
> Even better, add icons instead. It would still be nice to make redundant
> buildings and obsolete wonders be displayed differently, say, using an
> italic font. This patch adds icon support.

While I didn't test Daniel's patch I think that the list would be
faster to grasp if line's background is changed. But not to the full
colors like COLOR_STD_BLUE but some light versions of them.

Bug: I get a striked through diplomat. I think you missed a "!is_unit"
in the second half of cell_render_func.

A bad thing: The list uses a lot of space. You almost always have to
scroll.

        Raimar

-- 
 email: rf13@xxxxxxxxxxxxxxxxx
 "On the eigth day, God started debugging"