Hide advFatigue bar on map / spectator #4655
Conversation
|
Should do vehicles as well. |
|
Added, will close #4424 |
| [ACE_player, objNull] call FUNC(handlePlayerChanged); | ||
| }; | ||
| ["unit", FUNC(handlePlayerChanged)] call CBA_fnc_addPlayerEventHandler; | ||
| ["unit", FUNC(handlePlayerChanged), true] call CBA_fnc_addPlayerEventHandler; |
There was a problem hiding this comment.
The handlePlayerChanged function needs to run at least once. If the unit EH triggered already, this will completly break AF. Would recommend to just revert this to how it was.
There was a problem hiding this comment.
Third arg for addPlayerEventHandler was recently added to do the same thing:
_applyRetroactively - Call function immediately if player is defined already (optional, default: false) <BOOL>
There was a problem hiding this comment.
Oh, I see. That works then.
| @@ -13,6 +13,11 @@ params ["_stamina"]; | |||
|
|
|||
| private _staminaBarContainer = uiNamespace getVariable [QGVAR(staminaBarContainer), controlNull]; | |||
|
|
|||
| if ((vehicle ACE_player) != ACE_player) exitWith { // Hide when mounted | |||
There was a problem hiding this comment.
Why not use the vehicle player EH?
There was a problem hiding this comment.
I think we're still going to need the if statement here or it will be overwriten
There was a problem hiding this comment.
You could do it just like in the visibleMap EH. Both need checks if both conditions apply. I'd rather keep all the hiding/showing in one place.
|
Thank you @PabstMirror! |
No description provided.