[patch] Marker window

z9999 · 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]

Frank · January 19, 2009, 07:28:56 PM

Marker list add List tools?

z9999 · 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.

prissi · 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.

z9999 · 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]

prissi · January 21, 2009, 10:32:40 AM

Great, will test it and then include it.

VS · January 21, 2009, 11:02:04 AM

After this improvement, I may use marker tool, too

Frank · January 21, 2009, 01:49:32 PM

Thank you

IgorEliezer · 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.

VS · January 23, 2009, 11:08:49 PM

How do I delete a marker?

Maybe a button on the info window?

jamespetts · 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?

Combuijs · 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...

VS · 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!

gerw · 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.

News:

[patch] Marker window