[Freeciv-Dev] Re: [Patch] Add buy info

[Mike Kaufman <kaufman@xxxxxxxxxxxxxxxxxxxxxx>]

On Sat, Feb 16, 2002 at 06:38:28PM +0100, Christian Knoke wrote:
> On Sat, Feb 16, 2002 at 10:58:43AM -0600, Mike Kaufman wrote:
> > On Sat, Feb 16, 2002 at 04:31:53PM +0100, Raimar Falke wrote:
> > > On Sat, Feb 16, 2002 at 01:22:56PM +0100, Christian Knoke wrote:
> > > 
> > > > Besides this - to be honest - I'm not happy with both of them.
> > > > There should be a popup which asks you whether you really want
> > > > to buy / sell IMHO, like it is now. This is to have a safeguard
> > > > and because the user expects it to work this way.
> > 
> > I agree, but my version does make you click twice. Is this not safeguard
> > enough?
> 
> Hhm, your version is somewhat cool, but I fear the average user
> will stumble about it, because it is very unusual. If you make
> it this way, it should be more striking, maybe with symbols like
> a red cross or a small green hook (?) or so.
                              ^^^^^
                             check[mark]

yes it was cool (and not just somewhat), but sure to irritate vasco since 
it added more doodahs.

the new version attached is much simpler, keeps the popups, and
dispenses with an expensive widget, namely the progress bar.

-mike

> 
> > > > So these patches are about having the price for the unit / improvement
> > > > displayed? What about this suggestion: Put the price there where
> > > > the progess bar is now, i.e. remove the bar. I don't like this
> > > > thing since long, because you neither can see clear the scrollbar
> > > > nor the numbers at the same place. Then you can put it like this:
> > > > 
> > > > Musketeers
> > > > 4/30  9 turns   price 85  [buy] [change]
> > > > 
> > > > Deactivate the buy button if you can't afford it and deactivate
> > > > the change button if you've already bought this turn.
> > > > This solution also gives more space for the improvements list.
> > > 
> > > Looks ok. How implements it?
> > 
> > I agree this is probably the most elegant solution. 
> > I'll implement this today or tomorrow.
> > 
> > -mike
> 
> Christian
> 
> -- 
> http://www.enter.de/~c.knoke/  * * * * * * * * * * * * * * *
> * * * * * * * * *  Ceterum censeo Microsoft esse dividendum.

buy_info3.diff
Description: Text document