The International Simutrans Forum

Development => Patches & Projects => Considered Patches => Topic started by: smu42 on March 16, 2009, 10:13:55 PM

Title: [patch] factoryeditor gui: producer/consumer dependencies
Post by: smu42 on March 16, 2009, 10:13:55 PM
Hey folks,

I wrote a small patch which is IMO especially helpful for beginners.
It shows for a given factory f:
- A list of factories that produce the goods which are needed by f.
- A list of factories that consume the goods which are produced by f.

I have tested this patch with ubuntu linux with pak64: no known problems.

I'd like to make these lists clickable: If you click on a factory, it should be selected in the list on the left of the window.
BUT: The interface could be better in general. Maybe s.o. can help me out here.

There are two new translations ("factories producing " & "factories needing "), but I don't know were to add them (pak dependent?).
Is there a way to translate sentences? "factories producing X" would be "Fabriken, die X produzieren" in German.

smu42

[Edit: fixed tabs/spaces in patch]
Title: Re: [patch] factoryeditor gui: producer/consumer dependencies
Post by: VS on March 16, 2009, 10:55:08 PM
Hi! Welcome to the forum.

As to the patch... Sorry, i am not very active programmer at the moment. The improvements look nice. As you say, interface could be better ;) Apart from clickable links, I think it might be good to allow hiding these texts (show on/off).

A side note - you use spaces for indenting, while the original code uses tabs. That is source of some changes listed in diff, while nothing changed on these lines.

Translations have to be added separately. Let's leave that for when it is decided to add this?
Title: Re: [patch] factoryeditor gui: producer/consumer dependencies
Post by: z9999 on March 17, 2009, 04:45:04 AM
My first version of map ediotr have had this.
But prissi removed this when merged in simutrans.  :(
http://archive.forum.simutrans.com/topic/03405.0/index.html#msg60581
Title: Re: [patch] factoryeditor gui: producer/consumer dependencies
Post by: smu42 on March 17, 2009, 11:19:46 AM
Well, thats odd. But I think the main reason is the "bad interface" but not the idea. (Although z9999s gui was better than mine)

Prissi, if you don't like it this way, could you give me some hints? I'm not very good at designing interfaces, but I can certainly do the programming.

Ideas:
1) Hide/Show button. (simple implementation)
2) No texts. Add a filter filtering all factories which are supplier or consumer of f and show them in the window on the left. (a bit more complicated)
3) [insert your idea here]
Title: Re: [patch] factoryeditor gui: producer/consumer dependencies
Post by: prissi on March 23, 2009, 02:15:54 PM
Not sure, need to look more at the code. I certainly want to have the interface as less cluttered as possible. The information what factories can deliver is imho not most urgent, as generally the editor is only to be used for building scenarios and not during real games, as it will mess up the industry balance. This was the reason I left it out of the code. But zou are under considerated thus there is hope.

Chanching UI is time coonsuming, as those strings need to be put into svn and so on ...
Title: Re: [patch] factoryeditor gui: producer/consumer dependencies
Post by: smu42 on March 23, 2009, 08:19:56 PM
Quote from: prissi on March 23, 2009, 02:15:54 PM
Not sure, need to look more at the code.

I read that as "I will contact you, when I have detailed information"


Quote from: prissi on March 23, 2009, 02:15:54 PM
I certainly want to have the interface as less cluttered as possible.

I agree. IMO the On/Off button is the best solution in that case.


Quote from: prissi on March 23, 2009, 02:15:54 PM
The information what factories can deliver is imho not most urgent, as generally the editor is only to be used for building scenarios and not during real games, as it will mess up the industry balance.

I disagree. Yes, the info is not "most urgent" but very helpful.
There are also users that place each factory with this editor (I do this at least from time to time). It's just another way of playing simutrans.


Quote from: prissi on March 23, 2009, 02:15:54 PM
But zou are under considerated thus there is hope.

:D


Quote from: prissi on March 23, 2009, 02:15:54 PM
Chanching UI is time coonsuming, as those strings need to be put into svn and so on ...

If I would have infos about this, I would definitely help, but I haven't.

Yes, I do want this "feature" in simutrans :)

We could discuss the details in German, if you prefer.
Title: Re: [patch] factoryeditor gui: producer/consumer dependencies
Post by: prissi on June 02, 2009, 09:14:15 PM
Did someone save the patch?
Title: Re: [patch] factoryeditor gui: producer/consumer dependencies
Post by: smu42 on June 04, 2009, 08:57:10 PM

Here it is again.

http://simutrans-germany.com/~patches/upload/factoryeditorgui.patch
Title: Re: [patch] factoryeditor gui: producer/consumer dependencies
Post by: IgorEliezer on June 04, 2009, 08:59:56 PM
Quote from: smu42 on June 04, 2009, 08:57:10 PM
Here it is again.

Thanks. I hope Prissi finds it.
Title: Re: [patch] factoryeditor gui: producer/consumer dependencies
Post by: prissi on September 06, 2009, 06:38:41 AM
Patch seems gone again ...
Title: Re: [patch] factoryeditor gui: producer/consumer dependencies
Post by: Frank on September 06, 2009, 09:13:44 AM
Quote from: prissi on September 06, 2009, 06:38:41 AM
Patch seems gone again ...

save time by patches.simutrans-germany.com 30 days

EDIT changed to 90 days
Title: Re: [patch] factoryeditor gui: producer/consumer dependencies
Post by: VS on September 06, 2009, 09:15:54 AM
I don't have this one saved :( But smu42 is active reader, so there is hope :)
Title: Re: [patch] factoryeditor gui: producer/consumer dependencies
Post by: smu42 on September 09, 2009, 06:37:50 PM
http://simutrans-germany.com/~patches/upload/patch_factorybuildergui_1.1.patch (http://simutrans-germany.com/~patches/upload/patch_factorybuildergui_1.1.patch)
http://87.230.77.206/svenu/simutrans/patch_factorybuildergui_1.1.patch (http://87.230.77.206/svenu/simutrans/patch_factorybuildergui_1.1.patch)

I'm sorry it took so long.