-
Notifications
You must be signed in to change notification settings - Fork 739
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.
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
Hearing - Add Electronic Hearing Protection #10476
Conversation
Co-authored-by: Grim <69561145+LinkIsGrim@users.noreply.github.com>
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.
Having some sort of arsenal stat would be helpful (I imagine a simple text based "yes/no" for EHP would be sufficient, but maybe one might want to combine it with the existing hearing protection bar).
|
||
// Unequip electronic hearing protection when gear has one built in | ||
if (_hasEHPOn && _hasBuiltInEHP) then { | ||
[ACE_player, false] call FUNC(removeEHP); |
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.
If a unit's inventory is full and currently has EHP (from inventory item) equipped, then proceeds to equip a helmet with EHP built in, it won't remove the EHP. Do we care about that?
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 don't
Stat preview, also fixed headgear not showing earpro stats, now requires #10559 |
When merged this pull request will:
playSoundUI
for ear ringing to not mute it when EHP takes effectNoise reduction amount and duration were measured from this video: https://www.youtube.com/watch?v=4A30yaoj5Yo
Could be extended with a custom model, but looks odd when attached to head (see BI Feedback Tracker).
Would like to add a put on / take off sound later, but not a priority.