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

Modified GameEventListeners to include AddComponent #60

Conversation

jeffcampbellmakesgames
Copy link
Contributor

Summary

This PR aims to add a small quality of life improvement by organizing the game event listener types into a more easily searchable menu when using [AddComponent] on a GameObject inspector. It also updates the template for game event listeners to include the [AddComponent] attribute so future classes can automatically gain the benefit of this change.

AddComponentMenusForGameEventListeners

Testing

I would create a GameObject in the scene and try navigating the add component menu to ensure that all game event listener types are present and that the menu item name for each one is consistent for casing/spelling with that same type used in other menus such as for Variables or Events.

There will also likely be a small conflict with #58 as both PRs modify this template, but should be easy to resolve.

Changes

Modified GameEventListeners for AddComponent

  • Modified SOArchitecture_Utility to include new constants for [AddComponent] menu structure for library and event listeners (currently only component type in library).
  • Modified code generation template to include [AddComponent] setup.
  • Modified all closed GameEventListener types to include [AddComponent] attribute with consistent menu names as used elsewhere for asset creation (case for menu item matches type name case).

* Modified SOArchitecture_Utility to include new constants for [AddComponent] menu structure for library and event listeners (currently only component type in library).
* Modified code generation template to include [AddComponent] setup.
* Modified all closed GameEventListener types to include [AddComponent] attribute with consistent menu names as used elsewhere for asset creation (case for menu item matches type name case).
@DanielEverland DanielEverland added this to the Release 1.6.0 milestone Apr 18, 2019
@DanielEverland DanielEverland merged commit e03bd76 into DanielEverland:master Apr 18, 2019
@jeffcampbellmakesgames jeffcampbellmakesgames deleted the feat/add_reference_component_menu branch April 18, 2019 13:42
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.

3 participants