Skip to content

Fix missing collision and occluder on tower prop - #2803

Closed
KarmDK wants to merge 4 commits into
endlessm:mainfrom
KarmDK:2248_tower_physics
Closed

KarmDK wants to merge 4 commits into
endlessm:mainfrom
KarmDK:2248_tower_physics

Conversation

@KarmDK

@KarmDK KarmDK commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

This pull request adds collision physics and visual occlusion to the tower. Previously, the tower only existed as a visual sprite, allowing the player to pass through it and guards to detect them through the structure.

The tower scene has been updated with a StaticBody2D containing collision shapes for environmental collisions and visual occlusion, preventing player overlap and ensuring the guard's line of sight is properly blocked. A LightOccluder2D was also added to block the visual projection of the guard's light cone.

Resolves
#2248

@KarmDK
KarmDK requested a review from a team as a code owner August 28, 2026 15:46
@github-actions

Copy link
Copy Markdown

Play this branch at https://play.threadbare.game/branches/KarmDK/2248_tower_physics/.

(This launches the game from the start, not directly at the change(s) in this pull request.)

@manuq manuq left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This PR carries your changes of your previous task (horizontal door). Please sync your fork with main before continuing, and make this PR show only the changes of the current task.

@KarmDK KarmDK closed this Aug 28, 2026
@KarmDK
KarmDK deleted the 2248_tower_physics branch August 28, 2026 21:43
@manuq

manuq commented Aug 31, 2026 •

Copy link
Copy Markdown
Collaborator

@KarmDK for the record you don't have to close the pull request and open a new one, you can always update it.

@KarmDK

KarmDK commented Sep 1, 2026

Copy link
Copy Markdown
Contributor Author

Oh, I understand. So, should I update this one or wait for a reply to the other one? Sorry.

@manuq

manuq commented Sep 2, 2026

Copy link
Copy Markdown
Collaborator

@KarmDK no worries, let's continue on #2805 this time.

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