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

Feature/new input #897

Merged
merged 6 commits into from
Oct 8, 2020
Merged

Feature/new input #897

merged 6 commits into from
Oct 8, 2020

Conversation

stevehalliwell
Copy link
Collaborator

close #832
close #874

Description

Introduces alternate logic for core Fungus systems when the new unity InputSystem package is in use.

What is the current behavior?

Fungus is presently unable to function when the user switches to the new InputSystem unity Package due to hard coded use of UnityEngine.InputSystem.

What is the new behavior?

  • Following the instructions in the New InputSystems Example folder, readme.md
  • Conversations and other core Fungus input related systems function as they did with the old UnityEngine.Input class.
  • Adds a new EventHandler to handle executing a block in response to an InputAction.

Important Notes

  • My change require modifcations or additions to documentation (Regenerate auto doco)
  • My change adds demos to FungusExamples

Add initial input system driven handlers
Start support for InputSystem in Fungus components
New InputSystem logic updates within existing Fungus systems.
Remove dangling endregion from FungusConstants
Remove Input System Command, the overly generic nature of it is too error prone and counter to the event driven desire of the new Input System.
Update readme.md on New InputSystem example with steps to add new Input System and use the demos within.
@stevehalliwell stevehalliwell added this to the 3.14 milestone Oct 5, 2020
@stevehalliwell stevehalliwell merged commit 1e300cd into develop Oct 8, 2020
@stevehalliwell stevehalliwell deleted the feature/new_input branch October 8, 2020 19:51
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.

1 participant