-
Notifications
You must be signed in to change notification settings - Fork 42
Conversation
I don't know who you are, but I love you. |
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks real good to me overall! 👍
addons/field_rations/$PBOPREFIX$
Outdated
@@ -0,0 +1 @@ | |||
z\acex\addons\field_rations |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Empty newline missing.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I thought $PBOPREFIX$
files did not have newline at end, or at least according to this: https://community.bistudio.com/wiki/PBOPREFIX
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's false. And we don't even use BI's Addon Builder anyways.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
BI Addon builder doesn't even use PBOPREFIX file.
addons/field_rations/data/mre.rvmat
Outdated
@@ -0,0 +1,32 @@ | |||
ambient[]={1,1,1,1}; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This file should be cleaned up to conform to same config standards.
- Braces on same line
- Spaces around
=
Can we get some screenshots of the status UI icons @mharis001 ? |
Looks good IMO! Are those icons always up? If so, could they be made so they are hidden until you are actually lower on those supply (maybe a setting)? I am one for minimal UI on those status indicators.
Could also be a setting! 😆 On that note, can you drag it around like other elements? That'd be pretty neat too. |
The HUD shows when below 30% on either status or hovering interaction and I like the idea about moving elements around. |
Awesome! CfgUIGrids, MicroDAGR should be good reference. |
I have tested it and it works really well. Awesome! |
5c769a1
|
Added a delay on consume because progress bar will sometimes be closed by interaction menu closing Animations look fantastic, I'm not real familiar with |
I am working on adding slightly different animations and sounds for the soda cans (changing bottle cap opening to opening a can). I see you left the action offset commented out due to needing a typeOf. I think for some objects it is somewhat needed, what about adding a second array for corresponding offsets for the p3ds? |
I wasn't sure if the position offset was necessary, I think [0,0,0] works well enough for all the water sources, but it really does look nice having the point being exactly at the nozzle. I guess having placeable items might be nice |
Changed Other than that, I think this is finally good to go 👍 |
Merging now for the RC, can still add tweaks if we find anything |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very nice in-code documentation 👍
diffuse[] = {0.5,0.5,0.5,1}; | ||
forcedDiffuse[] = {0.5,0.5,0.5,0}; | ||
emmisive[] = {0,0,0,1}; | ||
specular[] = {0.30000001,0.30000001,0.30000001,0}; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Where are these excessive 0's from? I saw them in a ACE rvmat too.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure, this is how the file was from the original PR. Just conformed it to config guidelines.
When merged this pull request will:
Noteworthy differences:
ace_medical
&ace_weather
implementation)CBA_settings_fnc_init
instead ofACE_Settings
@PabstMirror @thojkooi