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

Properly clear the alpha discard param in IconComponent & QuadBackgroundComponent #115

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

Conversation

Ali-RS
Copy link
Contributor

@Ali-RS Ali-RS commented Feb 14, 2023

This should properly clean up the alpha discard param in QuadBackgroundComponent and IconComponent when alpha discard is disabled (is set to 0).

…ent and IconComponent when alpha discard is disabled (is set to 0).
@Ali-RS Ali-RS changed the title Properly clean up the alpha discard param in IconComponent & QuadBackgroundComponent Properly clear the alpha discard param in IconComponent & QuadBackgroundComponent Feb 14, 2023
@pspeed42
Copy link
Contributor

99% sure that setting alphaDiscardThreshold to 0 is equivalent of disabling it... ie: any of the in-shader ifdefs will be false and so on.

@Ali-RS
Copy link
Contributor Author

Ali-RS commented Feb 14, 2023

I just followed up based on your comment below that said it is an oversight to not "clear" it.

https://hub.jmonkeyengine.org/t/lemur-iconcomponent-issue-with-transparent-icons/45257/10?u=ali_rs

I do not have much knowledge about shaders (but 0 meaning "not-defined" sounds weird to me though) but anyway If you think these changes are not required then please feel free to close the PR.

By the way, thanks for the response

@pspeed42
Copy link
Contributor

Ahah. So I'm just trying to be thorough. I'll try to incorporate this the next time I do a pre-release pass. Thanks for the contribution.

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.

None yet

2 participants