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

[Framework] Rework Extras draw #1422

Closed
wants to merge 2 commits into from

Conversation

chaserli
Copy link
Contributor

Everyone (e.g. #1287 #1309 #1395 #1418 #1148) is doing their own work separately and creating overhead and incompatibilities.
As we've stated before the entire Healthbar drawing system needs to be overwritten completely.

This settles the framework for all these.

TODO:

  • BarType + DigitalsType
  • For : HP, SP, IC timer, Temporal, ROF timer, Reload timer, Factory progress, SW progress, ...
  • Condition: selected/always
  • Location: Attached/ Global

@chaserli chaserli added the Minor Documentation is not required label Nov 11, 2024
@chaserli chaserli marked this pull request as draft November 11, 2024 12:36
Copy link

github-actions bot commented Nov 11, 2024

Nightly build for this pull request:

This comment is automatic and is meant to allow guests to get latest nightly builds for this pull request without registering. It is updated on every successful build.

@chaserli chaserli closed this Feb 24, 2025
@Metadorius
Copy link
Member

Why?

@chaserli
Copy link
Contributor Author

Why?

To reduce PR number.

I'm currently unavailable to work on this subject, more specifically properly dehardcoding stuff and design ini structure, which might require constant feedback and discussion. Meanwhile people might be impatient for things like #1287 #1309 etc.

The common issue here and in the current digital display is the overhead, like the same draw point being calculated multiple times. Similar overhead also exists in vanilla code, if you don't use gdi ddraw you might not notice.

Will resume later.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Minor Documentation is not required
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants