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

Update to new Unity version add better support for new Input system #6

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

aliakseikalosha
Copy link

@aliakseikalosha aliakseikalosha commented Dec 31, 2024

Changes:

  • Update to support unity 6.
  • Update for easier handling the plugin, move Editor related classes to editor folder.
  • Add support for better integration with new Input system.
  • Add namespace to not collide with other classes in projects this plugin is imported.
  • Small clean up of a project.
  • Refactor to better fit with C# coding guide lines.
  • Add Unity prefab with setup controller.
  • Add demo scene.

For better use of a project I would suggest to add release with a unity export of a project like this:
image

You can find export from my PC in Zip file attached to pull request.
FPSMovementController.zip

@B0N3head
Copy link
Owner

B0N3head commented Jan 18, 2025

Hey man, cheers for this.
Sorry it took so long for me to check this.
I'll have a look into this and merge it into a refactor I'm currently doing myself (a revive I've started recently over the past week).
The original code definitely wasn't the cleanest😅

Interested to see how you refactored it 🤙

@aliakseikalosha
Copy link
Author

So you can look into it in at Files changed tab.
I have come to it because I didn't wanted to wright my own controller once again, done several for work projects, and you know to support the open source plugins.
No problem, like it was a 2 year old project I wasn't expecting answer, and if you are okay with my changes, you can merge it in.

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.

2 participants