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

Modify cover_map module to allow multiple uncovered sections of the map. #149

Open
Drofseh opened this issue Nov 18, 2017 · 10 comments
Open

Comments

@Drofseh
Copy link

Drofseh commented Nov 18, 2017

I'm making a mission in which I have a general AO for the mission, but I'd also like to have an aircraft use a more distant airfield.

With only a single marker controlling the location of the map cover I would have to either make a much larger percentage of the map be visible with a single marker or have the airfield covered by the cover_map module and make navigating to and from it much more difficult for the aircraft.

Here is an example of what I would like to be able to do:
Both the small island of Moray and the large AO on southern Malden would be uncovered.
Currently this is impossible.
Image

@PiZZAD0X
Copy link
Contributor

We can call the covermap and centermap functions on the fly, so it should be possible to have a self interact option to "switch maps" and automatically switch the covermap area and what the player is zoomed/centered on in the map, effectively acting like two separate maps.

@Drofseh
Copy link
Author

Drofseh commented Mar 30, 2018

effectively acting like two separate maps

That would be neat.

@PiZZAD0X
Copy link
Contributor

PiZZAD0X/Olsen-Framework-Arma-3@01855cc

MultipleAOCoverMapTest.WL_Route191.zip

@Drofseh
Copy link
Author

Drofseh commented Mar 30, 2018

@PiZZAD0X
I've been testing it out and I'm very impressed, it works really well!

A couple thing I have noticed though:

  1. The first time in open the map it immediately closes again.
  2. It would be nice if the maps were under their own sub-interaction point instead just under Equipment.
    So you would go Equipment -> Maps -> Map_Name

@PiZZAD0X
Copy link
Contributor

Are you testing it with a briefing in singleplayer or without briefing?

I can add that submenu, sure.

@Drofseh
Copy link
Author

Drofseh commented Mar 31, 2018

Single Player - M key opens map, map stays open.
Single Player w/Briefing - M key opens map, map stays open.
Locally Hosted Multiplayer - M key opens map, map closes on its own, press M key again, map behaves as normal for the rest of the mission.
https://www.youtube.com/watch?v=PtOS3uWsiVA

Would it be possible to add a maximum zoom limit per map as well?
That way I could force a less detailed AO map and also give small maps for towns that show more detail.

@PiZZAD0X
Copy link
Contributor

Hmm so it seems map needs to be fully initialized and open before any animation can be committed to it, meaning the map needs to be on screen, however brief, to get that initial centering. Maybe a blackout loading screen or getting the timing really tight so it isn't as noticeable.

I've put the interaction options under a "Switch Maps" submenu under equipment.

As for limiting map zoom, we use a separate addon that enforces a script for that. I'm not sure how modular that addon is and if it allows for scripts to run contrary to it (more detail than what the addon allows) but it is a good idea.

MultipleAOCoverMapTest.WL_Route191.zip

@Drofseh
Copy link
Author

Drofseh commented Mar 31, 2018

Thanks I'll test out this version tomorrow.

As for limiting map zoom, we use a separate addon that enforces a script for that. I'm not sure how modular that addon is and if it allows for scripts to run contrary to it (more detail than what the addon allows) but it is a good idea.

My unit doesn't use an addon or any script to limit map zoom
I think having it as an option for this and setting the default to false should be fine for you though.

@Drofseh
Copy link
Author

Drofseh commented Apr 4, 2018

Alright, I've played around with the updated version and it seems solid to me.

@TrainDoctor
Copy link
Contributor

TrainDoctor commented Jan 3, 2019

With the recent update, this issue can be marked as resolved, #183. @BlackHawkPL

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants