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

Revolver body reload #214

Merged
merged 2 commits into from
Aug 5, 2023
Merged

Revolver body reload #214

merged 2 commits into from
Aug 5, 2023

Conversation

Fueredoriku
Copy link
Contributor

@Fueredoriku Fueredoriku commented Aug 1, 2023

Adding a slooooow reload animation for the starting body so it isn't better than all the other bodies in the game

basicReload

I plan to further improve this animation in the future by making the player "cram" bullets into the opened slot during the animation, but this is something I want to implement together with actual "gun holding" IK animations. (Will be a whole other PR)

@Fueredoriku Fueredoriku self-assigned this Aug 1, 2023
@Fueredoriku Fueredoriku changed the base branch from main to dev August 1, 2023 21:04
@Fueredoriku Fueredoriku marked this pull request as ready for review August 4, 2023 19:34
Copy link
Contributor

@toberge toberge left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's great to finally have some actual reload for the basic body, even though it's a little placeholder-y.

The animation looks a little empty when you're just staring at the barrel for some seconds, but meh, that'll be fixed later. The smoke is a nice touch.
Looks a little off with the WIP cog barrel of course:
image
Other than that, it is now very noticeable that the shotgun has too much ammo, we should consider modifying ammo cap in barrels :)))))

@Fueredoriku Fueredoriku merged commit 42d3b30 into dev Aug 5, 2023
2 checks passed
@Fueredoriku Fueredoriku deleted the feature/revolver-reload branch August 5, 2023 20:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done 🎉
Development

Successfully merging this pull request may close these issues.

2 participants