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

Adds Clothing / sprites to various jobs. #18

Open
wants to merge 26 commits into
base: master
Choose a base branch
from

Conversation

rynkatze
Copy link
Collaborator

@rynkatze rynkatze commented Aug 6, 2024

About the PR

Clothes for unique SP14 roles.

Why / Balance

cooooooooool asf.

Media

image
image
image
image

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

Breaking changes

Changelog

Copy link
Contributor

github-actions bot commented Aug 6, 2024

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

Copy link
Contributor

github-actions bot commented Aug 6, 2024

RSI Diff Bot; head commit 0bcd0c3 merging into 76cabcd
This PR makes changes to 1 or more RSIs. Here is a summary of all changes:

Resources/Textures/Clothing/Head/Hardsuits/sp14liberator.rsi

State Old New Status
icon-flash Added
icon Added
off-equipped-HELMET-vox Added
off-equipped-HELMET Added
on-equipped-HELMET-vox Added
on-equipped-HELMET Added

Resources/Textures/Clothing/Head/Hardsuits/sp14liberatorcommander.rsi

State Old New Status
icon-flash Added
icon Added
off-equipped-HELMET-vox Added
off-equipped-HELMET Added
on-equipped-HELMET-vox Added
on-equipped-HELMET Added

Resources/Textures/Clothing/Head/Hardsuits/sp14warden.rsi

State Old New Status
icon-flash Added
icon Added
off-equipped-HELMET-vox Added
off-equipped-HELMET Added
on-equipped-HELMET-vox Added
on-equipped-HELMET Added

Resources/Textures/Clothing/Head/Hats/sp14wardencap.rsi

State Old New Status
equipped-HELMET-hamster Added
equipped-HELMET Added
icon Added

Resources/Textures/Clothing/Neck/Cloaks/sp14liberatorcommander.rsi

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

Resources/Textures/Clothing/Neck/Cloaks/sp14warden.rsi

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

Resources/Textures/Clothing/OuterClothing/Armor/sp14warden_carapace.rsi

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

Resources/Textures/Clothing/OuterClothing/Hardsuits/sp14liberator.rsi

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

Resources/Textures/Clothing/OuterClothing/Hardsuits/sp14liberatorcommander.rsi

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

Resources/Textures/Clothing/OuterClothing/Hardsuits/sp14warden.rsi

State Old New Status
Equipped-OUTERCLOTHING-vox Added
Equipped-OUTERCLOTHING Added
icon Added
inhand-left Added
inhand-right Added

Resources/Textures/Clothing/Uniforms/Jumpskirt/sp14wardens.rsi

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

Resources/Textures/Clothing/Uniforms/Jumpsuit/sp14liberator.rsi

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

Resources/Textures/Clothing/Uniforms/Jumpsuit/sp14maintenancetech.rsi

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

Resources/Textures/Clothing/Uniforms/Jumpsuit/sp14warden.rsi

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

Resources/Textures/Markers/jobs.rsi

State Old New Status
sp14warden Added

Resources/Textures/Structures/Storage/closet.rsi

State Old New Status
sp14warden_door Added
sp14warden_open Added

Edit: diff updated after 0bcd0c3

@rynkatze rynkatze closed this Aug 6, 2024
@rynkatze
Copy link
Collaborator Author

rynkatze commented Aug 6, 2024

forgot to revert my branch. its over.

@rynkatze rynkatze reopened this Aug 6, 2024
@rynkatze
Copy link
Collaborator Author

rynkatze commented Aug 6, 2024

fixing stupid pr mistake

@rynkatze rynkatze changed the title Add clothing Adds Clothing / sprites to various jobs. Aug 6, 2024
@rynkatze
Copy link
Collaborator Author

rynkatze commented Aug 6, 2024

Conflict is me changing the sprite for the warden spawner for the sake of my eyes.

TO DO:

  • Warden Jumpskirt loadout / implementation
  • Warden Gloves (?, right now using combat gloves)
  • Warden Boots (i just want boots with a pistol slot because i think thats sick)
  • Vox hardsuit variants. (misery and pain.)
  • Liberator hardsuits
  • Maintenance jumpsuits
  • Warden backpack
  • warden glasses item i have decided i want to make.
  • fix displacement on warden hardsuit (oops)

Copy link
Collaborator

@Truoizys Truoizys left a comment

Choose a reason for hiding this comment

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

definitions for new clothing and hardsuits should be in separate files to ss14 clothing.
e.g. Clothing/Head/SP14/hardsuit-helmets.yml

for the sake of consistency also might be worth changing ids from including SP14 to just including SP. (might do it myself in a sec)

Really cooked here boss! love the sprites, love the tooltips.

@Truoizys Truoizys added the Status: Changes Needed will not be merged without requested changes. label Aug 7, 2024
@rynkatze
Copy link
Collaborator Author

will be ready for merge once ive actually tested this. changes done.

Copy link
Collaborator

@Truoizys Truoizys Aug 22, 2024

Choose a reason for hiding this comment

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

as much as it may help your skyblock trading coflmod probably won't be any use in this repo

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I WAS WONDERING WHERE I PUT IT HAHAHAHA

Comment on lines -413 to +414
name: warden's uniform
description: A formal security suit for officers complete with Nanotrasen belt buckle.
name: chief custodial officer's uniform
description: A formal security skirt for officers complete with Nanotrasen belt buckle.
Copy link
Collaborator

Choose a reason for hiding this comment

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

gunna need to separate these to a sp14 file

name: warden's uniform
name: chief custodial officer's uniform
Copy link
Collaborator

Choose a reason for hiding this comment

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

same here

Comment on lines 545 to 563
- type: construction
name: plastitanium window
id: PlastitaniumWindow
graph: Window
startNode: start
targetNode: plastitaniumWindow
category: construction-category-structures
canBuildInImpassable: true
description: Extremely hardy window to keep an eye on prisoners with little risk of them reaching you.
conditions:
- !type:EmptyOrWindowValidInTile
- !type:NoWindowsInTile
icon:
sprite: Structures/Windows/plastitanium_window.rsi
state: full
objectType: Structure
placementMode: SnapgridCenter
canRotate: false

Copy link
Collaborator

Choose a reason for hiding this comment

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

should this still be here?

Copy link
Contributor

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

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.

2 participants