[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 Fri, Apr 02, 2004 at 12:14:35PM -0800, Vasco Alexandre da Silva Costa wrote:
> 
> <URL: http://rt.freeciv.org/Ticket/Display.html?id=3712 >
> 
> On Fri, 2 Apr 2004, Raimar Falke wrote:
> 
> > <URL: http://rt.freeciv.org/Ticket/Display.html?id=3712 >
> >
> > > 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.
> 
> For one, the color change does not tell you as much information as the
> icon.

You also have now icon and color. What not just color the whole line?

> For another, colors are always a bad idea because you may have
> clashes with GUI themes.

Is this really a valid point if you set the background on every line?

> > 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.
> 
> Would making the icons optional be a satisfactory solution?

Maybe a "cram" option which will also make a font a bit smaller?!

        Raimar

-- 
 email: rf13@xxxxxxxxxxxxxxxxx
  This message has been ROT-13 encrypted twice for extra security.