Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Integrate 1.60 marker shape color map interface #3860

Merged
merged 1 commit into from
Jun 2, 2016
Merged

Conversation

PabstMirror
Copy link
Contributor

1.60 adds ability to set color and marker type from the map interface (top right corner)
Luckily they use the same values and indexes as ACE 😉

This PR connects the two systems:
Selecting a color from the 1.60 map display will set the color in the
ace marker display and vice versa

1.60 adds ability to set color and marker type from the map interface
Luckily they use the same values and indexes as ACE,
This PR connects the two systems,
Selecting a color from the 1.60 map display will set the color in the
ace marker display and vice versa
@PabstMirror PabstMirror added the kind/enhancement Release Notes: **IMPROVED:** label Jun 2, 2016
@PabstMirror PabstMirror added this to the 3.6.0 milestone Jun 2, 2016
@commy2
Copy link
Contributor

commy2 commented Jun 2, 2016

Nice

@jonpas
Copy link
Member

jonpas commented Jun 2, 2016

Awesome!

Is config displayLoad preferred over scripted addDisplayEventHandler?

@commy2
Copy link
Contributor

commy2 commented Jun 2, 2016

addDisplayHandler only works for the missionDisplay (46). This is the diary display.

@jonpas
Copy link
Member

jonpas commented Jun 2, 2016

Ah I see.

@PabstMirror PabstMirror merged commit b6aaba7 into master Jun 2, 2016
@PabstMirror PabstMirror deleted the markers160 branch June 2, 2016 14:28
@nicolasbadano
Copy link
Contributor

Is it me or BIS totally ripped out our idea?

@jonpas
Copy link
Member

jonpas commented Jun 3, 2016

They totally did.

@bux
Copy link
Member

bux commented Jun 4, 2016

That's a good thing, imo :)

@commy2
Copy link
Contributor

commy2 commented Jun 4, 2016

Not the first time.
I like it.

@jonpas
Copy link
Member

jonpas commented Jun 4, 2016

👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/enhancement Release Notes: **IMPROVED:**
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants