Conversation
|
I don't know who you are, but I love you. |
|
|
jonpas
left a comment
There was a problem hiding this comment.
This looks real good to me overall! 👍
addons/field_rations/$PBOPREFIX$
Outdated
| @@ -0,0 +1 @@ | |||
| z\acex\addons\field_rations No newline at end of file | |||
There was a problem hiding this comment.
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.
That's false. And we don't even use BI's Addon Builder anyways.
There was a problem hiding this comment.
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.
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 |
dedmen
left a comment
There was a problem hiding this comment.
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.
Where are these excessive 0's from? I saw them in a ACE rvmat too.
There was a problem hiding this comment.
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_weatherimplementation)CBA_settings_fnc_initinstead ofACE_Settings@PabstMirror @thojkooi