UnityContent: support multiple event handlers #110
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Fixes building & testing on case-sensitive file system.
- remove IUnityEvent - add UnityEvents class * AddEventListener() adds a callback for an event name * DispatchEvent() calls all callbacks for an event name * errors from event handlers are ignored * the first event handler returning truthy aborts dispatch - UnityContent * add UnityEvents class property for global events * only install one callback per global event name * add UnityEvents instance property for instance events Fixes jeffreylanters#109
Events which are dispatched via triggerUnityEvents() are instance specific events that do not need a global event handler attached to ReactUnityWebGL object. All other event names are assumed to be global ones. Fixes jeffreylanters#109
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes #109