The International Simutrans Forum

Development => Patches & Projects => Incorporated Patches and Solved Bug Reports => Topic started by: z9999 on January 19, 2009, 07:16:08 PM

Title: [patch] Marker window
Post by: z9999 on January 19, 2009, 07:16:08 PM
New marker window with player color.
This include some fixes which don't match with label_t well.

Fixed these bugs:
- Marker changed the owner of cityroad and city buildings
- text_farbe() didn't return the color of marker owner
- Couldn't delete marker on station

Changed:
- Color of title bar is player color
- Marker list buttons have the color of marker owner
- Clicking marker list buttons don't close marker window
- Changing player close marker window

[edit] updated patch for some old games

[edit2] removed this patch. please use new one.

[attachment deleted by admin]
Title: Re: [patch] Marker window
Post by: Frank on January 19, 2009, 07:28:56 PM
Marker list add List tools?
Title: Re: [patch] Marker window
Post by: z9999 on January 19, 2009, 07:41:41 PM
Quote from: Frank on January 19, 2009, 07:28:56 PM
Marker list add List tools?

Maybe, not.
This is still not a normal list window.
When you clck ok/remove/cancel button, window will close.

I don't know the safety way to update list.
If someone can do it, please do so.
Title: Re: [patch] Marker window
Post by: prissi on January 19, 2009, 10:09:58 PM
Very good work on this.

I agree to FrankP taht all marker should be listed as stops, convois, ... player specific. And adding marker should be a special tool, and removal with the standard removal tool, as many players. The current marker system is quite hackish.
Title: Re: [patch] Marker window
Post by: z9999 on January 21, 2009, 08:56:59 AM
All new.
This contains some bug fixes.

To build a new marker:
- Click old marker icon on toolbar
- Click land with this tool
- A new marker info window will open, input text and hit "enter/return" key

I used wkz_marker_t for this. So, old marker window don't open anymore, sorry.

To change marker text:
- Click your marker with inspection tool
- A marker info window will open, change text and hit "enter/return" key

Yor can't change the text of other players, any more.

To remove marker text:
- Click your marker with remover tool

Removing priority is the first.

New Marker list window: dialog_tool[25]

Order: Name, Koord, Player
Filter: Active player only, Show all players

Well, we often use a marker to notice something to other players.
So, I added this filter. Default is "Active player only".


Fixed these bugs:
- Marker changed the owner of cityroad, groundobj and city buildings
- text_farbe() didn't return the color of marker owner
- Couldn't delete marker on station
- Couldn't delete marker if other player's object was build on it


[attachment deleted by admin]
Title: Re: [patch] Marker window
Post by: prissi on January 21, 2009, 10:32:40 AM
Great, will test it and then include it.
Title: Re: [patch] Marker window
Post by: VS on January 21, 2009, 11:02:04 AM
After this improvement, I may use marker tool, too :)
Title: Re: [patch] Marker window
Post by: Frank on January 21, 2009, 01:49:32 PM
Thank you
Title: Re: [patch] Marker window
Post by: IgorEliezer on January 21, 2009, 03:02:26 PM
*claps*

Big improvement.

I liked to use markers when when I wanted to give names to districts and neighborhoods in metropolises. I'd like too see how it'll work out now.
Title: Re: [patch] Marker window
Post by: VS on January 23, 2009, 11:08:49 PM
How do I delete a marker?

Maybe a button on the info window?
Title: Re: [patch] Marker window
Post by: jamespetts on January 23, 2009, 11:53:02 PM
Quote from: IgorTekton on January 21, 2009, 03:02:26 PM
*claps*

Big improvement.

I liked to use markers when when I wanted to give names to districts and neighborhoods in metropolises. I'd like too see how it'll work out now.

Perhaps, for that function, use the public player?
Title: Re: [patch] Marker window
Post by: Combuijs on January 24, 2009, 01:07:11 AM
QuoteHow do I delete a marker?

QuoteTo remove marker text:
- Click your marker with remover tool

Seems quite logic to me...  ;)
Title: Re: [patch] Marker window
Post by: VS on January 24, 2009, 11:37:41 AM
I must learn to read again. That's the university, powerpoint everywhere... ;)

I agree that this is great improvement! Thanks!
Title: Re: [patch] Marker window
Post by: gerw on January 28, 2009, 07:40:45 PM
It would be nice, if the input box for text of marker directly gains the focus when placing a marker.