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

Revert Revert Felinid Thievery #1183

Conversation

WarMechanic
Copy link
Contributor

@WarMechanic WarMechanic commented May 10, 2024

Readds #1102
Requires 'Do Not Merge' tag - the original PR was merged with pending admin discussion.

yeah also i got a singular rounds to test out changes in-game

  • i laughed at a security 'officer' cadet for having a book in their pocket
  • noticed july reiter doesnt put their appraisal tool in a utility belt
  • a felinid walked up to me and rummaged through my pockets
  • took an isolated persons pair of shoes and promptly got yelled at
  • stole stuff at round end when everyone was clumped up while concealing my identity

count tails in the bar and youll be fine

@github-actions github-actions bot added Changes: YML Changes any yml files Changes: C# Changes any cs files labels May 10, 2024
@deltanedas deltanedas added the S: Do Not Merge Don't merge this yet label May 10, 2024
@WarMechanic WarMechanic marked this pull request as ready for review May 10, 2024 09:06
@Bonktrauma
Copy link
Contributor

Does this mean that felinids can still steal without being noticed?

@WarMechanic
Copy link
Contributor Author

WarMechanic commented May 10, 2024

no, passive thieving gloves has been replaced by a liter version

felinids gain the ability to peek into other players' pockets and they strip faster
unlike passive thieving gloves this stacks with thieving gloves so go crazy

@Bonktrauma
Copy link
Contributor

no, passive thieving gloves has been replaced by a liter version

felinids gain the ability to peek into other players' pockets and they strip faster

So basically, they can steal faster but you can still see them stealing?

@WarMechanic
Copy link
Contributor Author

yes, not being able to see items stolen was the OP dealbreaker

@TadJohnson00
Copy link
Contributor

Probably needs some further balance discussion and input from administrators. Personally, I'm generally opposed to buffing thieving as I see it kill roleplay more often than it improves it, and I've always been against the "felinid = thief" stigma personally - but I'm open to hear what others have to say as well.

@WarMechanic
Copy link
Contributor Author

any word on this?
it seems that with the latest merge, dragging items has been fixed and now felinids dragging machines has been 'nerfed' due to their lower mass (it was always supposed to be this way). provides better justification for pocket vision :)

@NullWanderer
Copy link
Contributor

I don't believe features encouraging theft really fit in an MRP enviroment, and for that reason, this request will be denied entirely. Felinids should have other unique features that don't encourage LRP/NRP behaviour

Lyndomen is likely going to comment on this with a more in depth explanation later.

@WarMechanic WarMechanic deleted the revert-1177-revert-1102-thieving branch June 12, 2024 16:24
@WarMechanic
Copy link
Contributor Author

Thank you

LaryNevesPR pushed a commit to LaryNevesPR/Estacao-Andromeda that referenced this pull request Jan 3, 2025
# Description

Does what it says on the can, ports over the updates to the Health
Analyzer made in Wizden's
[#30834](space-wizards/space-station-14#30834),
[#31879](space-wizards/space-station-14#31879),
and
[#32636](space-wizards/space-station-14#32636)
because I need them for newmed


Credit to [Floof
DeltaV-Station#169](Fansana/floofstation1#169) too since I
yoinked it due to lazyness :trollface:

---

# Changelog

🆑 Mocho
- tweak: Ported over Wizden's updated Health Analyzer UI.

---------

Co-authored-by: FoxxoTrystan <[email protected]>
Co-authored-by: goet <[email protected]>
Co-authored-by: Saphire Lattice <[email protected]>
LaryNevesPR pushed a commit to LaryNevesPR/Estacao-Andromeda that referenced this pull request Jan 3, 2025
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 S: Do Not Merge Don't merge this yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants