fix(RequiredBehaviourStateAttribute): don't use isActiveAndEnabled

ExtendRealityLtd / Malimbe

[Obsolete - No longer maintained] A collection of tools to simplify writing public API components for the Unity software.

MIT License

51 stars 11 forks source link

fix(RequiredBehaviourStateAttribute): don't use isActiveAndEnabled #56

Closed dustinlacewell closed 4 years ago

dustinlacewell commented 4 years ago

Unity contains a bug wherein isActiveAndEnabled can be false even though both enabled and gameObject.activeSelf are true. isActiveAndEnabled is largely a convenience and by checking enabled and gameObject.activeInHierarchy this issue is avoided while remaining functionally identical.

This change was designed by @bddckr and discussed on Slack. I have tested the change and confirmed it indeed works around the Unity issue.

ExtendReality-Bot commented 4 years ago

:tada: This PR is included in version 9.6.2 :tada:

The release is available on:

npm package (@latest dist-tag)
v9.6.2
GitHub release

Your semantic-release bot :package::rocket: