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

Lasers Have No Recoil #603

Merged

Conversation

VMSolidus
Copy link
Member

Description

LASERS HAVE NO RECOIL, THEY SHOULD NOT PUSH YOU AROUND IN SPACE. THE CAPTAIN'S LASER PISTOL IS NOT AN INFINITE USE JET PACK. STUPID BUG. ME FIX.

There are MANY things I hate about the gun system's code, I am going to revisit this later with a more comprehensive update. There are many hardcoded constants in it related to physics that I thoroughly despise. Guns should just be allowed to declare how much force of recoil they have, rather than it being hardcoded as "CONSTANT 25f". Also, I would like that if the Recoil value is greater than your character's mass, to then knock your character on their ass and/or send them flying. Imagine a Felinid getting thrown back by firing a shotgun. There's my justification for doing so.

Changelog

🆑

  • fix: Lasers no longer function as jetpacks in space.

@github-actions github-actions bot added Changes: C# Changes any cs files Changes: YML Changes any yml files labels Jul 28, 2024
@VMSolidus VMSolidus added Type: Bug [Un]intentional issue with something Size: 5-Very Small For especially small issues/PRs Priority: 4-Low Should be resolved at some point labels Jul 28, 2024
@DangerRevolution DangerRevolution merged commit c0114c0 into Simple-Station:master Jul 31, 2024
12 checks passed
SimpleStation14 added a commit that referenced this pull request Jul 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: C# Changes any cs files Changes: YML Changes any yml files Priority: 4-Low Should be resolved at some point Size: 5-Very Small For especially small issues/PRs Type: Bug [Un]intentional issue with something
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants