Skip to content
This repository has been archived by the owner on May 24, 2022. It is now read-only.

Add Intel Items (Documents, Photos, Notepad) #11

Merged
merged 12 commits into from
Dec 17, 2019
Merged

Conversation

PabstMirror
Copy link
Contributor

@PabstMirror PabstMirror commented Jun 8, 2016

Edit: This PR was rewritten by mharis001 in 2019

Adds objects (carried by players as unique magazines) that hold intel.
Documents (printed text), Photos and a Notepad (editable text).

First two are designed for mission makers/zeus to add objectives in missions (marking cache locations, photos of a vip target).
The notepad can be used for anything, like recording fire missions, or leaving a message (notepads are unique)

Vanilla Arma has something similar for zeus (but not 3den as far as I can tell). But I don't really like how it works. This version uses actual objects that can be dropped/looted and doesn't just magically add briefings to an entire side.

Demo: https://www.youtube.com/watch?v=LWuGSXUVBbc
I'm honestly not sure if this fit's ACEX's scope. Aesthetically it's fairly basic, but functional.

@jonpas
Copy link
Member

jonpas commented Jun 8, 2016

What about putting the clickable icons to top bar in map view?

@nicolasbadano
Copy link

@PabstMirror, I really like this! the aesthetics could be improved, but it's a great adition nonetheless

What about putting the clickable icons to top bar in map view?

Nah, I wouldn't recommend it. Maybe they could be shown using the self-interact menu instead. idk

@jonpas
Copy link
Member

jonpas commented Jun 8, 2016

Self-intract menu on map sounds like a great place to put them IMO.

@jonpas jonpas modified the milestone: 3.1.0 Jun 18, 2016
@jonpas jonpas added the WIP label Jul 8, 2016
@jonpas jonpas modified the milestones: 3.1.0, 3.2.0 Oct 18, 2016
@jonpas jonpas modified the milestones: 3.3.0, 3.2.0 Feb 26, 2017
@654wak654
Copy link
Contributor

Would any of the new intel objects/ functions added with 1.76 be useful to this PR?

@PabstMirror PabstMirror modified the milestones: 3.3.0, 3.4.0 Dec 2, 2017
@stale
Copy link

stale bot commented Nov 10, 2018

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix label Nov 10, 2018
@PabstMirror PabstMirror modified the milestones: 3.4.0, Ongoing Nov 15, 2018
@thojkooi thojkooi changed the title Intel objects and notepad WIP: Intel objects and notepad Dec 1, 2018
@mharis001
Copy link
Member

Rewrote the component to have the aesthetic changes required. Less hard-coded and more of a framework now (as well as general improvements to the code).

No longer switches between a dialog and controls group. Simply a controls group that can be moved on the map display.

  • Fixes the flickering of the map cursor that would happen.
  • Removes the black background when in focus but I don't think that is necessary.

@mharis001 mharis001 changed the title WIP: Intel objects and notepad Add Intel Items Mar 13, 2019
@PabstMirror
Copy link
Contributor Author

Thanks for reviving this, all the changes look great
I pushed a small optimization that should allow us to skip running handleLoadout in most cases.
e.g. if you just reload your weapon
it does delay requesting mag id until you actually open the map
if you have high latency, it should still work fine, but actions just won't show up for a tiny bit func(addaction) fails safely with a nil value
let me know what you think, but I think it's worth it because it mostly eliminates the run-time cost of this addon

@mharis001
Copy link
Member

Good optimization 👍, and yeah don't think any delay should ever be a problem.

@jonpas jonpas modified the milestones: Ongoing, 3.5.0 Jun 24, 2019
@PabstMirror PabstMirror changed the title Add Intel Items Add Intel Items (Documents, Photos, Notepad) Dec 17, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants