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

Update Clothing Assignments of Suit/Skirt #146

Closed
wants to merge 2 commits into from

Conversation

Tirochora
Copy link
Contributor

Description

Changes some of the parent clothing for various outfits in the game, such as the colored jumpskirts, the monastic robes, the operative jumpskirt, and miscellaneous clothing pieces that have open bottoms such as the gladiator's outfit and the kimono.


Changelog

🆑

  • fix: basic colored jumpskirts, kimonos, monastic robes, and the operative jumpskirt all now have the jumpskirt parent (operative jumpskirt is still unsensored), meaning they should be able to be worn by harpies.

@github-actions github-actions bot added Status: Needs Review Someone please review this Changes: Sprite Changes any png or json in an RSI Changes: YML Changes any yml files labels Aug 24, 2024
@Floof-Station-Bot Floof-Station-Bot changed the title Update clothing assignments of suit/skirt Update Clothing Assignments of Suit/Skirt Aug 24, 2024
Copy link
Contributor

github-actions bot commented Aug 24, 2024

RSI Diff Bot; head commit 93fd4ce merging into d07b00e
This PR makes changes to 1 or more RSIs. Here is a summary of all changes:
Edit: diff updated after 93fd4ce

Copy link
Collaborator

@FoxxoTrystan FoxxoTrystan left a comment

Choose a reason for hiding this comment

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

This is going to be a nightmare with conflict merges.

Please add comments on changes
and move all new jumpskirts to the floof directory wil commenting the old ones instead of deleting them.

@FoxxoTrystan FoxxoTrystan added Priority: 2-Medium Needs to be resolved at some point Status: Awaiting Changes Do not merge due to requested changes and removed Status: Needs Review Someone please review this labels Aug 24, 2024
@Tirochora
Copy link
Contributor Author

This is going to be a nightmare with conflict merges.

Please add comments on changes and move all new jumpskirts to the floof directory wil commenting the old ones instead of deleting them.

I'm not quite sure what you mean with 'moving all new jumpskirts to the floof directory, as there are no 'new' jumpskirts. Should I make and move the ones I modified to a separate 'jumpskirts.yml' in the floof directory?

Comment on lines +12 to +70
# Moved gladiator uniform here instead of jumpsuits.yml, changed its parent to ClothingUniformSkirtBase, and moved the sprite location to the jumpskirts directory.
- type: entity
parent: ClothingUniformSkirtBase
id: ClothingUniformJumpsuitGladiator
name: gladiator uniform
description: Made for true gladiators (or Ash Walkers).
components:
- type: Sprite
sprite: Clothing/Uniforms/Jumpskirt/gladiator.rsi
- type: Clothing
sprite: Clothing/Uniforms/Jumpskirt/gladiator.rsi

# Moved purple bartender's uniform here instead of jumpsuits.yml, changed its parent to ClothingUniformSkirtBase, and moved the sprite location to the jumpskirts directory.
- type: entity
parent: ClothingUniformSkirtBase
id: ClothingUniformJumpsuitBartenderPurple
name: purple bartender's uniform
description: A special purple outfit to serve drinks.
components:
- type: Sprite
sprite: Clothing/Uniforms/Jumpskirt/bartender_purple.rsi
- type: Clothing
sprite: Clothing/Uniforms/Jumpskirt/bartender_purple.rsi

# Moved dark monstic robe here instead of jumpsuits.yml, changed its parent to ClothingUniformSkirtBase, and moved the sprite location to the jumpskirts directory.
- type: entity
parent: ClothingUniformSkirtBase
id: ClothingUniformJumpsuitMonasticRobeDark
name: dark monastic robe
description: It's a dark robe, often worn by religious folk.
components:
- type: Sprite
sprite: Clothing/Uniforms/Jumpskirt/monastic_robe_dark.rsi
- type: Clothing
sprite: Clothing/Uniforms/Jumpskirt/monastic_robe_dark.rsi

# Moved light monastic here instead of jumpsuits.yml, changed its parent to ClothingUniformSkirtBase, and moved the sprite location to the jumpskirts directory.
- type: entity
parent: ClothingUniformSkirtBase
id: ClothingUniformJumpsuitMonasticRobeLight
name: light monastic robe
description: It's a light robe, often worn by religious folk.
components:
- type: Sprite
sprite: Clothing/Uniforms/Jumpskirt/monastic_robe_light.rsi
- type: Clothing
sprite: Clothing/Uniforms/Jumpskirt/monastic_robe_light.rsi

# Moved kimono here instead of jumpsuits.yml, changed its parent to ClothingUniformSkirtBase, and moved the sprite location to the jumpskirts directory.
- type: entity
parent: ClothingUniformSkirtBase
id: ClothingUniformJumpsuitKimono
name: kimono
description: traditional chinese clothing
components:
- type: Sprite
sprite: Clothing/Uniforms/Jumpskirt/kimono.rsi
- type: Clothing
sprite: Clothing/Uniforms/Jumpskirt/kimono.rsi
Copy link
Collaborator

Choose a reason for hiding this comment

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

This need to be moved to a floof directory instead please.

# White Jumpskirt
- type: entity
parent: ClothingUniformBase
parent: ClothingUniformSkirtBase
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
parent: ClothingUniformSkirtBase
parent: ClothingUniformSkirtBase # FloofStation

@@ -27,7 +28,7 @@

# Grey Jumpskirt
- type: entity
parent: ClothingUniformBase
parent: ClothingUniformSkirtBase
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
parent: ClothingUniformSkirtBase
parent: ClothingUniformSkirtBase # FloofStation

@@ -58,7 +59,7 @@

# Black Jumpskirt
- type: entity
parent: ClothingUniformBase
parent: ClothingUniformSkirtBase
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
parent: ClothingUniformSkirtBase
parent: ClothingUniformSkirtBase # FloofStation

@@ -89,7 +90,7 @@

# Blue Jumpskirt
- type: entity
parent: ClothingUniformBase
parent: ClothingUniformSkirtBase
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
parent: ClothingUniformSkirtBase
parent: ClothingUniformSkirtBase # FloofStation

sprite: Clothing/Uniforms/Jumpsuit/monastic_robe_light.rsi
- type: Clothing
sprite: Clothing/Uniforms/Jumpsuit/monastic_robe_light.rsi
# Moved this to jumpskirts.yml, further modifications made there.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
# Moved this to jumpskirts.yml, further modifications made there.
# Moved this to jumpskirts.yml, further modifications made there. // FloofStation

sprite: Clothing/Uniforms/Jumpsuit/gladiator.rsi
- type: Clothing
sprite: Clothing/Uniforms/Jumpsuit/gladiator.rsi
# Moved this to jumpskirts.yml, further modifications made there.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
# Moved this to jumpskirts.yml, further modifications made there.
# Moved this to jumpskirts.yml, further modifications made there. // FloofStation

@@ -1,5 +1,5 @@
- type: entity
parent: ClothingUniformBase
parent: ClothingUniformSkirtBase
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
parent: ClothingUniformSkirtBase
parent: ClothingUniformSkirtBase # FloofStation

@@ -36,6 +36,11 @@
- type: Clothing
slots: [innerclothing]
femaleMask: UniformTop
# Adds the 'skirt' tag to UnsensoredClothingUniformSkirtBase to, well, make it a skirt.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
# Adds the 'skirt' tag to UnsensoredClothingUniformSkirtBase to, well, make it a skirt.
# Adds the 'skirt' tag to UnsensoredClothingUniformSkirtBase to, well, make it a skirt. // FloofStation

@@ -66,6 +71,6 @@
slots: [innerclothing]
femaleMask: UniformTop
- type: Tag #DeltaV, needed for species with nonhuman legs/can only wear skirts
# Removed the subsequent 'skirt' tag from ClothingUniformSkirtBase, as the unsensored parent now has it.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
# Removed the subsequent 'skirt' tag from ClothingUniformSkirtBase, as the unsensored parent now has it.
# Removed the subsequent 'skirt' tag from ClothingUniformSkirtBase, as the unsensored parent now has it. // FloofStation

Also doing this kinda make this base now worthless.

@Tirochora
Copy link
Contributor Author

I forgot to close this earlier - I brought this up to EE, as there's no reason not to and it makes things simpler.

@Tirochora Tirochora closed this Sep 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: Sprite Changes any png or json in an RSI Changes: YML Changes any yml files Priority: 2-Medium Needs to be resolved at some point Status: Awaiting Changes Do not merge due to requested changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants