Skip to content
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

missile_hud - Move HUD feature from laser to new component #10306

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

TheCandianVendingMachine
Copy link
Contributor

@TheCandianVendingMachine TheCandianVendingMachine commented Sep 10, 2024

When merged this pull request will:

  • Move HUD functionality from laser to missile_hud
  • Add API to add new HUD elements depending on conditions

the HUD for laser guided munitions lived in laser forever. This doesn't make any sense, as the HUD is meant to showcase missile information. A new API was made to be able to extend what is shown to the user, as well as move functionality from laser

It works in an immediate mode fashion. New elements are constructed each frame and added to the HUD. The elements which get generated are culled each time the HUD PFH gets rebuilt, so only relevant items are drawn. A mapping function exists to allow for pre-processing of any information needed

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}.

@TheCandianVendingMachine TheCandianVendingMachine added this to the Ongoing milestone Sep 10, 2024
@TheCandianVendingMachine TheCandianVendingMachine added the kind/enhancement Release Notes: **IMPROVED:** label Sep 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/enhancement Release Notes: **IMPROVED:**
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant