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

Security resprite and loadout revision #1273

Merged
23 commits merged into from
Jul 15, 2024

Conversation

TadJohnson00
Copy link
Contributor

@TadJohnson00 TadJohnson00 commented May 30, 2024

About the PR

Resprited detective gear as well as some other security and security-adjacent equipment, tweaked security's loadouts so they make sense. See #1197 for further details.

Why / Balance

For consistency in uniform. Mild balance changes were made with regards to replacement of warden's and HoS' jackets with standard coats, see PR #1197.

Technical details

n/a

Media

Please see #1197 for further media.
330685316-d904513d-f158-48d1-9765-e99a7d6fb472

  • I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

🆑

  • add: New security outer clothing for all security roles, as well as proper senior security officers' and cadets' jumpsuits.
  • add: Jetpacks are now available in all security suit storage. Thank goodness for increased funding.
  • tweak: Revamped security's clothing and equipment options; all gear is now up to code, and no further outdated equipment will be issued. This includes, most notably, the detective's uniform, which has received a whole makeover.
  • fix: Fixed security hardsuits not allowing you to turn on internals without a mask.
  • remove: Head of security's winter coat, warden's winter coat, security winter coat, security departmental winter boots, WizDen-pattern body armour, and some other minor changes necessary for respriting.

@github-actions github-actions bot added Changes: YML Changes any yml files Changes: Localization Changes any ftl files Changes: Sprite Changes any png or json in an rsi labels May 30, 2024
Copy link
Contributor

github-actions bot commented May 30, 2024

RSI Diff Bot; head commit fe557ea merging into 353eb66
This PR makes changes to 1 or more RSIs. Here is a summary of all changes:

Resources/Textures/DeltaV/Clothing/Uniforms/Jumpskirt/brigmedic.rsi

State Old New Status
equipped-INNERCLOTHING Modified

Resources/Textures/DeltaV/Clothing/Uniforms/Jumpskirt/centcom_officer.rsi

State Old New Status
equipped-INNERCLOTHING Modified

Resources/Textures/DeltaV/Clothing/Uniforms/Jumpskirt/hopmesskit.rsi

State Old New Status
equipped-INNERCLOTHING Modified

Resources/Textures/DeltaV/Clothing/Uniforms/Jumpskirt/hos.rsi

State Old New Status
equipped-INNERCLOTHING Modified

Resources/Textures/DeltaV/Clothing/Uniforms/Jumpskirt/hos_blue.rsi

State Old New Status
equipped-INNERCLOTHING Modified

Resources/Textures/DeltaV/Clothing/Uniforms/Jumpskirt/hos_grey.rsi

State Old New Status
equipped-INNERCLOTHING Modified

Resources/Textures/DeltaV/Clothing/Uniforms/Jumpskirt/security.rsi

State Old New Status
equipped-INNERCLOTHING Modified

Resources/Textures/DeltaV/Clothing/Uniforms/Jumpskirt/security_blue.rsi

State Old New Status
equipped-INNERCLOTHING Modified

Resources/Textures/DeltaV/Clothing/Uniforms/Jumpskirt/security_grey.rsi

State Old New Status
equipped-INNERCLOTHING Modified

Resources/Textures/DeltaV/Clothing/Uniforms/Jumpskirt/warden.rsi

State Old New Status
equipped-INNERCLOTHING Modified

Resources/Textures/DeltaV/Clothing/Uniforms/Jumpskirt/warden_blue.rsi

State Old New Status
equipped-INNERCLOTHING Modified

Resources/Textures/DeltaV/Clothing/Uniforms/Jumpskirt/warden_grey.rsi

State Old New Status
equipped-INNERCLOTHING Modified

Resources/Textures/DeltaV/Clothing/Uniforms/Jumpsuit/brigmedic.rsi

State Old New Status
equipped-INNERCLOTHING Modified
icon Modified
inhand-right Modified

Resources/Textures/DeltaV/Clothing/Uniforms/Jumpsuit/hopmesskit.rsi

State Old New Status
equipped-INNERCLOTHING Modified

Resources/Textures/Clothing/Mask/joy.rsi

State Old New Status
equipped-MASK-vulpkanin Added

Resources/Textures/Clothing/Mask/neckgaiter.rsi

State Old New Status
equipped-MASK-vulpkanin Added

Resources/Textures/DeltaV/Clothing/Head/Hats/beret_det.rsi

State Old New Status
equipped-HELMET Added
icon Added
inhand-left Added
inhand-right Added

Resources/Textures/DeltaV/Clothing/OuterClothing/Coats/labcoat_corpsman.rsi

State Old New Status
equipped-OUTERCLOTHING Added
icon Added
inhand-left Added
inhand-right Added

Resources/Textures/DeltaV/Clothing/OuterClothing/WinterCoats/coatdet.rsi

State Old New Status
equipped-OUTERCLOTHING Added
icon Added
inhand-left Added
inhand-right Added

Resources/Textures/DeltaV/Clothing/OuterClothing/WinterCoats/coathos.rsi

State Old New Status
equipped-OUTERCLOTHING Added
icon Added
inhand-left Added
inhand-right Added

Resources/Textures/DeltaV/Clothing/OuterClothing/WinterCoats/coatsec.rsi

State Old New Status
equipped-OUTERCLOTHING Added
icon Added
inhand-left Added
inhand-right Added

Resources/Textures/DeltaV/Clothing/OuterClothing/WinterCoats/coatwarden.rsi

State Old New Status
equipped-OUTERCLOTHING Added
icon Added
inhand-left Added
inhand-right Added

Resources/Textures/DeltaV/Clothing/Uniforms/Jumpskirt/detective.rsi

State Old New Status
equipped-INNERCLOTHING Added
icon Added
inhand-left Added
inhand-right Added

Resources/Textures/DeltaV/Clothing/Uniforms/Jumpskirt/secformal.rsi

State Old New Status
equipped-INNERCLOTHING Added
icon Added
inhand-left Added
inhand-right Added

Resources/Textures/DeltaV/Clothing/Uniforms/Jumpsuit/detective.rsi

State Old New Status
equipped-INNERCLOTHING Added
icon Added
inhand-left Added
inhand-right Added

Resources/Textures/DeltaV/Clothing/Uniforms/Jumpsuit/security_cadet.rsi

State Old New Status
equipped-INNERCLOTHING Added
icon Added
inhand-left Added
inhand-right Added

Resources/Textures/DeltaV/Clothing/Uniforms/Jumpsuit/security_tactical.rsi

State Old New Status
equipped-INNERCLOTHING Added
icon Added
inhand-left Added
inhand-right Added

Edit: diff updated after fe557ea

@DangerRevolution
Copy link
Contributor

This needs a change log

@TadJohnson00
Copy link
Contributor Author

Is the one in #1197 not sufficient?

@DangerRevolution
Copy link
Contributor

Is the one in #1197 not sufficient?

no, the bot doesn't see that PR and won't see it's changelog

@TadJohnson00
Copy link
Contributor Author

Got it, should be fixed now.

Copy link
Contributor

@DangerRevolution DangerRevolution left a comment

Choose a reason for hiding this comment

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

needs delta-v comments, needs more changelog entries, some of the uniforms need to be reparented appropriately instead of rehashing the same components

Resources/Prototypes/Catalog/Fills/Items/belt.yml Outdated Show resolved Hide resolved
Resources/Prototypes/Catalog/Fills/Lockers/heads.yml Outdated Show resolved Hide resolved
Resources/Prototypes/Catalog/Fills/Lockers/heads.yml Outdated Show resolved Hide resolved
Resources/Prototypes/Catalog/Fills/Lockers/security.yml Outdated Show resolved Hide resolved
Resources/Prototypes/Loadouts/loadout_groups.yml Outdated Show resolved Hide resolved
Resources/Prototypes/Loadouts/loadout_groups.yml Outdated Show resolved Hide resolved
Resources/Prototypes/Loadouts/loadout_groups.yml Outdated Show resolved Hide resolved
Resources/Prototypes/Loadouts/role_loadouts.yml Outdated Show resolved Hide resolved
@github-actions github-actions bot added the Status: Merge Conflict Fix your PR! label Jun 1, 2024
Copy link
Contributor

github-actions bot commented Jun 1, 2024

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@IamVelcroboy
Copy link
Member

@TadJohnson00 you mentioned that if/when this is loaded it would be able to play into the Hammurabi Warden needs. Is that still the case? Or should I be seeking out sprites form out contributors?

@github-actions github-actions bot removed the Status: Merge Conflict Fix your PR! label Jun 13, 2024
@TadJohnson00
Copy link
Contributor Author

@TadJohnson00 you mentioned that if/when this is loaded it would be able to play into the Hammurabi Warden needs. Is that still the case? Or should I be seeking out sprites form out contributors?

That's pretty much the case. The roles, their mechanical functionality, and their appearances will be almost identical, but I'll follow this up with a PR that actually defines the role itself.

@IamVelcroboy
Copy link
Member

That's pretty much the case. The roles, their mechanical functionality, and their appearances will be almost identical, but I'll follow this up with a PR that actually defines the role itself.

I already have all that ready here. I just need the textures to complete it.

@TadJohnson00
Copy link
Contributor Author

Should be all set, and ready for review.

@TadJohnson00
Copy link
Contributor Author

That's pretty much the case. The roles, their mechanical functionality, and their appearances will be almost identical, but I'll follow this up with a PR that actually defines the role itself.

I already have all that ready here. I just need the textures to complete it.

That's pretty much the case. The roles, their mechanical functionality, and their appearances will be almost identical, but I'll follow this up with a PR that actually defines the role itself.

I already have all that ready here. I just need the textures to complete it.

I'll make sure to work on a few RSIs for ya then. I can get you a jumpsuit, PDA, ID, and job icon, and some outerclothing if you think it's necessary.

And for what it's worth, I think the role of "Armorer" is more fitting than "Armory Commander", but that's not my call to make.

Copy link
Contributor

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@TadJohnson00
Copy link
Contributor Author

I can fix the merge conflict easily enough. Is everything else all good to go?

@Lyndomen
Copy link
Contributor

Jail guard? :godo:

@TadJohnson00
Copy link
Contributor Author

After this gets merged I'll do jail guard and armourer

@github-actions github-actions bot removed the Status: Merge Conflict Fix your PR! label Jun 28, 2024
@Timemaster99

This comment was marked as spam.

@github-actions github-actions bot added the Status: Merge Conflict Fix your PR! label Jul 9, 2024
Copy link
Contributor

github-actions bot commented Jul 9, 2024

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@github-actions github-actions bot removed the Status: Merge Conflict Fix your PR! label Jul 13, 2024
@TadJohnson00

This comment was marked as spam.

Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

Approving this with intent of merging; one caveat though - TJ, in a future PR, please add back a way to access the old detective coats / greatcoat that you removed in this PR. That is my one nitpick. Uniform printer or hacking the DetDrobe or SecDrobe or something works.

@ghost ghost merged commit 216a3f1 into DeltaV-Station:master Jul 15, 2024
13 checks passed
@TadJohnson00 TadJohnson00 deleted the secrespritefinally branch August 1, 2024 23:47
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: Localization Changes any ftl files Changes: Sprite Changes any png or json in an rsi Changes: YML Changes any yml files
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants