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

add tooltip flag requirements #2320

Open
wants to merge 2 commits into
base: dev
Choose a base branch
from

Conversation

SrRapero720
Copy link

How it looks without flags or impact
image

how it looks with flags or impact
image

@jellysquid3
Copy link
Member

Why is it necessary to indicate that a renderer update is needed? That's an internal only thing which basically only tells the renderer to not re-use the last frame's contents.

Most of these states for that matter don't seem useful outside of the asset reload & game restart.

@SrRapero720
Copy link
Author

Game restart and assets reload are the most relevant. But to not make more complex unnecessary extended logic to ignore render reload and render restart I keep these flags displayed in tooltip.

@SrRapero720
Copy link
Author

rebased + mojmapised.
Also i add a small (hardcoded) check for low ordinal flags (REQUIRES_RENDERER_RELOAD and REQUIRES_RENDERER_UPDATE)

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

Successfully merging this pull request may close these issues.

None yet

2 participants