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.

Sign up for GitHub

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

Feature/option clear jvmf #432

Merged
merged 14 commits into from
Oct 20, 2024
Merged

Feature/option clear jvmf #432

Show file tree
Hide file tree
Changes from 2 commits
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
059387c
Added in option to remove default JVMF messages
AdozenBABYdogs Nov 8, 2023
9d1f28d
Forgot to add one line of code
AdozenBABYdogs Nov 8, 2023
c37bf52
rearanged items order
AdozenBABYdogs Nov 8, 2023
50fcce9
added missing comma
AdozenBABYdogs Nov 8, 2023
312be9b
fixed array -> bools
AdozenBABYdogs Nov 8, 2023
28ca41a
fixed typo: Misplaced semicollin
AdozenBABYdogs Nov 8, 2023
90b847c
resolving bug fixfor JVMF messages not displaying
AdozenBABYdogs Nov 8, 2023
efca163
Additonal Bug fixes for JVMF 0 messages
AdozenBABYdogs Nov 8, 2023
618c5d7
attemping to fix JVMF not displaying
AdozenBABYdogs Nov 8, 2023
152475b
re-adjusted jvmf to try anddisplay message
AdozenBABYdogs Nov 8, 2023
af2eb08
resolving bug fixfor JVMF messages not displaying
AdozenBABYdogs Nov 10, 2023
633a4b0
Added in JVMF delete button TODO: fix new bugs
AdozenBABYdogs Nov 10, 2023
dfb2f5c
JVMF Delete message works as intended
AdozenBABYdogs Nov 10, 2023
852a50d
layout shift & debug messages
BroBeansCPG Oct 1, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 3 additions & 0 deletions addons/uh60_jvmf/XEH_preInit.sqf
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -114,6 +114,9 @@ VTX_JVMF_MESSAGES = [
]
*/
];
if (!vtx_uh60_jvmf_defaults) then {
VTX_JVMF_MESSAGES = [];
};
VTX_JVMF_SELECTED_IDX = 0;

ADDON = true;
8 changes: 8 additions & 0 deletions addons/uh60_jvmf/XEH_preStart.sqf
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,3 +1,11 @@
#include "script_component.hpp"

#include "XEH_PREP.hpp"

[
"vtx_uh60_jvmf_defaults"
,"CHECKBOX"
,["Enable JVMF Default Messages", "The JVMF Default messages provide an example of usage of the JVMF"]
,["UH-60M", "JVMF"]
[true];
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This doesn't look correct.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yep, I see it. I'm missing a comma. Does the syntax for cba_fnc_addSetting look correct?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I also think the order on where I'm executing the code if off as well, and this function belongs in the "XEH_preInit.sqf"

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bool, not an array.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you

] call CBA_fnc_addSetting;