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

Update Active Modules #5089

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

Conversation

machiecodes
Copy link
Contributor

@machiecodes machiecodes commented Jan 6, 2025

Type of change

  • Bug fix
  • New feature

Description

Organizes the settings in the module into groups, and more importantly adds a background setting in the same format as other hud elements. Also reworked the rendering logic to be slightly more efficient. First in a series of PRs I'm working on to standardize scale and background settings across all HUD modules

Related issues

N/A

How has this been tested?

2025-01-06.14-54-25.mp4

Checklist:

  • My code follows the style guidelines of this project.
  • I have added comments to my code in more complex areas.
  • I have tested the code in both development and production environments.

I hate big blobs of settings so this should be neater
Push stuff before bed, everything but inbetween quads and background are done
Block mode didn't look that good tbh
Since I hope to implement the same logic in other hud elements, some elements would also use getScale() in determining box size. Returning -1 would bork that up and is just shorthand for the text renderer to fallback to hud gettextscale anyway, so default to that instead
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.

1 participant