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

[MIRROR] [THE HALF-MODULAR PRINCE] Snalance (Snail Balance) and Snissues (Snail Issues) Adjustment #1219

Merged
merged 1 commit into from
Dec 27, 2023

Conversation

Steals-The-PRs
Copy link
Collaborator

Original PR: Skyrat-SS13/Skyrat-tg#25439

About The Pull Request

How's it going.

So it's been a bit since we've last reconvened about 'a pointless snail PR' and I have decided to reaffirm my commitment to being the only bitch that cares about this niche species.

Awhile ago, you may remember I made a PR to make snails boneless. This has, yknow, obvious benefits, but my associate @nikothedude has informed me that this makes them easier to delimb. What a tragedy.

Due to my numerous historical jokes about making their shell armor actually apply to the chest, I went ahead and didn't do that, because that would be stupid. Instead, I ported the effect that roach hearts have; snails take 50% brute damage from the rear (backshots???) and while resting.
image
This has the same usual values as the roach heart.
To have this actually function, snails now flavor-wise have two hearts and the effect is stored on their heart to prevent stacking with the infusion. This has double the health, but you can literally get a T3 heart for the same effect.

Secondly, I've made another noteworthy change but this one is more of a bugfix than a buff. Due to a lack of gravity making snails automatically play City Escape and start moving at supersonic speeds, snails are now not affected by gravity when crawling. You can think of this as how snails affix to surfaces irl. Settler also no longer works for them when crawling, but there's shit all I can do about the fact that settler makes them walk a hint faster with it.

Lastly, I've made the various comments around the code be a bit more clear, ported new and shiny organ icons from Bay for them, and made their unique and only-fixable-once organs printable. Yay. Sprites are from Baystation12 PR # 30468, by AzlanonPC/SomeAngryMiner.

How This Contributes To The Skyrat Roleplay Experience

Here's where I 'erm actually' about the balance.
The balance is 'the rest of playing a snail is still miserable combat-wise' due to their low melee damage, the fact that optimal usage of this requires you to be floored where grabs and whatnot still apply, and their lack of bones reinforcing their limbs.

I simply think these changes are thematic for what snails are.

Proof of Testing

Screenshots/Videos The screencap in the OP.

Changelog

🆑 Nerev4r
add: Due to increased availability of calcium supplements in Sector 13 due to Deep Spacer lobbying, snailpeople have found their shells to be more suited towards protection of themselves and their recently-discovered-to-be-binary hearts in a combat scenario.
add: Snailpeople have been invited to recent EVA training sessions, and have learned to attach themselves to the floors if gravity generators go out; who'd have thunk it?
fix: Snailpeople no longer benefit from Settler's equipment load reduction while crawling. I could not think of a funny caption for this.
/:cl:

…l Issues) Adjustment (#25439)

* initial d

* holy shit i forgot

* i got so much cheese in my pocket, they thought I was a fucking calzone

* opp was sneak-dissing on the 'gram, turned his city into pompeii

* Just fixing some diffs (line breaks should match tg)

* Fixes these edit comments

---------

Co-authored-by: Giz <[email protected]>
@Iajret Iajret merged commit e44cfc6 into master Dec 27, 2023
23 checks passed
@Iajret Iajret deleted the upstream-mirror-25439 branch December 27, 2023 20:45
AnywayFarus added a commit that referenced this pull request Dec 27, 2023
Iajret pushed a commit that referenced this pull request Mar 1, 2024
* move override early returns

* fix CI on abductor airlock
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants