[Freeciv-Dev] Re: (PR#12041) CMA / preset tab

["Christian Knoke" <chrisk@xxxxxxxxx>]

<URL: http://bugs.freeciv.org/Ticket/Display.html?id=12041 >

On Wed, Feb 02, 2005 at 03:51:22PM -0800, Vasco Alexandre da Silva Costa wrote:
> > 
> > This works ok. But I have still some issues, sorry.
> > 
> > The factor sliders are broken in S2 CVS + Patch.
> 
> How can I reproduce this issue?

Apply your patch to current S2_0 CVS, I'd say ;) 

Sorry, I don't know. PNG attached.
I have Debian Sarge here.

> > The help button is between the 'new preset' and the 'delete preset'
> > button (which should be positioned next to next). Better put it next to
> > the 'enable' button.
> 
> See how it looks with this patch.
> 
> > We've discussed already that the results are not tested/approved as long
> > as the CMA is not enabled. Now, in the top right result table, you can
> > see *no* difference whether there are valid results or just numbers.
> > This can be confusing. Maybe a simple solution is to grey out that
> > table, as long as the CMA is disabled?
> 
> This patch also does that.

Both look good. Now this is really an improvement, thanks.

Christian


-- 
Christian Knoke            * * *            http://cknoke.de
* * * * * * * * *  Ceterum censeo Microsoft esse dividendum.