Skip to content

Nightvision - Add support for helmet integrated devices - #11421

Draft
Alien314 wants to merge 6 commits into
acemod:masterfrom
Alien314:patch-3
Draft

Nightvision - Add support for helmet integrated devices#11421
Alien314 wants to merge 6 commits into
acemod:masterfrom
Alien314:patch-3

Conversation

@Alien314

@Alien314 Alien314 commented Jul 5, 2026

Copy link
Copy Markdown

When merged this pull request will:

  • Add a secondary check to fnc_onSlotItemChanged and the "unit" playerEventHandler in XEH_postInit for headgear integrated night vision.
  • Changes fnc_pfeh to prevent non-color effects applying for nvg generation -1.
  • Adds config to remove border from integrated nvg classes and set them to gen -1.
  • Adds WP variants of pilot and special purpose helmets that use integrated night vision.

IMPORTANT

  • If the contribution affects the documentation, please include your changes in this pull request so the documentation will appear on the website.
  • Development Guidelines are read, understood and applied.
  • Title of this PR uses our standard template Component - Add|Fix|Improve|Change|Make|Remove {changes}.

@SpicyBagpipes

Copy link
Copy Markdown
Contributor

imo this should be toggleable somehow - whether item config, gvar or setting.

Modern pilot helmets use digital night vision that are almost certainly far better than ground spec night vision - and aviation spec night vision was already leaps and bounds ahead a decade ago. These are 2035'd pilot helmets, it'd make no sense for them to be worse than what is in service now.

@Alien314

Copy link
Copy Markdown
Author

My misunderstanding if this does make them worse. Main goal was an option that's not as bad on the eyes.

@SpicyBagpipes

Copy link
Copy Markdown
Contributor

BI implementation of NV is incredibly incredibly forgiving, ace makes it more realistic and thus nerfs it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants