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

Port undergarments from ImpStation #2719

Merged
merged 1 commit into from
Jan 14, 2025

Conversation

alterae
Copy link
Contributor

@alterae alterae commented Jan 13, 2025

All races available in the character creator may wear undergarments.
Also lets slimefolk wear chest tattoos.

About the PR

Ports the undergarment markings from ImpStation.
A pretty direct port of impstation/imp-station-14#384.
EDIT: Actually a port of space-wizards/space-station-14#31221 by way of the above.

Why / Balance

With the recent addition of surgery, characters have to be stripped much more frequently than used to be the case.
This change helps preserve some dignity.

Additionally, it just allows for a little bit more RP flavor and variety in how characters present. And it will tie in nicely with the ability to tie jumpsuit jackets around your waist once I port impstation/imp-station-14#375.

Technical details

Markings. So many markings. Almost all YAML and locale files except for, like, two six lines of C#.

Media

image
image
image
image
image
image
image
image
image

Requirements

  • I have tested all added content and changes.
  • I have added media to this PR or it does not require an ingame showcase.

Breaking changes

To my knowledge there are no breaking changes. Two enums gain new fields, but this seems to not pose any issues (unless something is relying on enum variants' numerical indicies, which I'm pretty sure isn't really a thing in C#).

Changelog
🆑

  • add: Ported undergarments from Impstation
  • tweak: Slimefolk can now wear chest tattoos

@alterae alterae requested review from a team as code owners January 13, 2025 16:36
@github-actions github-actions bot added S: Needs Review Changes: YML Changes any yml files size/L 256-1023 lines Changes: C# Changes any cs files Changes: Localization Changes any ftl files Changes: Sprite Changes any png or json in an rsi and removed S: Needs Review labels Jan 13, 2025
Copy link
Contributor

github-actions bot commented Jan 13, 2025

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

Resources/Textures/_DV/Mobs/Customization/tattoos.rsi

State Old New Status
tattoo_nanotrasen_chest Added

Resources/Textures/_Impstation/Mobs/Customization/undershirt.rsi

State Old New Status
binder Added
classic Added
gross_sleeveless Added
nanotrasen Added
rolled_undershirt Added
rolled_undershirt_sleeveless Added
sleeveless Added
sports Added
strapless Added
undershirt Added

Resources/Textures/_Impstation/Mobs/Customization/underwear.rsi

State Old New Status
boxers Added
briefs Added
lowriders Added
satin Added
tanga Added

Edit: diff updated after f6c4ad3

Copy link
Contributor

@Radezolid Radezolid left a comment

Choose a reason for hiding this comment

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

Review time!

Any files that are actually from imp station place them in their folder _Impstation

Resources/Locale/en-US/_DV/markings/tattoos.ftl Outdated Show resolved Hide resolved
Resources/Locale/en-US/_DV/markings/undershirt.ftl Outdated Show resolved Hide resolved
Resources/Locale/en-US/_DV/markings/underwear.ftl Outdated Show resolved Hide resolved
Resources/Locale/en-US/preferences/ui/markings-picker.ftl Outdated Show resolved Hide resolved
Resources/Prototypes/Entities/Mobs/Species/base.yml Outdated Show resolved Hide resolved
@dvir001
Copy link
Contributor

dvir001 commented Jan 13, 2025

space-wizards/space-station-14#33185
I like this implementation way more IMO

@alterae
Copy link
Contributor Author

alterae commented Jan 13, 2025

space-wizards/space-station-14#33185 I like this implementation way more IMO

Interesting. However, I feel that it makes some changes that are out of scope, at least for my own priorities in implementing this change.

@alterae
Copy link
Contributor Author

alterae commented Jan 13, 2025

Any files that are actually from imp station place them in their folder _Impstation

Done!

@alterae
Copy link
Contributor Author

alterae commented Jan 13, 2025

Any files that are actually from imp station place them in their folder _Impstation

Done!

ahahaha whoops forgot to update file paths hang on :elp:

@alterae
Copy link
Contributor Author

alterae commented Jan 13, 2025

ahahaha whoops forgot to update file paths hang on :elp:

THERE we go

Copy link
Contributor

@Radezolid Radezolid left a comment

Choose a reason for hiding this comment

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

I lack permits as of right now, so I'll notify direction via discord.

@alterae
Copy link
Contributor Author

alterae commented Jan 13, 2025

Interesting. However, I feel that it makes some changes that are out of scope, at least for my own priorities in implementing this change.

to expand on this, i'm also not a fan of a toggle changing how other players' characters look without their input. is this a rational distaste? idk. but i don't like the vibe

@hivehum
Copy link

hivehum commented Jan 13, 2025

hey heads up that this is a port of space-wizards/space-station-14#31221. pinkbat5 brought it to impstation and modified the sprites but the code is originally from a upstream PR. sorry about our poor attribution from our earlier features, we didnt have standards for a while

edit: foldablecomponent is also originally from a closed or in limbo upstream PR, im asking airfryer to source it properly

@alterae
Copy link
Contributor Author

alterae commented Jan 13, 2025

hey heads up that this is a port of space-wizards/space-station-14#31221. pinkbat5 brought it to impstation and modified the sprites but the code is originally from a upstream PR.

noted. does this necessitate any changes to the contents of the PR?

@hivehum
Copy link

hivehum commented Jan 13, 2025

hey heads up that this is a port of space-wizards/space-station-14#31221. pinkbat5 brought it to impstation and modified the sprites but the code is originally from a upstream PR.

noted. does this necessitate any changes to the contents of the PR?

nope, you're all good, i just wanted to make sure we werent being mistakenly credited for code we didnt work on

@deltanedas
Copy link
Member

@Radezolid join the team you were invited to :trollface:

@Radezolid
Copy link
Contributor

@Radezolid join the team you were invited to :trollface:

Lmao, didn't get the notification, thanks.

@ThataKat
Copy link
Contributor

Direction is reviewing, get back to you in about a day!

All races available in the character creator may wear undergarments.
Also lets slimefolk wear chest tattoos.
@LadyDanger2743
Copy link

Direction approval! Go ahead and hit it!

@Radezolid Radezolid merged commit 97492b3 into DeltaV-Station:master Jan 14, 2025
14 checks passed
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: Localization Changes any ftl files Changes: Sprite Changes any png or json in an rsi Changes: YML Changes any yml files S: Needs Review size/L 256-1023 lines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants