Skip to content

Player and Object Collision #6

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Squird5
Copy link

@Squird5 Squird5 commented Jan 12, 2025

You can bump into shit

Description

My biggest complaint with SS14 is the lack of collision and everyone just passing through each other. This adds player collision and pushing. If pushing is not desired, it can easily be disabled without removing the collision.
You can also walk through open lockers AND be able to push the closed ones!

Note: While this is not mutually exclusive with Tile Movement, it is not really intended for it


TODO

There might be some variable tweaks needed here or there, but it will probably require a bit of field testing to determine.
Octagonal collision boxes might also be nice, but may cause the performance cost to double. These are circle colliders and not square colliders, so I am not completely certain


Media

2025-01-11.18-57-55.mp4


Changelog

🆑

  • add: Player-Player collision & pushing
  • add: Player-Object collision pushing
  • tweak: Put lockers on the crates collision layer

@@ -1,7 +1,7 @@
uplink-emp-flashlight-name = Электромагнитный фонарик
uplink-emp-flashlight-desc = Замаскированное под фонарик устройство. При ударе выпускает ЭМИ, поражающий электрические устройства.

#Armor
# Armor
Copy link
Collaborator

Choose a reason for hiding this comment

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

Can you revert this change? Just ends up being easier to maintain upstream parity in general if we avoid things like this.

Copy link
Author

Choose a reason for hiding this comment

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

Yeah, it was giving me a red warning and being a shit

@brndd
Copy link

brndd commented Jan 12, 2025

This is a bad idea, but do it anyway.

@Eneocho Eneocho added Balance Potentially going to upset people as it changes balance of the game. Content Adds something. Neat! and removed Changes: YML Changes: C# Changes: Localization labels Jan 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Balance Potentially going to upset people as it changes balance of the game. Content Adds something. Neat!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants