Skip to content

Ports Wall Blood Splatters#6065

Open
SagaStars wants to merge 10 commits intoMonkestation:mainfrom
SagaStars:blood_splat
Open

Ports Wall Blood Splatters#6065
SagaStars wants to merge 10 commits intoMonkestation:mainfrom
SagaStars:blood_splat

Conversation

@SagaStars
Copy link
Copy Markdown

About The Pull Request

Ports blood-wall-splatting from Azure-Peak/Azure-Peak#4772 alongside an extra change to make it so firearms always trigger a blood splatter.

Why It's Good For The Game

More feedback for the combat and overall violence, more brutal, more gruesome, more worthy of screaming your lungs out.

Changelog

🆑
add: Added blood splatter on wall from Caustic Cove - Azure Peak Port.
add: Firearms always trigger an blood wall splatter
/:cl:

@github-actions github-actions Bot added the Feature adds a new thing label Apr 9, 2026
Comment thread code/game/objects/effects/decals/cleanable/humans.dm Outdated
Comment thread code/game/atom/atoms.dm Outdated
Comment thread code/game/objects/effects/decals/cleanable/humans.dm Outdated
Comment thread code/modules/mob/living/blood.dm Outdated
Comment thread code/modules/projectiles/projectile.dm Outdated
Co-authored-by: CheffieGithub <113442598+CheffieGithub@users.noreply.github.com>
@CheffieGithub
Copy link
Copy Markdown
Collaborator

My review is not resolved

@SagaStars
Copy link
Copy Markdown
Author

image

Not ready yet!

Tried to do the throw atom port of TG but we had different way of using the move_manager so got stresse and thinking I'm gonna be giving up in that and instead focusing in make it more performant the tile checking, deciding a direction and basically doing the same logic but without throwing it, the end its gonna be mostly the same and more performant, tell me if you don't like this.

Aside from that did the changes needed, moved the proc to the wallsplatter and removed unnecessary parenting .../blood/splatter/walls => .../blood/wallsplatter blabla

There's also something in which i'd like your guiance, atm the wall splatters have no limit, so you could stack 200 decals on a wall, we should put a limit in like 5-10, but how should i tackle it? Merge the icons once it hits the cap? Is it more performant? Check the contents of a wall, or add a walls_sprays variable to walls, so its just checks how many are there (cleaning them would reduce the counter)

@CheffieGithub
Copy link
Copy Markdown
Collaborator

A limit of 1 would be best actually, just prevent it from spawning if there is an existing decal if(locate(.../blood/wallsplatter) in loc)

@CheffieGithub
Copy link
Copy Markdown
Collaborator

I would only consider more than one when existing blood is changed to allow for colour mixing and stuff otherwise it's kinda pointless

@SagaStars SagaStars requested a review from CheffieGithub April 21, 2026 19:23
@SagaStars
Copy link
Copy Markdown
Author

Added the TG Impmentation, guarantee blood splatter on neak artery injurues, stacking yada yada

Comment thread code/modules/mob/living/blood.dm Outdated
@github-project-automation github-project-automation Bot moved this from In progress to Labeling Complete in Monkestation-TG Issues Tracker Apr 26, 2026
@realforest2001 realforest2001 added the FROZEN DNM but it's not your fault label May 4, 2026
@CheffieGithub CheffieGithub removed FROZEN DNM but it's not your fault labels May 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Feature adds a new thing

Projects

Status: Labeling Complete

Development

Successfully merging this pull request may close these issues.

4 participants