Complete.Org: Mailing Lists: Archives: freeciv-dev: February 2002:
[Freeciv-Dev] Re: [Patch] Add buy info
Home

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

[Top] [All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
To: freeciv-dev@xxxxxxxxxxx
Subject: [Freeciv-Dev] Re: [Patch] Add buy info
From: Christian Knoke <ChrisK@xxxxxxxx>
Date: Sat, 16 Feb 2002 13:22:56 +0100

On Fri, Feb 15, 2002 at 11:31:10PM -0600, Mike Kaufman wrote:
> On Thu, Feb 14, 2002 at 06:43:56PM +0100, Raimar Falke wrote:
> > 
> > Attached is a patch which adds a buy info in the city dialog. It also
> > disables the buy button.
> 
> I don't like this patch mainly because you can buy something just by
> clicking too low and have no safeguard against it (It's also kinda ugly).
> 
> Attached is a patch which fixes those problems and reimplements the sell 
> button 
> as well.

Problem with Raimar's patch: you can't sell anything after
you have bought something.

Problem with Mike's patch: Gold is not updated when I sell
a building - neither in this or in another city.

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.

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.

Just my cents.

Christian

-- 
http://www.enter.de/~c.knoke/  * * * * * * * * * * * * * * *
* * * * * * * * *  Ceterum censeo Microsoft esse dividendum.


[Prev in Thread] Current Thread [Next in Thread]